Hi,

> That would be great! I also suspect that it would fix the --perfix=/ case 
> as well, which could be the default at some point because of the reasons 
> I just mentioned in my previous email (mount failures during boot, etc).

Hm... I'm not sure I got correctly what the problem with --prefix=/ is, so I 
left out that part for now, but I'm willing to help if there is anything to do.

Regarding defaults, please don't make --prefix=/ the default; use --prefix=/usr 
--exec-prefix=/. The reason is very simple: /man is not in groff's search path, 
/lib/pkgconfig is not in pkg-config's search path, /include is not in gcc's 
search path. => Only executable files can be in /, data files are not allowed 
there and must remain elsewhere: /usr or /usr/local in most cases.

Anyway, attached is a patch that adds a --disable-ldconfig option. I also 
reorganized a few things in src/Makefile.am for simplicity reasons: The 
symlinks can be created at any time, they don't need to be created as the last 
thing, so they can be done in *-local instead of *-hook. This allows me to put 
the entire *-hook target into a conditional, which is easier and more 
compatible with old automake versions than having just parts of a target inside 
a conditional.

Comments are welcome, I can change this if desired.

Andreas Hanke
-- 
"Feel free" - 10 GB Mailbox, 100 FreeSMS/Monat ...
Jetzt GMX TopMail testen: www.gmx.net/de/go/mailfooter/topmail-out
--- configure.ac
+++ configure.ac
@@ -59,6 +59,13 @@
 	enable_warnings=no
 )
 
+AC_ARG_ENABLE(ldconfig,
+	AS_HELP_STRING(--disable-ldconfig,do not update dynamic linker cache
+		       using ldconfig), ,
+	enable_ldconfig=yes
+)
+AM_CONDITIONAL(RUN_LDCONFIG, test "$enable_ldconfig" = yes)
+
 # Use GNU extensions if available.
 AC_GNU_SOURCE
 
@@ -67,7 +74,10 @@
 AC_PROG_GCC_TRADITIONAL
 AC_PROG_INSTALL
 AC_PATH_PROG(RM, rm, rm)
-AC_PATH_PROG(LDCONFIG, ldconfig, true, [/sbin /usr/sbin $PATH])
+# No need to check for ldconfig if --disable-ldconfig was given
+if test "$enable_ldconfig" = yes; then
+	AC_PATH_PROG(LDCONFIG, ldconfig, true, [/sbin /usr/sbin $PATH])
+fi
 AC_PROG_LN_S
 AC_PROG_MAKE_SET
 AC_PROG_LIBTOOL
--- src/Makefile.am
+++ src/Makefile.am
@@ -35,12 +35,16 @@
 libs:
 	(cd ../libntfs-3g && $(MAKE) libs) || exit 1;
 
+if RUN_LDCONFIG
 install-exec-hook:
+	$(LDCONFIG)
+endif
+
+install-exec-local:
 	$(INSTALL) -d $(DESTDIR)/sbin
 	$(LN_S) -f $(bindir)/ntfs-3g $(DESTDIR)/sbin/mount.ntfs-3g
-	$(LDCONFIG)
 
-install-data-hook:
+install-data-local:
 	$(INSTALL) -d $(DESTDIR)$(man8dir)
 	$(LN_S) -f ntfs-3g.8 $(DESTDIR)$(man8dir)/mount.ntfs-3g.8
 
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
ntfs-3g-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ntfs-3g-devel

Reply via email to