zsh-workers
 help / color / mirror / code / Atom feed
From: "Bart Schaefer" <schaefer@brasslantern.com>
To: zsh-workers@sunsite.auc.dk
Subject: Re: Make dependencies broken in pws-6 + latest patches
Date: Fri, 5 Feb 1999 08:58:41 -0800	[thread overview]
Message-ID: <990205085841.ZM23447@candle.brasslantern.com> (raw)
In-Reply-To: <9902050948.AA57511@ibmth.df.unipi.it>

On Feb 5, 10:48am, Peter Stephenson wrote:
} Subject: Re: Make dependencies broken in pws-6 + latest patches
}
} "Bart Schaefer" wrote:
} > I've noticed previously that changes to Config/version.mk don't cause the
} > Src/version.h file to be updated.
} > 
} > Now, with PWS's bashautolist patch, rebuilding with the changed zsh.h left
} > the opts[] array messed up.
} 
} Me not neither.  I noticed this, too, and discovered that main.o
} wasn't being remade --- removing main.o and making should work.  This
} is because $(MAIN_OBJS) depends on Makemod, which was up to date.
} 
} As for the version number, there are already explicit dependencies for
} this in Makemod.
} 
}  version.h: $(sdir_top)/Config/version.mk
} 	echo '#define ZSH_VERSION "'$(VERSION)'"' > $@

OK, I think the problem is that Makemod itself isn't being rebuilt.  The
$(VERSION) is set by AC_FILE_SUBST-ing version.mk into Makemod.in.in, but
Makemod doesn't have a dependency for Makemod.in nor does Makemod.in have
one for version.mk.  Similarly, it's only sufficient for $(MAIN_OBJS) to
be dependent solely on Makemod only if Makemod depends on zsh.h.

I think somebody made the assumption that "configure" would ALWAYS be
run (manually) before running "make".

However, I'm still not sure where the right place to add those dependencies
is.  It'd be obvious to make $(MAIN_OBJS) depend on zsh.h, but really *all*
the files should depend on zsh.h, no?

And would the Makemod.in dependency on version.mk go *in* version.mk?  That
would work after the first manual reconfigure, I think, because Makefile.in
depends on Makemod.in so (I think) the whole make system would get rebuilt
upon a change to version.mk.

-- 
Bart Schaefer                                 Brass Lantern Enterprises
http://www.well.com/user/barts              http://www.brasslantern.com


      reply	other threads:[~1999-02-05 16:58 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
1999-02-04 16:25 Bart Schaefer
1999-02-05  9:48 ` Peter Stephenson
1999-02-05 16:58   ` Bart Schaefer [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=990205085841.ZM23447@candle.brasslantern.com \
    --to=schaefer@brasslantern.com \
    --cc=zsh-workers@sunsite.auc.dk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).