Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 435496 - media-libs/mesa-8.0.4-r1 have non POSIX test construction
Summary: media-libs/mesa-8.0.4-r1 have non POSIX test construction
Status: RESOLVED FIXED
Alias: None
Product: Gentoo/Alt
Classification: Unclassified
Component: FreeBSD (show other bugs)
Hardware: AMD64 FreeBSD
: Normal normal (vote)
Assignee: Gentoo/BSD Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-19 10:55 UTC by KANATSU Minoru
Modified: 2012-10-04 08:38 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
simpe patch to fix non POSIX test construction (patch,482 bytes, patch)
2012-10-03 22:28 UTC, Naohiro Aota
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description KANATSU Minoru 2012-09-19 10:55:10 UTC
Found a non POSIX test construction in a configure script
Comment 1 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-09-19 11:18:31 UTC
Where exactly is that located, and is there already a bug report upstream at https://bugs.freedesktop.org/ ?
Comment 2 KANATSU Minoru 2012-09-19 11:25:13 UTC
when emerge mesa,
I was said

>>> Messages generated by process 14046 on 2012-09-19 19:44:56 JST for package media-libs/mesa-8.0.4-r1:

ERROR: compile
Found a non POSIX test construction in a configure script
The configure checks of this package may not function properly
Please report this on Gentoo Bugzilla in Gentoo/Alt product for component FreeBSD.
http://bugs.gentoo.org/enter_bug.cgi?product=Gentoo%2FAlt&component=FreeBSD&op_sys=FreeBSD
LOG: postinst
USE="bindist" was not set. Potentially patent encumbered code was
enabled. Please see patents.txt for an explanation.
Comment 3 Matt Turner gentoo-dev 2012-10-02 22:21:53 UTC
BSD team: I hope you can handle this bug, because there's no useful information in it for the X11 team.
Comment 4 Naohiro Aota gentoo-dev 2012-10-03 22:28:28 UTC
Created attachment 325628 [details, diff]
simpe patch to fix non POSIX test construction

# ebuild /usr/portage/media-libs/mesa/mesa-8.0.4-r1.ebuild unpack
# cd /dev/shm/portage/media-libs/mesa-8.0.4-r1/work/Mesa-8.0.4
# grep  "\\<test .*==" . -r --include configure
./configure:        if test "x$HAVE_XF86VIDMODE" == xyes; then
# grep HAVE_XF86VIDMODE configure.ac
        PKG_CHECK_MODULES([XF86VIDMODE], [xxf86vm], HAVE_XF86VIDMODE=yes, HAVE_XF86VIDMODE=no)
        if test "$HAVE_XF86VIDMODE" = yes ; then
        if test "x$HAVE_XF86VIDMODE" == xyes; then
AC_SUBST([HAVE_XF86VIDMODE])

The patch is attached.
Comment 5 Matt Turner gentoo-dev 2012-10-03 22:41:28 UTC
Ah, I see. Thank you.

This was fixed (not purposefully) by http://cgit.freedesktop.org/mesa/mesa/commit/?id=e273ed37eaf9de57883631f223afbe50f1c78291 in August and so this isn't applicable to the 9.0 or master branch.

If it's important for BSD, feel free to add that patch to the 8.0 and older releases.
Comment 6 Naohiro Aota gentoo-dev 2012-10-04 08:38:28 UTC
(In reply to comment #5)

> If it's important for BSD, feel free to add that patch to the 8.0 and older
> releases.

I've added my patch to the tree. Thanks!