On Thursday 17 November 2011 00:33:44 Robert Xu wrote:
> On Wed, Nov 16, 2011 at 18:16, Andrea Cascio <andrea@...> wrote:
> > So the right repo is tde3.5.13, not the one that will "destroy my cat",
> > ok? ;)
> 
> Yup ;) Right now qt3 is only in there >_> and there are patches to
> remove from it.
> However, arts, libtqt4, and the dbus bindings are finished and tested,
> so that can be uploaded ASAP (when qt3 succeeds in building).
> 
> > Yes, got it. Now, I hope you will advice me, regarding the correct way to
> > work with it. Should I make my own branch or track your suse branch or
> > work on your branch locally than sending you the patches? Sorry if the
> > question sounds stupid but I'm just a beginner with GIT.
> 
> Don't worry, no question is stupid :)
> You can track my suse branch, makes life so much easier :)
> I use Eclipse with the RPM Spec file editor plugin, so you might see
> some metadata files there; feel free to ignore them.
> (I have on record everything to tdebase (kdebase), which means other
> core apps can be packaged. I have created the directories necessary
> for them already, so all that's left is importing and modifying)
> 
> In .gitignore, I set ignore to backup files (~), _service:*, and .osc,
> so if you initialize osc in any of the directories, rest assured,
> nothing will be sent to git :)
> 
> There is a package called tde-packaging that I will push ASAP. It
> contains all your %_tde_prefix and %_tde_bindir and handy directory
> macros that makes packaging so much easier.
> In addition, when you are calling cmake, use the following:
> 
> %cmake_tde -d build -- -DSOMEOPTION=ON
> 
> %make_tde -d build
> 
> %makeinstall_tde -d build
> 
> This ensures that everything is properly built in the right prefixes
> and build directories. :)
> No need to set cmake prefix or anything, that is what %cmake_tde is for.
> (btw, if you want to specify any extra cmake options, you need the --)
> 
> > Still, there are much more patches than I expected, expecially in core
> > packages. Oh well, more patches, more glory. :)
> > Something to start with: if I understand correctly "qt3-3.3.8d" is
> > absolutely required, the qt3-3.3.8c maintained in the standard openSUSE
> > repo won't do the job, right? Do you mind if I try to get a working
> > package for it, as a start? The one on the OBS does not appear to build.
> 
> Sure. I started removing some of the "already applied" patches, feel
> free to finish that.

Many thanks for the help Robert!
Ok, now I finally have a coherent set of patches. Many of them are just for 
specific builds, but it seems to me that some should be included in the main 
tree. Anyway, I don't know the codebase enough. :) 

Just 2 questions left:
- are the translations in qt3-3.3.8b-translations still good for 3.3.8d?
- it seems that the patch gcc46.diff is only halfway applied in the tarball of 
3.3.8d. The part which includes <cstddef> is there, but the part wich converts 
ptrdiff_t into std::ptrdiff_t is not. I'd keep the patch cutting away the 
first part, ok?

Tomorrow I will try to get a set of successful OBS builds, let's start 
populating that repo. :)

Andrea