[Mesa-dev] [PATCH] configure.ac: Check for expat21 if expat is not found.
emil.l.velikov at gmail.com
Tue Aug 8 10:58:24 UTC 2017
On 7 August 2017 at 19:13, Vinson Lee <vlee at freedesktop.org> wrote:
> On Mon, Aug 7, 2017 at 8:18 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>> On 5 August 2017 at 21:53, Vinson Lee <vlee at freedesktop.org> wrote:
>>> Fixes build error on CentOS 6.9.
>>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102052
>>> Fixes: 5c007203b73d ("configure.ac: drop manual detection of expat header/library")
>>> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
>>> configure.ac | 4 +++-
>>> 1 files changed, 3 insertions(+), 1 deletions(-)
>>> diff --git a/configure.ac b/configure.ac
>>> index 5b12dd8..0dfd47b 100644
>>> --- a/configure.ac
>>> +++ b/configure.ac
>>> @@ -1792,7 +1792,9 @@ if test "x$with_dri_drivers" = xno; then
>>> # Check for expat
>>> -PKG_CHECK_MODULES([EXPAT], [expat])
>>> +PKG_CHECK_MODULES([EXPAT], [expat],,
>>> + [PKG_CHECK_MODULES([EXPAT], [expat21])]
>> Hmm... the CenOS guys have gone "interesting" ways with this.
>> I doubt we have any actual users for such setups, so I'm wondering if
>> you can keep a WA for this locally.
>> Say simply create a link/copy?
> CentOS 6 and RHEL 6 have expat 2.0 by default. expat 2.0 does not
> provide expat.pc.
No arguments there - it's their decision to use 10+ year old software.
Do we have any actual RHEL/CentOS 6 users, apart from the build
testing that you do?
If yes, sure let's have the workaround - otherwise, please keep it
locally. Some solutions include:
- cp/symlink the file
- set EXPAT_CFLAGS and EXPAT_LIBS alongside/prior to calling configure
More information about the mesa-dev