[Freeciv-Dev] Re: Compiling with readline < 4.2

[Daniel Speyer <speyer1@xxxxxxxx>]

thue@xxxxxxx,Internet writes:
>On Sunday 01 July 2001 20:25, Daniel Speyer wrote:
>> thue@xxxxxxx,Internet writes:
>> >On Sunday 01 July 2001 00:26, Thue wrote:
>> >> Here is an addion to INSTALL describing how to compile with
>> >> readline < 4.2.
>> >>
>> >> -Thue
>> >
>> >Oh, and the patch.
>> >
>> >-Thue *going to bed*
>>
>> This seems a bit awkward, seeing as Redhat 7.1 and Mandrake 8.0 both
>> ship with readline 4.1.  Shouldn't it be possible to add something
>> like
>>
>> #ifndef RL_STATE_NONE /*Only exists in readline 4.2*/
>> #define rl_filename_completion_function filename_completion_function
>> #endif
>>
>> to some header file (probably server/stdinhand.h) and do something
>> similar with freeciv_completion()?  It's not really fair that users
>> of current distros should have to upgrade a library or modify source.

>
>Blrg.
>Yes, I see your point.
>But I really like to just require the latest readline lib; so much
>cleaner... And readline is not critical for running a server...
>
>-Thue
>
>

I certainly see your point about clemliness, but this would kill
Freeciv's out-o
f-the-box ( ;) ) installability for unneeded functionality.  Maybe
there's a thi
rd way?

Would it be possible to include a script that fixes the source or a set
of diffs
 (a sed-based shell script could probably go without alteration longer
than a di
ff could) to make installing with old readline easier?  Maybe even make
./config
ure take care of it automatically?  (I really don't know if that's
possible).

In any case, ease of installation is one of the few points in which
proprietary
software is often ahead of us, so it would be good to make this nice.

--Daniel Speyer
"May the /src be with you, always"

P.S. Please ignore the change in my e-mail address.  My regular account
seems to be down.