URL: https://github.com/freeipa/freeipa/pull/502
Title: #502: Make pylint and jsl optional

lslebodn commented:
"""
It was explained on IRC
> < cheimes> lslebodn: Your proposal is missing the point of the ticket. It 
> doesn't not simplify
>                 building, but rather improves error messages. The whole point 
> of the ticket is a more
>                 pleasant experience for outsiders that are not FreeIPA core 
> contributors.

But there is main problem with this PR. The design document expected these 
option enabled by default.
http://www.freeipa.org/page/V4/Build_system_refactoring

This is a reason why I mentioned to log just a hint for optional *lint 
dependencies.

And ticket https://pagure.io/freeipa/issue/6604 says that options were made 
optional a moth ago

master:

*    5c18feaa206bbaee692fc3640b7b79c8d9d6a638 CONFIGURE: Fix detection of pylint
*    3f91469f327d8d9f3b27e0b67c54a4f47ad845c1 CONFIGURE: Update help message 
for jslint
*    b82d285a4a75e11cc9291ecca12d2fcc26f43ed1 SPEC: Fix build in mock
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/502#issuecomment-283335089
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to