On 01/02/2013 09:48 AM, Jonathan Nieder wrote:
> Jeff King wrote:
>> It seems I am late to the party. But FWIW, this looks the most sane to
>> me of the patches posted in this thread.
> Thanks.  config.status runs ./configure itself, though, so the rule
> should actually be
>       config.status: configure.ac
>               $(QUIET_GEN)$(MAKE) configure && \
>               if test -f config.status; then \
>                 ./config.status --recheck; \
>               else \
>                 ./configure;
>               fi
> Rather than screw it up yet again, I'm going to sleep. :)  If someone
> else corrects the patch before tomorrow, I won't mind.
FYI, this seems a sane approach to me.  At least until Autoconf is
improved to offer better (read: some :-) support to "dynamic" package
version numbers specified at configure runtime.  I hope that day
isn't too far, since the current Autoconf limitation has been causing
its share of annoyances small woes in Automake and Gnulib as well.

The only nit I have to offer is that I'd like to see more comments in
the git Makefile about why this "semi-hack" is needed.

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to