On Thu, 30 Jul 2009 11:25:52 -0500, Eric Sandeen <[email protected]> wrote: > Ryusuke Konishi wrote: > > Hi, > > On Thu, 30 Jul 2009 09:59:09 -0500, Eric Sandeen <[email protected]> wrote: > > ... > > >> I don't think there's any reason for the chown; can this just be removed? > > > > Well, I see. > > > > I will once remove the chown and normalize it unless the removal makes > > inconvenience to someone. > > > > Thank you for the comment. > > > > Ryusuke Konishi > > thanks - on a similar topic, the Makefile in lib/ wants to run ldconfig, > I think it's a similar problem for non-root builds. > > For now I've just worked around these problems in the Fedora build.
Yes, It's been in my mind. Here I attach the patch to remove these install-exec-hooks. After some tests, I will push it to the git tree in two separate patches. Thanks, Ryusuke Konishi diff --git a/lib/Makefile.am b/lib/Makefile.am index 131702b..1dbe74f 100644 --- a/lib/Makefile.am +++ b/lib/Makefile.am @@ -23,6 +23,3 @@ libnilfs_la_CFLAGS = -Wall libnilfs_la_CPPFLAGS = -I$(top_srcdir)/include libnilfs_la_LDFLAGS = -version-info $(VERSIONINFO) libnilfs_la_LIBADD = libnilfsmisc.la - -install-exec-hook: - $(LDCONFIG) $(DESTDIR)$(libdir) diff --git a/sbin/cleanerd/Makefile.am b/sbin/cleanerd/Makefile.am index edde74e..6fac7ff 100644 --- a/sbin/cleanerd/Makefile.am +++ b/sbin/cleanerd/Makefile.am @@ -14,9 +14,3 @@ nilfs_cleanerd_CPPFLAGS = -I$(top_srcdir)/include \ nilfs_cleanerd_LDADD = $(top_builddir)/lib/libnilfs.la dist_sysconf_DATA = nilfs_cleanerd.conf - -install-exec-hook: - list='$(sbin_PROGRAMS)'; \ - for p in $$list; do \ - chown root $(DESTDIR)$(sbindir)/$$p$(EXEEXT); \ - done diff --git a/sbin/mkfs/Makefile.am b/sbin/mkfs/Makefile.am index bae054a..3fd1288 100644 --- a/sbin/mkfs/Makefile.am +++ b/sbin/mkfs/Makefile.am @@ -10,9 +10,3 @@ mkfs_nilfs2_SOURCES = mkfs.c bitops.c ../../lib/crc32.c mkfs.h mkfs_nilfs2_CFLAGS = -Wall mkfs_nilfs2_CPPFLAGS = -I$(top_srcdir)/include mkfs_nilfs2_LDADD = -luuid - -install-exec-hook: - list='$(sbin_PROGRAMS)'; \ - for p in $$list; do \ - chown root $(DESTDIR)$(sbindir)/$$p$(EXEEXT); \ - done diff --git a/sbin/mount/Makefile.am b/sbin/mount/Makefile.am index 9aea415..0f8d55f 100644 --- a/sbin/mount/Makefile.am +++ b/sbin/mount/Makefile.am @@ -20,9 +20,3 @@ mount_nilfs2_LDADD = $(top_builddir)/lib/libnilfsmisc.la umount_nilfs2_SOURCES = umount.nilfs2.c $(COMMONSOURCES) $(COMMONHEADERS) umount_nilfs2_LDADD = $(top_builddir)/lib/libnilfsmisc.la - -install-exec-hook: - list='$(sbin_PROGRAMS)'; \ - for p in $$list; do \ - chown root $(DESTDIR)$(sbindir)/$$p$(EXEEXT); \ - done _______________________________________________ users mailing list [email protected] https://www.nilfs.org/mailman/listinfo/users
