On 2016-03-18 10:22, Martin Basti wrote: > > > On 29.02.2016 16:02, David Kupka wrote: >> Hello Christian, >> sorry for letting this patch rot for so long. I've forget about it the >> minute Fraser replied. >> To compensate a little I've fixed pep8 error, rebased it and attaching two >> versions for master and for 4.3 branch. >> I haven't found any missing cases and it works for me. If you're OK with the >> modified patches it can be pushed. >> >> David >> >> ----- Original Message ----- >> From: "Christian Heimes" <chei...@redhat.com> >> To: "Fraser Tweedale" <ftwee...@redhat.com> >> Cc: "freeipa-devel" <email@example.com> >> Sent: Wednesday, January 20, 2016 11:57:42 AM >> Subject: Re: [Freeipa-devel] [PATCH 0029] Move user/group constants for PKI >> and DS into ipaplatform >> >> On 2016-01-20 02:54, Fraser Tweedale wrote: >>> On Tue, Jan 19, 2016 at 02:20:27PM +0100, Christian Heimes wrote: >>>> ipaplatform.constants has platform specific names for a couple of system >>>> users like Apache HTTPD. The user names for PKI_USER, PKI_GROUP, DS_USER >>>> and DS_GROUP are defined in other modules. Similar to #5587 the patch my >>>> patch moves the constants into the platform module. >>>> >>>> https://fedorahosted.org/freeipa/ticket/5619 >>> I see a few remaining cases: >>> >>> ipaserver/install/dsinstance.py >>> 712: pent = pwd.getpwnam("dirsrv") >>> >>> ipatests/test_integration/test_backup_and_restore.py >>> 167: self.master.run_command(['userdel', 'dirsrv']) >>> 168: self.master.run_command(['userdel', 'pkiuser']) >>> >>> ipaplatform/redhat/tasks.py >>> 441: if name == 'pkiuser': >>> >>> When these are included, ACK. >> Good catch! >> >> My new patch takes care of remaining cases. >> >> >> >> > > Christian do you agree with proposed changes, can we push it? > Martin^2
Oh, the patch is still open? ACK!
Description: OpenPGP digital signature
-- 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