[Freeciv-Dev] Re: (PR#8635) [PATCH] Correct FREECIV_PATH for binaries form civ & ser scripts

["Jason Short" <jshort@xxxxxxxxxxxxxx>]

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

Hmm, I don't quite understand the problems you're trying to fix.

Marko Lindqvist wrote:
>   export_FREECIV_PATH.diff: FREECIV_PATH must be exported to binaries.

Really?  I didn't think so.  It works for me, that is (I haven't 
retested, though).

>   Simple 'make' does not regenerate civ & ser scripts after 
> $(top_srcdir)/%.in changes, running configure or config.status is required.
>   Second patch, rebuild_civ_ser.diff, is at least partial solution to 
> that. Whenever civ.in or ser.in changes, config.status runs.
>   But I think there is more to do:
>  > cd freeciv_build
>  > rm civ
>  > make
> ...
>  > find civ
> find: civ: No such file or directory

Sure, but is this actually a problem?  How often are these scripts ever 
going to change?

>   However, targets for 'civ' and 'ser' do exist, even if 'all' does not 
> depend on them:
>  > cd freeciv_build
>  > rm civ
>  > make civ
> ...
>  > find civ
> civ

Interesting.

>   So I'm back to automake & autoconf manuals to find out what is right 
> way to make 'all' depend on those targets (and if we can even trust them 
> to always exist)

Try adding

civ: civ.in

to Makefile.am?  This sets the dependency (presumably autoconf knows 
this already) but doesn't cause civ to be built.  Hmm, so maybe add civ 
to noinst_PROGRAMS (or whatever it's called)?

jason