D15070: Bindings: Support using sys paths for python install directory
Stefan BrĂ¼ns
noreply at phabricator.kde.org
Wed Oct 24 14:16:29 BST 2018
bruns added a comment.
In D15070#347945 <https://phabricator.kde.org/D15070#347945>, @cgiboudeaux wrote:
> In D15070#347944 <https://phabricator.kde.org/D15070#347944>, @bruns wrote:
>
> > So, after another week, no reason has been given not to accept this.
> >
> > 1. It fixes broken behavior on several platforms
> > 2. It does not break current setups
> > 3. It is consistent with other config variables
>
>
> That's not true, you're refusing to fix the issues. Why should we invest time reviewing your changes, exactly?
I answered this inline
> - The wrong hardcoded lib/ destination wasn't fixed
If I change this, it breaks the backwards compatibility. It currently is broken, and you have to opt-in in the current fixed behaviour.
> - The empty 'if' is still there
And I answered why it is there. Your proposals how to "fix" this leads to inconsistent behavior. Changing this in a way which keeps consistent behaviour makes the code less readable (either more nesting or longer conditions in the if statement). Policy is not there to be followed blindly.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D15070
To: bruns, #frameworks
Cc: cgiboudeaux, bcooksley, kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20181024/daaf9283/attachment.html>
More information about the Kde-buildsystem
mailing list