Superbuild in kdelibs

[Aurélien Gâteau]

Le vendredi 22 novembre 2013 19:55:52 Ben Cooksley a écrit :
> On Fri, Nov 22, 2013 at 3:52 AM, Aurélien Gâteau <agateau at kde.org> wrote:
> > Hi,
> 
> Hi,
> 
> > This morning I merged my superbuild changes in the frameworks branch.
> > 
> > Superbuild support was originally enabled by default, but I turned it
> > off because it broke on build.kde.org. You can turn it on by passing
> > -DSUPERBUILD=ON to cmake. When superbuild is enabled, you can build any
> > framework standalone with `make sb_$framework`. This will:
> > 
> > - create a separate build dir for the framework in
> > 
> >   $build/superbuild/$framework
> > 
> > - run cmake in this dir
> > - run make
> > - run make install
> > 
> > You can also build all frameworks standalone at once with `make sb_all`.
> > 
> > Regarding build.kde.org: build failed because it defines a DESTDIR
> > environment variable but does not set CMAKE_SKIP_RPATH, which superbuild
> > 
> > does not like:
> >     CMake Error at superbuild/SuperBuild.cmake:61 (message):
> >       The DESTDIR environment variable is set to
> >       "/srv/jenkins/workspace/kdelibs_frameworks_qt5/install", but
> >       CMAKE_SKIP_RPATH is not set to TRUE.  This would produce binaries
> > 
> > with bad
> > 
> >       RPATHs.
> >     
> >     Call Stack (most recent call first):
> >       superbuild/CMakeLists.txt:7 (include)
> > 
> > I am not sure what is the best approach to this, advices are welcome.
> > 
> > 
> > Once this is sorted out, I would like to see build.kde.org run `make
> > sb_all` before running the classic `make` command so that we can catch
> > standalone breakage.
> 
> We can do that.
> 
> In terms of CMAKE_SKIP_RPATH, that sounds like a bad idea as it will
> probably break things like meinproc5 for the regular (non-Super) build.
> I've no idea why it is complaining about DESTDIR being set - it should have
> no effect as far as I know, but i'm no expert in these matters.
> 
> I'd suggest disabling this check if we can, but i'll defer to those who
> know more in the area of RPATH and equivalent variables.
> Up to this point it has not caused us any issues wrt execution of tests for
> any of the binaries used on the CI system - otherwise CMake itself,
> kdeinit, kded, etc wouldn't work.

I just disabled it, so that you can give it a try. Can you modify kdelibs job 
definition so that cmake is called with -DSUPERBUILD=ON and "make sb_all" is 
called first, then "make"?

Aurélien