[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 tiran commented: """ memo for me: - [ ] /freeipa*.tar.gz is not removed - [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__ and other pyc/pyo. add ```clean-local: rm -rf

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 tiran commented: """ memo for me: - [ ] /freeipa*.tar.gz is not removed - [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__ and other pyc/pyo. add ```clean-local: rm -rf

[Freeipa-devel] [freeipa PR#213][+ack] Build system refactoring phase 3

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 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/Code

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 tiran commented: """ The patch has some minor creases but works. Let's merge it to master and iron out the remaining small issues with PRs. """ See the full comment at

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 tiran commented: """ memo for me: - [ ] /freeipa*.tar.gz is not removed - [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__ and other pyc/pyo. add ```clean-local: rm -rf

[Freeipa-devel] [freeipa PR#213][+ack] Build system refactoring phase 3

2016-11-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 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/Code

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 mbasti-rh commented: """ ACK """ See the full comment at https://github.com/freeipa/freeipa/pull/213#issuecomment-259399255 -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 pspacek commented: """ @tiran > autoconf and automake files are not removed (Makefile.in, /config.sub ...) According to [Automake manual section 13 What Gets

[Freeipa-devel] [freeipa PR#220][opened] Build: fix make clean to remove build artifacts from top-level directory

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/220 Author: pspacek Title: #220: Build: fix make clean to remove build artifacts from top-level directory Action: opened PR body: """ make lint and make dist were generating files which were not removed by make clean.

[Freeipa-devel] Build system refactoring was pushed to master

2016-11-09 Thread Petr Spacek
Hi FreeIPA gang, we just pushed Build system refactoring to master. Most visible change is that you need to use command "./makerpms.sh" instead of "make rpms" when building FreeIPA from clean Git tree. "make rpms" will work as usual after initial autoreconf -i && ./configure combo so this

[Freeipa-devel] [freeipa PR#195][edited] [WIP] Make ipaclient pip install-able

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/195 Author: tiran Title: #195: [WIP] Make ipaclient pip install-able Action: edited Changed field: body Original value: """ ## proof of concept This makes ipaclient and dependencies pip install-able by adding install requirements to all

[Freeipa-devel] [freeipa PR#216][+pushed] libexec scripts: ldap conn management

2016-11-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/216 Title: #216: libexec scripts: ldap conn management Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/c48e5fd811326dc64e19490f88003e442815a052

[Freeipa-devel] [freeipa PR#213][+pushed] Build system refactoring phase 3

2016-11-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 Label: +pushed -- 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/Code

[Freeipa-devel] [freeipa PR#213][closed] Build system refactoring phase 3

2016-11-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/213 Author: pspacek Title: #213: Build system refactoring phase 3 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/213/head:pr213 git checkout pr213 -- Manage your

[Freeipa-devel] [freeipa PR#217][synchronized] change certificate processing code to use python-cryptography

2016-11-09 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/217 Author: frasertweedale Title: #217: change certificate processing code to use python-cryptography Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#217][comment] change certificate processing code to use python-cryptography

2016-11-09 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/217 Title: #217: change certificate processing code to use python-cryptography frasertweedale commented: """ The travis-ci failure is due to two minor pep8 violations, which I intend :) """ See the full comment at

[Freeipa-devel] [freeipa PR#195][edited] [WIP] Make ipaclient pip install-able

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/195 Author: tiran Title: #195: [WIP] Make ipaclient pip install-able Action: edited Changed field: body Original value: """ ## proof of concept This makes ipaclient and dependencies pip install-able by adding install requirements to all

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 pspacek commented: """ @tiran > autoconf and automake files are not removed (Makefile.in, /config.sub ...) According to [Automake manual section 13 What Gets

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 tiran commented: """ memo for me: - [ ] /freeipa*.tar.gz is not removed - [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__ and other pyc/pyo. add ```clean-local: rm -rf

[Freeipa-devel] [freeipa PR#216][+ack] libexec scripts: ldap conn management

2016-11-09 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/216 Title: #216: libexec scripts: ldap conn management Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#216][comment] libexec scripts: ldap conn management

2016-11-09 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/216 Title: #216: libexec scripts: ldap conn management flo-renaud commented: """ Thanks for the update. Works for me. """ See the full comment at https://github.com/freeipa/freeipa/pull/216#issuecomment-259406309 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#216][comment] libexec scripts: ldap conn management

2016-11-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/216 Title: #216: libexec scripts: ldap conn management mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/33f7b8dc32bc95e0db067ac4df49807ee2b5120e """ See the full comment at

[Freeipa-devel] [freeipa PR#216][closed] libexec scripts: ldap conn management

2016-11-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/216 Author: tomaskrizek Title: #216: libexec scripts: ldap conn management Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/216/head:pr216 git checkout pr216 --

Re: [Freeipa-devel] [Test][Patch-0049, 0050] Certs in ID overrides test

2016-11-09 Thread Milan Kubík
On 10/25/2016 10:24 AM, Oleg Fayans wrote: Integration part of the tests is ready. 2 tests: 1. Adds a cert to idoverride of a windows user 2. sssd part - looks up user by his certificate using dbus-sssd Second and third dbus call are executed as a string insted of as array of strings because

[Freeipa-devel] [freeipa PR#188][closed] Move Python egg-info to top level directory

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/188 Author: tiran Title: #188: Move Python egg-info to top level directory Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/188/head:pr188 git checkout pr188 --

[Freeipa-devel] [freeipa PR#188][comment] Move Python egg-info to top level directory

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/188 Title: #188: Move Python egg-info to top level directory tiran commented: """ I'm closing this PR because the issue can't be fixed w/o patching setuptools heavily. I got it mostly right except for SOURCES.txt. We need to revise the project

[Freeipa-devel] [freeipa PR#187][synchronized] Register entry points of Custodia plugins

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/187 Author: tiran Title: #187: Register entry points of Custodia plugins Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/187/head:pr187 git checkout pr187 From

[Freeipa-devel] [freeipa PR#182][synchronized] Use env var IPA_CONFDIR to get confdir for 'cli' context

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/182 Author: tiran Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/182/head:pr182 git

[Freeipa-devel] [freeipa PR#218][+ack] test_ipagetkeytab: use system-wide IPA CA cert location in tests

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/218 Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

Re: [Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

2016-11-09 Thread Milan Kubík
On 11/03/2016 04:56 PM, Oleg Fayans wrote: Hi Martin, The commit message was updated with the correct ticket link Thanks for review! On 11/03/2016 04:22 PM, Martin Basti wrote: almost ACK, but the ticket in commit message is closed as invalid. So I'm quite puzzled now what to do. On

[Freeipa-devel] [freeipa PR#222][opened] Fix ipa-replica-install when upgrade from ca-less to ca-full

2016-11-09 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/222 Author: flo-renaud Title: #222: Fix ipa-replica-install when upgrade from ca-less to ca-full Action: opened PR body: """ When ipa-replica-prepare is run on a master upgraded from CA-less to CA-full, it creates the replica file with a copy of

[Freeipa-devel] [freeipa PR#223][opened] LDAP refactoring: remove admin_conn

2016-11-09 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/223 Author: tomaskrizek Title: #223: LDAP refactoring: remove admin_conn Action: opened PR body: """ This first commit removes the admin_conn alias for api.Backend.ldap2 that was previously used in services. When trying to get rid of it, I found

[Freeipa-devel] [freeipa PR#197][synchronized] Make setup.py files PyPI compatible

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/197 Author: tiran Title: #197: Make setup.py files PyPI compatible Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/197/head:pr197 git checkout pr197 From

[Freeipa-devel] [freeipa PR#182][comment] Use env var IPA_CONFDIR to get confdir for 'cli' context

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/182 Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context tiran commented: """ ipapython.admintool.ScriptError still prints the full traceback: ``` $ IPA_CONFDIR=/tmp/ipa ./ipa [2016-11-09T16:35:38Z ipa] : ScriptError: IPA_CONFDIR

[Freeipa-devel] [freeipa PR#218][comment] test_ipagetkeytab: use system-wide IPA CA cert location in tests

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/218 Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests pspacek commented: """ Jenkins tests now pass. """ See the full comment at https://github.com/freeipa/freeipa/pull/218#issuecomment-259439106 -- Manage your

[Freeipa-devel] [freeipa PR#221][opened] gitignore: ignore tar ball

2016-11-09 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/221 Author: tomaskrizek Title: #221: gitignore: ignore tar ball Action: opened PR body: """ Add tar ball generated by build to gitignore. https://fedorahosted.org/freeipa/ticket/6418 """ To pull the PR as Git branch: git remote add ghfreeipa

[Freeipa-devel] [freeipa PR#218][closed] test_ipagetkeytab: use system-wide IPA CA cert location in tests

2016-11-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/218 Author: martbab Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/218/head:pr218 git

[Freeipa-devel] [freeipa PR#218][+pushed] test_ipagetkeytab: use system-wide IPA CA cert location in tests

2016-11-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/218 Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#218][comment] test_ipagetkeytab: use system-wide IPA CA cert location in tests

2016-11-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/218 Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/3ecda74d14066f6609d72422041bcc0c6499de77 """ See the full comment at

[Freeipa-devel] [freeipa PR#143][synchronized] Issue6386 nss dir

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/143 Author: tiran Title: #143: Issue6386 nss dir Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/143/head:pr143 git checkout pr143 From

[Freeipa-devel] [freeipa PR#143][comment] Issue6386 nss dir

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/143 Title: #143: Issue6386 nss dir tiran commented: """ I have fixed all places that don't depend on hard-coded paths. The other places are used for client enrolment and depend on hard-coded paths for sysrestore. Some places use the path before

[Freeipa-devel] [freeipa PR#215][comment] Add script to setup krb5 NFS exports

2016-11-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/215 Title: #215: Add script to setup krb5 NFS exports mbasti-rh commented: """ Hello, can you please provide user cases for this, or some kind of top level design what problem are you solving, why and how do you plan resolve it? We may help you

Re: [Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

2016-11-09 Thread Milan Kubík
On 11/09/2016 04:34 PM, Milan Kubík wrote: On 11/03/2016 04:56 PM, Oleg Fayans wrote: Hi Martin, The commit message was updated with the correct ticket link Thanks for review! On 11/03/2016 04:22 PM, Martin Basti wrote: almost ACK, but the ticket in commit message is closed as invalid. So

[Freeipa-devel] [freeipa PR#180][synchronized] Make api.env.nss_dir relative to api.env.confdir

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/180 Author: tiran Title: #180: Make api.env.nss_dir relative to api.env.confdir Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/180/head:pr180 git checkout

[Freeipa-devel] [freeipa PR#195][synchronized] [WIP] Make ipaclient pip install-able

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/195 Author: tiran Title: #195: [WIP] Make ipaclient pip install-able Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/195/head:pr195 git checkout pr195 From

[Freeipa-devel] [freeipa PR#195][synchronized] [WIP] Make ipaclient pip install-able

2016-11-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/195 Author: tiran Title: #195: [WIP] Make ipaclient pip install-able Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/195/head:pr195 git checkout pr195 From

[Freeipa-devel] [freeipa PR#217][synchronized] change certificate processing code to use python-cryptography

2016-11-09 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/217 Author: frasertweedale Title: #217: change certificate processing code to use python-cryptography Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#217][opened] change certificate processing code to use python-cryptography

2016-11-09 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/217 Author: frasertweedale Title: #217: change certificate processing code to use python-cryptography Action: opened PR body: """ This commit changes certificate processing code to use python-cryptography instead of NSS. Part of the refactoring

[Freeipa-devel] [freeipa PR#177][comment] Add options to write lightweight CA cert or chain to file

2016-11-09 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/177 Title: #177: Add options to write lightweight CA cert or chain to file frasertweedale commented: """ Bump for review """ See the full comment at https://github.com/freeipa/freeipa/pull/177#issuecomment-259387372 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#218][opened] test_ipagetkeytab: use system-wide IPA CA cert location in tests

2016-11-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/218 Author: martbab Title: #218: test_ipagetkeytab: use system-wide IPA CA cert location in tests Action: opened PR body: """ Since /etc/ipa/ca.crt should be always present on the test runner, we should use it in bind method tests and not rely on

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 tiran commented: """ memo for me: - [ ] /freeipa*.tar.gz is not removed - [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__ and other pyc/pyo. add ```clean-local: rm -rf

[Freeipa-devel] [freeipa PR#219][opened] Refactor installer code requesting certificates

2016-11-09 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/219 Author: flo-renaud Title: #219: Refactor installer code requesting certificates Action: opened PR body: """ With this PR, the certificates requested during server installation are now consistently obtained through certmonger (applies to

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 pspacek commented: """ @tiran > add ipasetup.py to dist_noinst_SCRIPTS ? `ipasetup.py` file is auto-generated from `ipasetup.py.in` so it should not be part of distibution tarball. I've marked this

[Freeipa-devel] [freeipa PR#216][synchronized] libexec scripts: ldap conn management

2016-11-09 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/216 Author: tomaskrizek Title: #216: libexec scripts: ldap conn management Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/216/head:pr216 git checkout pr216

[Freeipa-devel] [freeipa PR#219][edited] Refactor installer code requesting certificates

2016-11-09 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/219 Author: flo-renaud Title: #219: Refactor installer code requesting certificates Action: edited Changed field: body Original value: """ With this PR, the certificates requested during server installation are now consistently obtained through

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 pspacek commented: """ @tiran I've tested the find command and it works. The trick is `-o` which acts like OR and allows you to specify different conditions and associate different actions to them.

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 tiran commented: """ memo for me: - [ ] /freeipa*.tar.gz is not removed - [ ] ```MOSTLYCLEANFILES``` only cleans ipasetup.py[co] but keeps __pycache__ and other pyc/pyo. add ```clean-local: rm -rf

[Freeipa-devel] [freeipa PR#221][+ack] gitignore: ignore tar ball

2016-11-09 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/221 Title: #221: gitignore: ignore tar ball 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/Code

[Freeipa-devel] [freeipa PR#187][comment] Register entry points of Custodia plugins

2016-11-09 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/187 Title: #187: Register entry points of Custodia plugins jcholast commented: """ Just curious, why is the store class called `iSecStore` and not `IPASecStore` or something similar to make it apparent that it comes from IPA? """ See the full

[Freeipa-devel] [freeipa PR#219][comment] Refactor installer code requesting certificates

2016-11-09 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/219 Title: #219: Refactor installer code requesting certificates frasertweedale commented: """ Although there are conflicts with `master`, there are problems when the patches are rebased. Server installation (CA-ful) fails when requesting the RA

[Freeipa-devel] [freeipa PR#219][comment] Refactor installer code requesting certificates

2016-11-09 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/219 Title: #219: Refactor installer code requesting certificates frasertweedale commented: """ Although there are no conflicts with `master`, there are problems when the patches are rebased. Server installation (CA-ful) fails when requesting the RA

[Freeipa-devel] [freeipa PR#213][comment] Build system refactoring phase 3

2016-11-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/213 Title: #213: Build system refactoring phase 3 mbasti-rh commented: """ commit `Build: add rpms target and makerpms.sh script` misses makerpms.sh and ticket, and it looks like you forgot to squash this commit """ See the full comment at

[Freeipa-devel] [freeipa PR#216][opened] libexec scripts: ldap conn management

2016-11-09 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/216 Author: tomaskrizek Title: #216: libexec scripts: ldap conn management Action: opened PR body: """ Certificate renewal scripts require connection to LDAP. Properly handle connects and disconnects from LDAP.