Review Request 124613: kdesrc-build: improve message about why a full refresh is needed

Michael Pyne mpyne at kde.org
Sun Aug 9 14:42:29 UTC 2015



> On Aug. 9, 2015, 9:34 a.m., David Faure wrote:
> > modules/ksb/BuildSystem.pm, line 73
> > <https://git.reviewboard.kde.org/r/124613/diff/1/?file=389799#file389799line73>
> >
> >     Are you sure? I found code in Module.pm line 509 that creates a .refresh-me file right after printing out "Unable to configure".

Oh, good catch. .refresh-me was added for the purpose I described but I see that I did take advantage of that elsewhere. Since a user is unlikely to ever use .refresh-me themselves I think you're right here after all, I withdraw the issue. Though perhaps a comment pointing to Module.pm might be appropriate if this even caught myself. :)


> On Aug. 9, 2015, 9:34 a.m., David Faure wrote:
> > modules/ksb/BuildSystem.pm, line 79
> > <https://git.reviewboard.kde.org/r/124613/diff/1/?file=389799#file389799line79>
> >
> >     what does // 0 mean btw?

This returns the value 0 if the expression on the left is undefined. similar to "$foo || 0" except that "|| 0" also works when $foo is defined-but-boolean-false.

This is just quick inline validation that avoids annoying Perl warnings about comparison with undefined values.


> On Aug. 9, 2015, 9:34 a.m., David Faure wrote:
> > modules/ksb/BuildSystem.pm, line 82
> > <https://git.reviewboard.kde.org/r/124613/diff/1/?file=389799#file389799line82>
> >
> >     Well, either that or the user manually wiped out the contents of the builddir (but not the builddir itself).
> >     (or kdesrc-build looks for the wrong file, in case of a bug).
> >     
> >     I'd like to keep the output as precise as possible, so one can figure out why exactly this is happening...

Fair enough, I withdraw the issue.


- Michael


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


On Aug. 9, 2015, 9:36 a.m., David Faure wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124613/
> -----------------------------------------------------------
> 
> (Updated Aug. 9, 2015, 9:36 a.m.)
> 
> 
> Review request for Build System and Michael Pyne.
> 
> 
> Repository: kdesrc-build
> 
> 
> Description
> -------
> 
> i.e. be more precise than "meets other building criteria"
> 
> REVIEW: 124613
> 
> 
> Diffs
> -----
> 
>   modules/ksb/IPC.pm 0e55c5fe5c2a8771f6b9b159a684bdcc6dda52a7 
>   modules/ksb/Module.pm a7a7fbcc5e30cbb4c52979d87407a3250b80f2b2 
>   modules/ksb/l10nSystem.pm e5756f85026ad9b95533d2989746ab91f87920a6 
>   modules/ksb/BuildSystem.pm 24963040d77dc93f8f86a2f19755c617713bbd8a 
> 
> Diff: https://git.reviewboard.kde.org/r/124613/diff/
> 
> 
> Testing
> -------
> 
> Example:
> 
> $ kdesrc-build --refresh plasma-nm
> 
> Building plasma-nm from kf5-network-management (1/1)
>         Updating plasma-nm (to branch master)
>         No source update, but the option refresh-build was set
>         Source update complete for plasma-nm: no files affected
>         Preparing build system for plasma-nm.
> ...
> 
> 
> Thanks,
> 
> David Faure
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20150809/099f0d61/attachment.html>


More information about the Kde-buildsystem mailing list