This is the mail archive of the automake@gnu.org mailing list for the automake project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

.texi.html: Add a touch?


makeinfo does not update timestamps of directories:


% ls -dltr maintain.{texi,html}              nostromo 15:12
drwxr-xr-x    2 akim     lrde         4096 2004-05-25 14:38 maintain.html/
-rw-r--r--    1 akim     lrde        16132 2004-05-25 15:01 maintain.texi
% LC_ALL=C /bin/sh /home/akim/lectures/maintain/config/missing --run makeinfo --html --enable-encoding --number-sections --fill-column=76  -I . \
>  -o maintain.html maintain.texi
% ls -dltr maintain.{texi,html}
drwxr-xr-x    2 akim     lrde         4096 2004-05-25 14:38 maintain.html/      
-rw-r--r--    1 akim     lrde        16132 2004-05-25 15:01 maintain.texi


which keeps this rule unsatisfied:

.texi.html:
	$(MAKEINFOHTML) $(AM_MAKEINFOHTMLFLAGS) $(MAKEINFOFLAGS) -I $(srcdir) \
	 -o $@ $<
	if test ! -d $@ && test -d $(@:.html=); then \
	  mv $(@:.html=) $@; else :; fi


This can cause bad problems when trying to install under a different
identity:

% sudo -u doc make install                   nostromo 15:06
LC_ALL=C /bin/sh /home/akim/lectures/maintain/config/missing --run makeinfo --html --enable-encoding --number-sections --fill-column=76  -I . \
 -o maintain.html maintain.texi
/home/akim/lectures/maintain/maintain.html/index.html: Permission denied
make: *** [maintain.html] Erreur 1


so I suggest the following change:

Index: ChangeLog
from  Akim Demaille  <akim@epita.fr>

	* lib/am/texibuild.am (.texi.html): Touch the output to update its
	timestamp, which is not performed by makeinfo when producing a
	directory.

Index: lib/am/texibuild.am
===================================================================
RCS file: /cvs/automake/automake/lib/am/texibuild.am,v
retrieving revision 1.21
diff -u -u -r1.21 texibuild.am
--- lib/am/texibuild.am 15 May 2004 16:12:07 -0000 1.21
+++ lib/am/texibuild.am 25 May 2004 13:21:50 -0000
@@ -87,6 +87,8 @@
 ## instead of foo.html/).
 	if test ! -d $@ && test -d $(@:.html=); then \
 	  mv $(@:.html=) $@; else :; fi
+## makeinfo does not update the timestamp of the directory.
+	touch $@
 
 ## If we are using the generic rules, we need separate dependencies.
 ## (Don't wonder about %DIRSTAMP% here, this is used only by non-generic



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]