Complete.Org: Mailing Lists: Archives: freeciv-dev: August 2004:
[Freeciv-Dev] Re: (PR#4712) Re: (PR#9748) (PR #4712) Comments about vers
Home

[Freeciv-Dev] Re: (PR#4712) Re: (PR#9748) (PR #4712) Comments about vers

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: jdorje@xxxxxxxxxxxxxxxxxxxxx
Cc: jdwheeler42@xxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#4712) Re: (PR#9748) (PR #4712) Comments about version.diff
From: "Marko Lindqvist" <marko.lindqvist@xxxxxxxxxxx>
Date: Fri, 20 Aug 2004 12:33:35 -0700
Reply-to: rt@xxxxxxxxxxx

<URL: http://rt.freeciv.org/Ticket/Display.html?id=4712 >

Jason Short wrote:

> <URL: http://rt.freeciv.org/Ticket/Display.html?id=4712 >
> 
> What about this line in configure.in/configure.ac:
>   VERSION=${MAJOR_VERSION}.${MINOR_VERSION}.${PATCH_VERSION}${VERSION_LABEL}
> 
> this requires that configure.* knows what the version # is.  AFAIK this 
> is needed so "make dist" works.

  configure.* gets those definitions by including version.in, that's the 
whole point of this patch! (You probably missed that 
'm4_include(version.in)' had '+' and not '-' in front of it in diff)

> Also, what does this line do:
> 
> +AC_SUBST([CONFIGURE_DEPENDENCIES], ['$(top_srcdir)/version.in'])
> 

  It's automake feature (since version 1.4) that causes configure 
-script to be rebuild whenever version.in changes, just like it is 
rebuilt when configure.ac changes.
  Before this patch version changes were written directly to configure.* 
and that caused configure to be rebuilt. We move version information out 
of configure.* -> they no longer change when version changes -> 
configure is not rebuilt. Above line reverts that by effectively saying 
to automake that version.in should be considered as part of configure.*
  This all probably requires --enable-maintainer-mode to work at all, I 
have not tested otherwise.


  - Caz




[Prev in Thread] Current Thread [Next in Thread]