Re: Config Bug

["Jay 'Eraserhead' Felice" <eraserhead@xxxxxxxxxxxxxx>]

On Wed, May 30, 2001 at 10:47:29AM -0500, Scott Klement wrote:
> 
> 
> On 31 May 2001, Carey Evans wrote:
> > 
> > autoconf has very good reasons for doing this the way that it does,
> > and the manual says that the current configure.in is wrong to try to
> > use $sysconfdir like that at all - see the "Installation Directory
> > Variables" node in the manual, at least with autoconf 2.50.
> 
> I can't find this node in the 2.13 manual, but then I have a hard time
> finding _anything_ in the autoconf info-pages.  :( 
> 
> At any rate, I didn't write that code -- and I'm not particularly attached
> to it -- I just wanted it to _work_, since I was tired of editing
> tn5250-config.h manually... :)
> 
> > 
> > Can you try this patch instead, which does it properly, and cleans up
> > a couple of other bits of configure as well?
> > 
> 
> Your patch appears to work just fine.  
> 
> I've got a few other things I'd like to fix in the configuration stuff,
> since there's a lot of steps you have to take manually on FreeBSD systems
> right now  (you have to patch xt5250, replace XTerm, install termcap
> entry, etc)   Should I try to avoid calling anything from the shell in
> configure.in?  
> 

Nah...  The only advice I've really ever heard is that you should try to do
things based on whether or not certain features are present as opposed to
what kind of system it is.  For example, a Windows version might use
WSAAsyncSelect() instead of select(); if we support this, we should use
WSAAsyncSelect() based on whether the function was found, not whether the
host type is Windows.

Of course, this is not always possible, but I've found it's made things much
more portable and I've been pleasantly surprised when a package just happens
to work on a new system ;-)

-Jay

+---
| This is the LINUX5250 Mailing List!
| To submit a new message, send your mail to LINUX5250@midrange.com.
| To subscribe to this list send email to LINUX5250-SUB@midrange.com.
| To unsubscribe from this list send email to LINUX5250-UNSUB@midrange.com.
| Questions should be directed to the list owner/operator: david@midrange.com
+---