Bug 415293

Summary:	media-libs/libwmf-0.2.8.4-r4: fix a typo in the ebuild
Product:	Gentoo Linux	Reporter:	Rafał Mużyło <galtgendo>
Component:	[OLD] Library	Assignee:	No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status:	RESOLVED FIXED
Severity:	trivial
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---

Description Rafał Mużyło 2012-05-09 19:29:30 UTC

In one place '$(use_with xml libxml2)' (correctly) is used, two lines below '--disable-libxml2' is used.

Comment 1 Pacho Ramos gentoo-dev

2012-05-11 08:55:22 UTC

Looks valid to me:

        if use expat; then
                myconf+=" --disable-libxml2"
        else
                myconf+=$(use_with xml libxml2)
        fi

When USE expat is enabled, --disable-libxml2 is always passed, when it's disabled, libxml2 can be enabled/disabled

Comment 2 Pacho Ramos gentoo-dev

2012-05-11 08:57:40 UTC

And looks consistent with deps:

        xml? (
                expat? ( dev-libs/expat )
                !expat? (  dev-libs/libxml2 )

Comment 3 Pacho Ramos gentoo-dev

2012-05-11 08:58:48 UTC

This was introduced in bug 373919

Comment 4 Rafał Mużyło 2012-05-11 14:17:22 UTC

No, it's *obviously* not valid (as the previous part makes it clear), that part should be '--without-libxml2'.

Comment 5 Pacho Ramos gentoo-dev

2012-05-12 07:00:02 UTC

+  12 May 2012; Pacho Ramos <pacho@gentoo.org> libwmf-0.2.8.4-r4.ebuild:
+  Fix typo, move from 'disable' to 'without' configure option, bug #415293 by
+  Rafał Mużyło.
+