On Fri, Jul 08, 2005 at 06:27:51PM +0100, Gary V. Vaughan wrote:
> Hallo Ralf,
> 
> Sorry for the late review:
> 
> On 8 Jul 2005, at 16:57, Ralf Wildenhues wrote:
> >2005-07-08  Peter Breitenlohner  <[EMAIL PROTECTED]>
> >
> >    * Makefile.am (install-data-local): Have root own of installed
> >    files (but do not fail, for user installs).
> >
> >Index: Makefile.am
> >===================================================================
> >RCS file: /cvsroot/libtool/libtool/Makefile.am,v
> >retrieving revision 1.153
> >diff -u -r1.153 Makefile.am
> >--- Makefile.am    2 May 2005 09:20:24 -0000    1.153
> >+++ Makefile.am    8 Jul 2005 15:54:01 -0000
> >@@ -254,6 +254,8 @@
> >     done; \
> >     ( cd $(srcdir)/m4 && $(AMTAR) chf - $$aclocalfiles; ) \
> >       | ( umask 0 && cd $(DESTDIR)$(aclocaldir) && $(AMTAR) xf -; )
> >+    chown -R root $(DESTDIR)$(ltdldatadir) && \
> >+    chgrp -R root $(DESTDIR)$(ltdldatadir) || :
> 
> What about:
> 
>     -chown -R root $(DESTDIR)$(ltdldatadir) 2>/dev/null
>     -chgrp -R root $(DESTDIR)$(ltdldatadir) 2>/dev/null

I like this too.

-- 
albert chin ([EMAIL PROTECTED])


Reply via email to