Ethereal-dev: Re: [Ethereal-dev] Patch: sed check in configure.in

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

Date Index Thread Index Other Months All Mailing Lists
Date Prev Date Next Thread Prev Thread Next
From: Guy Harris <gharris@xxxxxxxxx>
Date: Sat, 2 Feb 2002 13:11:11 -0800
On Sat, Feb 02, 2002 at 09:14:16PM +0200, Heikki Vatiainen wrote:
> After not having compiled Ethereal in Solaris for some months, I today
> did cvs update and got the FAQ problem 3.3 "Output line too
> long". The problem is already addressed in CVS but maybe the patch
> below adds a little more safequard.

Checked in with...

>  2. Adds AC_ETHEREAL_GNU_SED_CHECK in acinclude.m4. This checks if GNU
>     sed is the first sed found from PATH.  Sets HAVE_GNU_SED if so.

...a change to that check to redirect the standard input of "sed" to
"/dev/null" when running "sed --version"; I'd argue that this is broken
behavior, but the "sed" on FreeBSD 3.4, at least (and possibly other
BSDs) blocks foreve when run with "--version" as its only argument - I
suspect it's just reading from the standard input (I suspect "getopt()"
stops parsing arguments when it sees "--", so "sed" thinks it was run
with no arguments).