Summary: | x11-libs/pixman: non POSIX test | ||
---|---|---|---|
Product: | Gentoo/Alt | Reporter: | Naohiro Aota <naota> |
Component: | FreeBSD | Assignee: | Gentoo/BSD Team <bsd+disabled> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | x11 |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
files/pixman-0.22.2-posix-test.patch
pixman-0.22.2.ebuild.patch |
Description
Naohiro Aota
2011-10-14 03:46:54 UTC
Created attachment 289773 [details, diff]
files/pixman-0.22.2-posix-test.patch
Patch for the configure.ac
Created attachment 289775 [details, diff]
pixman-0.22.2.ebuild.patch
Patch for the ebuild
x11 team, Could you review these patch please? This patch should replace non POSIX test with POSIX compliant one. Confirmed it get built both on Linux and FreeBSD without QA warns. Thanks in advance. (In reply to comment #1) > Created attachment 289773 [details, diff] > files/pixman-0.22.2-posix-test.patch > > Patch for the configure.ac Want me to send this upstream? (In reply to comment #4) > (In reply to comment #1) > > Created attachment 289773 [details, diff] > > files/pixman-0.22.2-posix-test.patch > > > > Patch for the configure.ac > > Want me to send this upstream? oh, please go ahead :) LGTM. Although you should take a look what upstream style is, autoconf mans prefer x"$foo" == x"", I think. Reported upstream and they added the patch. https://bugs.freedesktop.org/show_bug.cgi?id=42588 Applied in Gentoo ebuild also. Thanks. |