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

2016-10-21 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/177 Author: frasertweedale Title: #177: Add options to write lightweight CA cert or chain to file Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/177/head:pr177

[Freeipa-devel] [freeipa PR#171][+ack] Build system cleanup phase 2

2016-10-21 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/171 Title: #171: Build system cleanup phase 2 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#171][comment] Build system cleanup phase 2

2016-10-21 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/171 Title: #171: Build system cleanup phase 2 pspacek commented: """ Re-adding ACK which was removed while we were "resolving" our disagreement with Lukas. """ See the full comment at https://github.com/freeipa/freeipa/pull/171#issuecomment-2554080

[Freeipa-devel] [freeipa PR#171][comment] Build system cleanup phase 2

2016-10-21 Thread lslebodn
URL: https://github.com/freeipa/freeipa/pull/171 Title: #171: Build system cleanup phase 2 lslebodn commented: """ We (lslebodn and pspacek) agree that we disagree about maintainability of client_only in one monolithic configure.ac. But we agreed that the refactoring of build system[1] is more

[Freeipa-devel] [freeipa PR#148][+ack] Unaccessible variable self.attrs in Tracker

2016-10-21 Thread apophys
URL: https://github.com/freeipa/freeipa/pull/148 Title: #148: Unaccessible variable self.attrs in Tracker 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/Contribu

[Freeipa-devel] [freeipa PR#148][comment] Unaccessible variable self.attrs in Tracker

2016-10-21 Thread apophys
URL: https://github.com/freeipa/freeipa/pull/148 Title: #148: Unaccessible variable self.attrs in Tracker apophys commented: """ Thank you, ack. """ See the full comment at https://github.com/freeipa/freeipa/pull/148#issuecomment-255363641 -- Manage your subscription for the Freeipa-devel mai

[Freeipa-devel] [freeipa PR#178][opened] ipatests: Fix assert_deepequal outside of pytest process

2016-10-21 Thread apophys
URL: https://github.com/freeipa/freeipa/pull/178 Author: apophys Title: #178: ipatests: Fix assert_deepequal outside of pytest process Action: opened PR body: """ https://fedorahosted.org/freeipa/ticket/6420 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa

[Freeipa-devel] [freeipa PR#174][comment] add log module

2016-10-21 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/174 Title: #174: add log module mbasti-rh commented: """ > Yes. As you see, our use case is I'm admin and want to audit what commads > were used. > In this case, if it is only for admins I endorse you to use centralized logging: - http://www.freeip

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

2016-10-21 Thread Oleg Fayans
Added one more test, resolved the pep8 issues On 10/19/2016 12:32 PM, Oleg Fayans wrote: Hi Martin, As you suggested, I've extended the test_xmlrpc/test_add_remove_cert_cmd.py to contain basic tests for certs in idoverrides. The integration part still needs some polishing in the part related to

Re: [Freeipa-devel] [PATCH] 0097 Add options to write lightweight CA cert or chain to file

2016-10-21 Thread Fraser Tweedale
Patches have been reborn as https://github.com/freeipa/freeipa/pull/177. Brief commentary inline. If any further issues, let us continue discussion at GitHub. Thanks, Fraser On Thu, Oct 06, 2016 at 10:02:55AM +0200, Jan Cholasta wrote: > On 23.9.2016 05:29, Fraser Tweedale wrote: > > Bump for r

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

2016-10-21 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/177 Author: frasertweedale Title: #177: Add options to write lightweight CA cert or chain to file Action: opened PR body: """ Administrators need a way to retrieve the certificate or certificate chain of an IPA-managed lightweight CA. Add params to

[Freeipa-devel] [freeipa PR#171][comment] Build system cleanup phase 2

2016-10-21 Thread lslebodn
URL: https://github.com/freeipa/freeipa/pull/171 Title: #171: Build system cleanup phase 2 lslebodn commented: """ On (21/10/16 00:41), Petr Špaček wrote: >> Correct me If I am wrong. The configure script is and will be used >just for detection of build dependencies for C-code. > >Maybe thi

[Freeipa-devel] [freeipa PR#171][comment] Build system cleanup phase 2

2016-10-21 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/171 Title: #171: Build system cleanup phase 2 pspacek commented: """ > Correct me If I am wrong. The configure script is and will be used just for detection of build dependencies for C-code. Maybe this is why we do not understand each other. Autotoo

[Freeipa-devel] [freeipa PR#171][comment] Build system cleanup phase 2

2016-10-21 Thread lslebodn
URL: https://github.com/freeipa/freeipa/pull/171 Title: #171: Build system cleanup phase 2 lslebodn commented: """ On (20/10/16 10:29), Petr Špaček wrote: >@lslebodn I'm really trying to explain this but I'm still not able to get the >point across. >> My concerns are related purely to C-code.

[Freeipa-devel] [freeipa PR#171][comment] Build system cleanup phase 2

2016-10-21 Thread lslebodn
URL: https://github.com/freeipa/freeipa/pull/171 Title: #171: Build system cleanup phase 2 lslebodn commented: """ On (20/10/16 10:29), Petr Špaček wrote: >@lslebodn I'm really trying to explain this but I'm still not able to get the >point across. >> My concerns are related purely to C-code.