[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