On Mon, 2012-05-14 at 10:00 +0200, Martin Kosek wrote: > On Thu, 2012-05-10 at 13:07 +0200, Petr Viktorin wrote: > > This is the second and likely the next-to-last part of disabling extra > > command options (after this it's just test fixes and turning the > > checking on). > > > > Part of the work for https://fedorahosted.org/freeipa/ticket/2509 > > > > This patch looks and works OK. I just think you missed a pkey_only > attribute for aci_find command. pkey_only is being passed to aci_find > command by selfservice_find and delegation_find commands and it would > fail in the hardened tests because it is not defined in aci_find command > as it is not based on LDAPSearch class but crud.Search. > > It just needs to be added to aci_find command and it should be fine. > > Martin
PetrĀ³ noticed that pkey_only was already explicitly added to takes_options of aci_find command. The patch is OK then. ACK. Pushed to master. Martin _______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel