Bug 1498539

Summary:	/usr/bin/xslt-config: line 101: --cflags: command not found
Product:	[Fedora] Fedora	Reporter:	Igor Gnatenko <ignatenko>
Component:	libxslt	Assignee:	Daniel Veillard <veillard>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	high
Version:	rawhide	CC:	quantum.analyst, veillard
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	libxslt-1.1.30-2.fc28	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-10-05 08:15:10 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Igor Gnatenko 2017-10-04 15:02:09 UTC

libxslt-devel-1.1.30-1.fc28.x86_64

[brain@ignatenko-w541 lxml]$ xslt-config --cflags
/usr/bin/xslt-config: line 101: --cflags: command not found

Comment 1 Igor Gnatenko 2017-10-04 16:25:49 UTC

From config.log:
> XML_CONFIG=''

That's the root cause.

Comment 2 Igor Gnatenko 2017-10-04 16:29:18 UTC

Looking at configure.in:

> if test "x$LIBXML_CONFIG_PREFIX" == "x" -a "x$LIBXML_SRC" == "x"
> then
>   PKG_CHECK_MODULES([LIBXML], [libxml-2.0 >= $LIBXML_REQUIRED_VERSION], [
>     LIBXML_MANUAL_SEARCH=no
>     WITH_MODULES="`$PKG_CONFIG --variable=modules libxml-2.0`"
>   ],
>   [
>     LIBXML_MANUAL_SEARCH=yes
>   ])
> else
>   LIBXML_MANUAL_SEARCH=yes
> fi
> 
> if test "x$LIBXML_MANUAL_SEARCH" != "xno"
> then
> 
> dnl
> dnl where is xml2-config
> dnl
> 
> AC_SUBST(LIBXML_REQUIRED_VERSION)
> AC_MSG_CHECKING(for libxml libraries >= $LIBXML_REQUIRED_VERSION)
> if test "x$LIBXML_CONFIG_PREFIX" != "x"
> then
>   AC_PATH_TOOL([XML_CONFIG], [xml2-config], [false], [${LIBXML_CONFIG_PREFIX}/bin])
> else
>   AC_PATH_TOOL([XML_CONFIG], [xml2-config], [false])
> fi

which means if libxml-2.0 pkg-config found, XML_CONFIG is basically not defined...

Comment 3 Igor Gnatenko 2017-10-04 16:56:49 UTC

I've sent Pull Request which fixes issue in a bit hack-ish way: https://src.fedoraproject.org/rpms/libxslt/pull-request/1

But at least it fixes issue and fixes broken xslt-config ;)

Comment 4 Elliott Sales de Andrade 2017-10-04 20:02:59 UTC

Upstream patch is available here:

https://git.gnome.org/browse/libxslt/commit/?id=76df4c960c7247d82bf1c26344e76236cb37a175

Comment 5 Igor Gnatenko 2017-10-04 21:27:14 UTC

(In reply to Elliott Sales de Andrade from comment #4)
> Upstream patch is available here:
> 
> https://git.gnome.org/browse/libxslt/commit/
> ?id=76df4c960c7247d82bf1c26344e76236cb37a175

Thanks a lot for pointer, I changed PR to apply linked patch instead.