[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Debian WWW CVS: joy



On Tue, May 29, 2001 at 03:02:30PM +0200, Martin Schulze wrote:
> Josip Rodin wrote:
> > On Tue, May 29, 2001 at 08:01:10AM +0200, Martin Schulze wrote:
> > > Debian WWW CVS wrote:
> > > > CVSROOT:	/cvs/webwml
> > > > Module name:	webwml
> > > > Changes by:	joy	01/05/28 08:24:25
> > > > 
> > > > Modified files:
> > > > 	english        : Makefile 
> > > > 
> > > > Log message:
> > > > erm, actually removed sitemap from index deps, and added missing deps for sitemap itself
> > > 
> > > You've added the calls to make to ensure that subscribe.wml and
> > > unsubscribe.wml are in place.  Was that intentional?
> > 
> > I _have_? The Makefile says:
> > 
> > sitemap.$(LANGUAGE).html: $(ENGLISHDIR)/sitemap.wml \
> >   $(TEMPLDIR)/links.tags.wml $(TEMPLDIR)/template.wml \
> >   $(ENGLISHDIR)/releases/info $(ENGLISHDIR)/MailingLists/mklist.tags
> > [three methods to build it]
> > 
> > What do you mean subscribe.wml and unsubscribe.wml?
> 
> I was - no shit - talking about these files without which make would
> break.  They are auto-generated and thus do not exist on a new checkout.
> 
> $(ENGLISHDIR)/MailingLists/subscribe.wml
> $(ENGLISHDIR)/MailingLists/unsubscribe.wml
> 
> Here's the diff that shows what I missed in detail.
> 
> -ifeq "$(LANGUAGE)" "en"
> -sitemap.$(LANGUAGE).html: $(ENGLISHDIR)/sitemap.wml
> -       make -C MailingLists subscribe.$(LANGUAGE).html unsubscribe.$(LANGUAGE).html
> -else
> -sitemap.$(LANGUAGE).html: $(ENGLISHDIR)/sitemap.wml
> -       make -C MailingLists subscribe.$(LANGUAGE).html unsubscribe.$(LANGUAGE).html

Oh, better catch up with the rest of your debian-www mail, you'll see how
that has been fixed :)

-- 
Digital Electronic Being Intended for Assassination and Nullification



Reply to: