On Thu, 08 Dec 2011, Rob Crittenden wrote: > >>>Other than those two minor points, the patch looks very good. I'm > >>>going to give it a run on Friday. > >>I tested the patch and it works for me on a new install. On upgrade of > >>existing installation I've got few errors during run of > >>ipa-ldap-updater for SELinux schema changes. Unfortunately, didn't > >>save the log as it was 2.1 -> 2.99 upgrade as well. > >> > > > >It turns out that other characters are just as troublesome and require > >escaping (space and \). I"m going to leave it as $ unless someone comes > >up with something better that the shell isn't going to whine about. > > > >I fixed some other minor issues and rebased. > > > >Upgrading isn't really testable at this point yet, other things in 3.0 > >need to be addressed as well. We have a separate ticket to look into the > >schema updates so I've removed the update file for now. > > > >rob > > Rebased patch ACK. Pushed to master.
I had to fix freeipa.spec.in to have proper samba4-devel dependency now that ipa-devel repository includes fixed samba4 package (instead of samba-4.0). -- / Alexander Bokovoy _______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel