[ANNOUNCE] xf86-video-nv 1.2.1

Alan Coopersmith alan.coopersmith at sun.com
Mon Dec 18 18:23:20 PST 2006


Andres Salomon wrote:
> Alan Coopersmith wrote:
> [...]
>> It's failing for me too since the sed operations in the configure
>> script aren't working with Solaris sed.    config.h ends up with:
>>
>> /* Define to the version of this package. */
>> #define PACKAGE_VERSION "1.2.1"
>>
>> /* Major version of this package */
>> #define PACKAGE_VERSION_MAJOR
>>
>> /* Minor version of this package */
>> #define PACKAGE_VERSION_MINOR 0
>>
>> /* Patch version of this package */
>> #define PACKAGE_VERSION_PATCHLEVEL 0
>>
>> If I substitute GNU sed, it works.
>>
> 
> Ugh.  The way we're calling sed is:
> 
> sed -ne 's/^\([[^\.]]\+\).*/\1/p'
> sed -ne 's/^\([[^\.]]\+\)\.\([[^\.]]\+\).*/\2/p'
> sed -ne 's/^\([[^\.]]\+\)\.\([[^\.]]\+\)\.\([[^\.]]\+\).*/\3/p'
> 
> Do you know what part of that Solaris sed is unhappy with?

I poked around a bit and wasn't able to find why it wasn't working, but
it appears Matthieu Herrb did:
https://bugs.freedesktop.org/show_bug.cgi?id=9368

That fixes the problem for me:
/* Major version of this package */
#define PACKAGE_VERSION_MAJOR 1

/* Minor version of this package */
#define PACKAGE_VERSION_MINOR 2

/* Patch version of this package */
#define PACKAGE_VERSION_PATCHLEVEL 2

I'll go ahead and commit Matthieu's patch to the util-macros git repository.

-- 
	-Alan Coopersmith-           alan.coopersmith at sun.com
	 Sun Microsystems, Inc. - X Window System Engineering



More information about the xorg mailing list