[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Subversion 1.6 requires GNU grep 2.5

From: Blair Zajac <blair_at_orcaware.com>
Date: Tue, 21 Jul 2009 23:14:18 -0700

On Jul 21, 2009, at 10:29 PM, Ryan Schmidt wrote:

> I have discovered that Subversion 1.6.0 and later requires GNU grep
> 2.5 or greater. This became apparent when Subversion 1.6.3's
> configure script failed to run on a Mac OS X 10.3.9 Panther system:
>
>
> checking whether libtool needs -no-undefined... no
> checking whether to avoid circular linkage at all costs... no
> checking for trang... none
> checking for socket in -lsocket... no
> /usr/bin/grep: invalid option -- o
> Usage: grep [OPTION]... PATTERN [FILE]...
> Try `grep --help' for more information.
> configure: error: Invalid syntax of argument of --with-berkeley-db
> option
>
>
> Panther includes GNU grep 2.4.2. After installing GNU grep 2.5.3, all
> is well. The -o (a.k.a. --only-matching) option is mentioned as a new
> feature of GNU grep 2.5 in its NEWS file. I am also informed by a
> ChangeLog entry from 2005-08-24 that -o is a GNU-specific extension,
> and is not defined by POSIX.
>
> Subversion 1.5.6 did not have a problem with the older grep (because
> it did not use $EGREP -o), and Subversion's INSTALL file does not
> mention GNU grep as a dependency.
>
> I believe either the software should work with older versions of GNU
> grep (and with non-GNU grep), or GNU grep 2.5 should be listed as a
> dependency in the INSTALL file. I feel the former is preferable, and
> since $EGREP -o is only used in one place it shouldn't be that hard
> to do. I searched the issue tracker for "grep" in issue summaries and
> there was only one hit which was not this issue. May I file a bug
> report for this issue?

Sure, go for it. I did a recursive grep through the code and it looks
like there's only two places:

./build/ac-macros/berkeley-db.m4:65: if test -n "`echo "$withval"
| $EGREP -o ":.*:.*:"`"; then
./build/ac-macros/kwallet.m4:43: for d in [`
$PKG_CONFIG --cflags QtCore QtDBus QtGui | $GREP -o -- -
D[^[:space:]]*`]; do

Do you want to supply a patch?

Blair

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=1065&dsMessageId=2374305

To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_subversion.tigris.org].
Received on 2009-07-22 08:15:15 CEST

This is an archived mail posted to the Subversion Users mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.