[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-11-03 Thread jhrozek
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

jhrozek commented:
"""
master: cf5357ae83cc9fe2240038b8bdccec2cb98991fc
sssd-1-14: e1c2aead482cd4bf83a7fe5e68630a981389e82b

If any of the maintainers think we should apply the patch to sssd-1-13 as well, 
please just cherry-pick the patch.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-258108034
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-11-03 Thread sumit-bose
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

sumit-bose commented:
"""
Thank you for the patch, ACK
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-258102535
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-28 Thread jhrozek
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

jhrozek commented:
"""
On Thu, Oct 27, 2016 at 08:48:46AM -0700, taupehat wrote:
> @jhrozek I did not intend to remove anything from the last iteration. It's 
> possible that I used the editor wrong. Do I need to delete the prior pull 
> request and just create a new one with all changes in place?

I normally just update the old commit and force-push the branch, github
should be able to pick up the changes. Like this:

vi path/to/file
# hack, hack, hack
# try to build the manpage with "make" to be sure the XML validates
# when finished:
git add /path/to/file
git commit --amend
git push $yourgithubfork $yourbranch --force

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-256849283
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-27 Thread taupehat
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

taupehat commented:
"""
@jhrozek I did not intend to remove anything from the last iteration. It's 
possible that I used the editor wrong. Do I need to delete the prior pull 
request and just create a new one with all changes in place?
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-256684610
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-27 Thread sumit-bose
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

sumit-bose commented:
"""
Just a small nitpick. Can you put the URL into a  element like e.g. in 
https://github.com/SSSD/sssd/blob/master/src/man/sssd-ldap.5.xml#L1047 ? This 
will make sure the URL is formatted correctly with different output format.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-256583607
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-27 Thread jhrozek
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

jhrozek commented:
"""
The documentation now looks correct, I just wonder why did you remove the 
example paragraph in the last iteration of the patch? IMO it would be useful..
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-256576929
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-27 Thread amitkumar50
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

amitkumar50 commented:
"""
Thanks Islebodn . Conflicts are resolved..
Posting What I did may help some one.

Steps for 
1. Getting source Code
2. Doing Code Changes
3. Running Integration Tests...
[root@dlp] dnf -y install git
[root@dlp] git config --global user.name "GIVENNAME SURNAME"
[root@dlp] git config --global user.email "usern...@domain.com"
[root@dlp] git config --global color.ui auto//Enable Syntax 
highlighting
[root@dlp] dnf -y install meld//Tool for resolving Merge 
conflicts
[root@dlp] git config --global merge.tool meld
[root@dlp]# git clone https://git.fedorahosted.org/git/sssd.git 
[root@dlp]# cd sssd
[root@dlp sssd]# . contrib/fedora/bashrc_sssd
[root@dlp sssd]# dnf -y install rpm-build dnf-plugins-core libldb-devel
[root@dlp sssd]# contrib/fedora/make_srpm.sh//Create source rpm
[root@dlp sssd]# dnf builddep rpmbuild/SRPMS/sssd-*.src.rpm
//Install sssd dependencies
[root@dlp sssd]# reconfig && chmake
make[1]: Leaving directory '/root/sssd/x86_64'
[root@dlp x86_64]# pwd<=See Directory Changed Automatically
/root/sssd/x86_64
[root@dlp x86_64]# 
[root@dlp x86_64]# dnf install -y openldap-servers fakeroot
[root@dlp x86_64]# vim../src/providers/ipa/access_ipa.c
//Do some changes...
[root@dlp x86_64]# make
[root@dlp x86_64]# make  intgcheck
=== 119 passed in 302.77 seconds ===
make[1]: Leaving directory '/root/sssd/x86_64'
[root@dlp x86_64]# 
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-256557508
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-26 Thread lslebodn
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

lslebodn commented:
"""
retest this please

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-256445927
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-26 Thread amitkumar50
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

amitkumar50 commented:
"""
On latest clone of project.
# make intgcheck
checking for style of include used by make... GNU
checking whether make supports nested variables... yes
configure: error: source directory already configured; run "make distclean" 
there first
Makefile:27077: recipe for target 'intgcheck-prepare' failed
make: *** [intgcheck-prepare] Error 1
# 
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-256438804
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-26 Thread taupehat
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

taupehat commented:
"""
Could we get a rebuild on this? The previous failed with an error indicating 
trouble with the build host:
make-intgcheck: Write failed: Broken pipe
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-256436456
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-26 Thread taupehat
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

taupehat commented:
"""
Could we get a rebuild on this. The previous failed with an error indicating 
trouble with the build host:
make-intgcheck: Write failed: Broken pipe
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-256436456
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-25 Thread abbra
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

abbra commented:
"""
Please use this URL: https://msdn.microsoft.com/en-us/library/cc223367.aspx
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-256004734
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-25 Thread jhrozek
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

jhrozek commented:
"""
Additionally, if there is some link to MSDN explaining what the OID is, it 
would be nice to add that link
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-255976616
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-25 Thread sumit-bose
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

sumit-bose commented:
"""
Thank you for the patch. Given the related discussion in ticket 
https://fedorahosted.org/sssd/ticket/3218 I think it would help if you can add 
a sentence saying that becasue of the ':' characters in the filter part the 
option must always use the full format with DOM or FOREST and the name.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-255974002
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-19 Thread jhrozek
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

jhrozek commented:
"""
ok to test
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-254876611
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-19 Thread centos-ci
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

centos-ci commented:
"""
Can one of the admins verify this patch?
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-254872627
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-19 Thread centos-ci
  URL: https://github.com/SSSD/sssd/pull/60
Title: #60: Document ad_access_filter search for nested groups

centos-ci commented:
"""
Can one of the admins verify this patch?
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/60#issuecomment-254872632
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org