[Freeipa-devel] [freeipa PR#502][comment] Make pylint and jsl optional

lslebodn freeipa-github-notification at redhat.com
Wed Mar 1 09:51:22 UTC 2017


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

lslebodn commented:
"""
May I know why the default was changed without design discussion?

IIRC pscacek intentionally enabled it by default. Much better approach would be to print hint at configure time detection that it is optional and can be disabled.

Code wise; the change is too complicated (too many nested if condition ...) 
which does not improve readability of configure.ac
"""

See the full comment at https://github.com/freeipa/freeipa/pull/502#issuecomment-283294552


More information about the Freeipa-devel mailing list