URL: https://github.com/freeipa/freeipa/pull/69
Title: #69: Fix ipa-replica-install with RHEL 6.8 master
Label: +rejected
--
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/Cont
URL: https://github.com/freeipa/freeipa/pull/69
Author: flo-renaud
Title: #69: Fix ipa-replica-install with RHEL 6.8 master
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/69/head:pr69
git checkout pr69
--
Manag
URL: https://github.com/freeipa/freeipa/pull/69
Title: #69: Fix ipa-replica-install with RHEL 6.8 master
flo-renaud commented:
"""
Please ignore this PR as the issue has been fixed in IPA 3.0 (in
ipa-replica-prepare).
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/69#issu
URL: https://github.com/freeipa/freeipa/pull/69
Title: #69: Fix ipa-replica-install with RHEL 6.8 master
pvoborni commented:
"""
Is it still needed? Seems that fixing IPA 3.0 fixes the issue.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/69#issuecomment-247603640
--
Mana
URL: https://github.com/freeipa/freeipa/pull/82
Author: mbasti-rh
Title: #82: Fix regexp in user/group name
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/82/head:pr82
git checkout pr82
From 15f781fdda2a09
URL: https://github.com/freeipa/freeipa/pull/89
Title: #89: client: remove hard dependency on pam_krb5
stlaz commented:
"""
Works as expected.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/89#issuecomment-247579650
--
Manage your subscription for the Freeipa-devel mailin
URL: https://github.com/freeipa/freeipa/pull/89
Title: #89: client: remove hard dependency on pam_krb5
Label: +ack
--
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/
URL: https://github.com/freeipa/freeipa/pull/82
Title: #82: Fix regexp in user/group name
stlaz commented:
"""
Sure. But from what I see, this patch may possibly break idviews plugin for
users of username of about 255 characters (see idoverrideuser - 'uid'), same
for groups (with 'cn' ofc).
NA
abbra commented on a pull request
"""
'uid' in user object and 'cn' in group object have meaning in POSIX
environments. 'cn' in other objects is not subject for strict limits.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/82#issuecomment-247541965
--
Manage your subscrip
stlaz commented on a pull request
"""
Would the gist of this patch also apply to any of idviews, servicedelegation or
topology plugins where a similar or slightly different regexp is used? Could we
rather have the regexp as a constant somewhere so it does not have to be
changed at all places it
Sorry for stealing your thread, but you started asking about github
review emails :)
Standard review inline comments are disabled on purpose, each comment
generates one email, so we decided that is better after review to write
a regular comment "NACK, please see inline comments" or so.
I wo
Akasurde's pull request #83: "Added a fix for setting Priority as required
field " label *ack* has been added
See the full pull-request at https://github.com/freeipa/freeipa/pull/83
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-de
Akasurde's pull request #83: "Added a fix for setting Priority as required
field " was synchronized
See the full pull-request at https://github.com/freeipa/freeipa/pull/83
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/83/hea
13 matches
Mail list logo