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.

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

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


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

See the full comment at 
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to