-
Notifications
You must be signed in to change notification settings - Fork 251
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecate and make support of 'ad_allow_remote_domain_local_groups' #7741
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
[ AC_ARG_WITH([allow-remote-domain-local-groups], | ||
[AC_HELP_STRING([--with-allow-remote-domain-local-groups], | ||
[Whether to build support of ad_allow_remote_domain_local_groups | ||
sssd.conf option [no].] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi,
I would like to suggest to mention here as well that the option is deprecated.
bye,
Sumit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed (and rebased).
sssd.conf option conditional :config: 'ad_allow_remote_domain_local_groups' option is deprecated and will be removed in future releases. :packaging: Support of deprecated 'ad_allow_remote_domain_local_groups' sssd.conf option isn't built by default. It can be enabled using '--with-allow-remote-domain-local-groups' ./configure option.
b1ec38c
to
9480d55
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi,
thank you for the update, ACK.
bye,
Sumit
sssd.conf option conditional
:config: 'ad_allow_remote_domain_local_groups' option is deprecated and will be removed in future releases.
:packaging: Support of deprecated 'ad_allow_remote_domain_local_groups' sssd.conf option isn't built by default. It can be enabled using '--with-allow-remote-domain-local-groups' ./configure option.