On Fri, May 06, 2005 at 06:53:58PM +0200, Frank Lichtenheld wrote:
> On Thu, May 05, 2005 at 08:13:57PM +0200, Nicolas Bertolissio wrote:
> > Thanks, I also read a part of the GNU Make manual and created the right
> > targets, this is committed now and next rebuild will fix the troubles.
Thanks, this was indeed missing.
> Hmm, that's probably not enough. I think all files also need to depend
> on all other files since the votebar can change if any of the vote files
> changes...
Nicolas, your patch included:
Index: vote/2002/platforms/Makefile
===================================================================
RCS file: /cvs/webwml/webwml/english/vote/2002/platforms/Makefile,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- vote/2002/platforms/Makefile 27 Feb 2002 19:34:43 -0000 1.1
+++ vote/2002/platforms/Makefile 6 May 2005 15:57:05 -0000 1.2
@@ -8,5 +8,8 @@
include $(WMLBASE)/Make.lang
+index.$(LANGUAGE).html: index.wml \
+ $(TEMPLDIR)/template.wml $(TEMPLDIR)/votebar.wml
+
index.$(LANGUAGE).html: index.wml $(wildcard vote_*.wml) \
$(TEMPLDIR)/template.wml $(TEMPLDIR)/recent_list.wml
I doubted this works so I tested it and read the make info page. It's OK.
It seems that
target: prerequisite
adds prerequisite to targets dependencies which is a GNU extension
since in the past (on Sun/HP workstations with make make not gmake) I was
always forced to use
target: prerequisite
touch target && sleep 1
(The sleep is necessary when multiple make commands are started during one
second and make (or touch?) ignored microseconds, ...)
Nevertheless I doubt that it is necessary to write two
index.$(LANGUAGE).html rules. It should be save to merge both and since
$(wildcard vote_*.wml) expands to "" below vote/*/platforms/ this can be
removed as well.
Please change it to:
index.$(LANGUAGE).html: index.wml $(TEMPLDIR)/template.wml \
$(TEMPLDIR)/votebar.wml $(TEMPLDIR)/recent_list.wml \
+ all other dependencies according to Frank
Jens
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
