[Freeipa-devel] [PATCH 7 of 8] Backup system state in ipa-server-install

[Mark McLoughlin]

On Mon, 2008-01-14 at 11:13 -0500, Rob Crittenden wrote:
> Mark McLoughlin wrote:

> > @@ -107,6 +107,7 @@ fi
> >  %attr(755,root,root) %{plugin_dir}/libipa-memberof-plugin.so
> >  %attr(755,root,root) %{plugin_dir}/libipa-dna-plugin.so
> >  
> > +%dir %{_localstatedir}/cache/ipa
> >  
> >  %changelog
> >  * Fri Dec 21 2007 Karl MacMillan <kmacmill at redhat.com> - 0.6.0-1
> 
> Is it adequate for the rpm to own just the top level directory or does 
> it need to own all the subdirs as well? I can't recall if a %dir is 
> recursive.

	No, %dir is not recursive. So we also need:

%dir %{_localstatedir}/cache/ipa/sysrestore

	Fixed version attached.

> In any case in testing I get:
> 
> Configuring Kerberos KDC
>    [0/13]: setting KDC account password
> Unexpected error - see ipaserver-install.log for details:
>   [Errno 2] No such file or directory: '/var/cache/ipa/sysrestore/var'
> 
> I gather we need to make this in a 'make install' as well, right?

	We do need /var/cache/sysrestore, but not subdirs of that - the
sysrestore code handles creating the subdirs.

	I only tested with "make install", not by installing the ipa-server
RPM, so that's probably why I didn't see this.

Thanks,
Mark.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ipa-server-sysrestore.patch
Type: text/x-patch
Size: 28381 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20080114/96fcc151/attachment.bin>