Review Request 113506: Add a sb_all target and sb_$framework targets to make it easier to build frameworks standalone

Aleix Pol Gonzalez aleixpol at kde.org
Wed Nov 13 15:11:05 UTC 2013



> On Nov. 12, 2013, 5:11 p.m., Aleix Pol Gonzalez wrote:
> > CMakeLists.txt, line 18
> > <http://git.reviewboard.kde.org/r/113506/diff/5/?file=211419#file211419line18>
> >
> >     I don't think we want that, superbuild shouldn't be used after the splitting but the kde build script. We will need it anyway and it will have all the needed information anyway.
> 
> Aurélien Gâteau wrote:
>     I did this to make it as simple as possible to use superbuild: no need to run cmake again, just use the already available targets. When superbuild is removed it can go away with it so I don't think it is a problem.
> 
> Aleix Pol Gonzalez wrote:
>     I have 2 kdelibs build directories: one with monolithic and one with superbuild, I don't feel like this is a problem to me.
> 
> Aurélien Gâteau wrote:
>     Sure it is not, but you made a conscious effort to set it up.
>     
>     Furthermore, not requiring a separate build dir makes like easier for build.kde.org maintainers: they just need to add another target to the "make" call.

Well, I'd say that we probably want to have a separate install for both in bko.

I have no idea of how hard this is to set up in jenkins, if the goal is to make it possible for build.kde.org to test it, I won't oppose.


- Aleix


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113506/#review43533
-----------------------------------------------------------


On Nov. 13, 2013, 1:07 p.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113506/
> -----------------------------------------------------------
> 
> (Updated Nov. 13, 2013, 1:07 p.m.)
> 
> 
> Review request for Build System, KDE Frameworks, Alexander Neundorf, and Stephen Kelly.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> This patch rework superbuild to integrate it more closely with kdelibs build system: one no longer needs to run cmake path/to/kdelibs/superbuild separately. Instead targets are created for all frameworks declared in superbuild/CMakeLists.txt. For example to build and install ki18n standalone, one can run `make sb_ki18n`. It also adds a special "sb_all" target, which builds and install all standalone frameworks.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 879fed4 
>   superbuild/CMakeLists.txt cbe9630 
>   superbuild/README 7a4e52e 
>   superbuild/SuperBuild.cmake 33ed151 
> 
> Diff: http://git.reviewboard.kde.org/r/113506/diff/
> 
> 
> Testing
> -------
> 
> kdelibs still builds standalone, all sb_* targets work as expected.
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131113/fcd0be50/attachment.html>


More information about the Kde-frameworks-devel mailing list