Update of bug #16142 (project freeciv):

                Category:                    None => bootstrap              
                 Release:                         => 2.2.1                  
                 Summary: Freeciv 2.2.1 doesn’t compile on Mac OS X 10.6.3
=> FTWL sources not included in tarball

    _______________________________________________________

Follow-up Comment #7:

Copying discussion from RT ticket #40349, will also attach patch from it.


From: "Marko Lindqvist" <cazf...@gmail.com>

2008/7/13 Daniel Markstedt:
>
>> [cazf...@gmail.com - Tue Jul 01 20:47:26 2008]:
>>
>> 2008/7/1 Marko Lindqvist:
>> > 2008/7/1 Jason Dorje Short:
>> >
>> >> "make dist" assumes all conditionals pass to find out what needs
> to be
>> >> distributed.
>> >
>> > I believe "make dist" collects files from both "conditional pass"
>> > and "conditional fails" possibilities.
>>
>> In the case of ftwl-directory, I suspect that old automake is not
>> properly handling conditionally built library (even though it's in
>> "conditional pass" branch) and subsequently not adding source files it
>> would require.
>>
>> Attached patch removes conditionality from library and makes entering
>> to the directory in the first place conditional instead. Conditional
>> subdirs are already used in other places.
>> This works for me (automake 1.10.1) at least. I can compile both
>> ftwl-client and other clients, and 'make dist' adds ftwl-files no
>> matter which client is enabled from configure.
>
> I cannot currently test this patch, as configuring S2_0 with --enable-
> ftwl fails both with and without it. The last messages are:
>
> checking for unistd.h... yes
> configure: error: bad value yes for --enable-ftwl

Try --enable-ftwl=x11 or --enable-ftwl=sdl

Please test also 'make dist' when ftwl is *not* selected as client at
configure time. 'make dist' should make full tarball, no matter the
configure time selections.


- ML

(file #9340)
    _______________________________________________________

Additional Item Attachment:

File name: FTWLdist_40349.diff            Size:1 KB


    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?16142>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to