> On Feb. 12, 2014, 2:52 a.m., Michael Pyne wrote:
> > kf5-applications-build-include, line 23
> > <https://git.reviewboard.kde.org/r/115632/diff/1/?file=243097#file243097line23>
> >
> >     Would be better for these 2 modules to be a 'kde-projects'-based 
> > module-set.
> >     
> >     This allows kdesrc-build to automatically select the right branch (per 
> > the branch-group setting) if the kde-build-metadata is up to date (which 
> > seems to be the case now).
> >     
> >     Additionally it allows kdesrc-build to use dependency data in 
> > kde-build-metadata to ensure the right build order.
> >     
> >     Would look something like:
> >     
> >     module-set
> >       repository kde-projects
> >       use-modules kdevplatform kdevelop/kdevelop
> >     end module-set
> >     
> >     The "kdevelop/kdevelop" is just to disambiguate, otherwise kdesrc-build 
> > would pull in "kdevelop/*" for everything not ignored in 
> > kde-build-metadata/build-script-ignore.
> >     
> >     What is here is fine too, but if the appropriate branch for kdevelop 
> > ever changes that could lead to problems later, which is why I'm trying to 
> > push back toward branch-groups.
> 
> Kevin Funk wrote:
>     Hm. So, where should I put the following lines then?
>     
>     module-set
>       repository kde-projects
>       use-modules kdevplatform kdevelop/kdevelop
>     end module-set
>     
>     This is no longer depending on 'frameworks' or anything. So is it suited 
> to be put into kdesrc-buildrc-setup or something? At least that's the place 
> where I see lots of 'module-set'-based constructs.
>     
>     It's also fine if you just commit whatever you think fits best here, 
> given that I basically have no clue about kdesrc-build. All I wanted to do is 
> to be able to 'kdesrc-build kdevelop' under KF5 :)

Sorry, I could have been more clear.

The theory is that in the "global" section of the main kdesrc-buildrc you have 
a setting called "branch-group kf5-qt5", as mentioned in the top-level comment 
of this configuration file. This tells kdesrc-build automagically which branch 
to pull (in this case, "frameworks").

In the future when the right branch to use for kdevelop becomes something else 
(presumably "master") then we the developers have to change one file in 
kde-build-metadata and then all kdesrc-build users using the branch-group 
setting will switch to the right branch on their next update.

If you have that branch-group setting in *your* .kdesrc-buildrc's global 
setting then doing "kdesrc-build kdevelop" might already work even without a 
specific configuration file command. :)

I'll go ahead and add it in at some point today.


- Michael


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115632/#review49619
-----------------------------------------------------------


On Feb. 10, 2014, 5:09 p.m., Kevin Funk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115632/
> -----------------------------------------------------------
> 
> (Updated Feb. 10, 2014, 5:09 p.m.)
> 
> 
> Review request for Build System.
> 
> 
> Repository: kdesrc-build
> 
> 
> Description
> -------
> 
> Two commits:
> 
> Add .reviewboardrc
> 
> Add frameworks-based kdevplatform + kdevelop
> 
> 
> Diffs
> -----
> 
>   .reviewboardrc PRE-CREATION 
>   kf5-applications-build-include dab8498dfd5a5f6dd7a63d8d06d1701770760b33 
> 
> Diff: https://git.reviewboard.kde.org/r/115632/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Kevin Funk
> 
>

_______________________________________________
Kde-buildsystem mailing list
Kde-buildsystem@kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem

Reply via email to