Review Request 127910: kdesrc-build: improve error messages by showing the right filename

David Faure faure at kde.org
Sun May 15 12:56:01 UTC 2016


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

(Updated May 15, 2016, 12:56 p.m.)


Review request for Build System and Michael Pyne.


Changes
-------

Use a stack of filenames. Not exactly the solution you had in mind, but the "basepath" stuff could be refactored to use this new stack if wanted, pushBasePath being internal.


Repository: kdesrc-build


Description
-------

Before:
"Don't use module libaccounts-qt on line 20 of /path/kdesrc-buildrc, use options libaccounts-qt"
 but line 20 is unrelated, some global option.

After:
"Don't use module libaccounts-qt on line 20 of /path/extragear/utils/kdesrc-build/kf5-workspace-build-include, use options libaccounts-qt"


Diffs (updated)
-----

  modules/ksb/Application.pm 5dbd224c7ba06242b53cd77cfa0764c28da76579 
  modules/ksb/RecursiveFH.pm 6892320bb5e8f8c1e4979ef137bb975775940908 

Diff: https://git.reviewboard.kde.org/r/127910/diff/


Testing
-------


Thanks,

David Faure

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


More information about the Kde-buildsystem mailing list