Bug 19710 – xfree-4.3.0-r2.ebuild: USE flag "xml" is used when it appears "xml2" should be

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description Phil Richards 2003-04-21 08:17:48 UTC]

Now that I've turned on -v for doing my "what do I need to update" queries,
I noticed that I got:

root # emerge -v -uUDp xfree

These are the packages that I would merge, in order:

Calculating dependencies ...done!
[ebuild    U ] x11-base/xfree-4.3.0-r2 [4.3.0-r1] -3dfx -sse +mmx +3dnow -xml
+truetype +nls -cjk -doc


Checked my settings:

root # emerge info
Portage 2.0.47-r10 (default-x86-1.4, gcc-3.2.2, glibc-2.3.2-r1)
=================================================================
System uname: 2.4.20-gentoo-r1 i686 AMD Athlon(tm) Processor
GENTOO_MIRRORS="http://gentoo.oregonstate.edu/
http://distro.ibiblio.org/pub/Linux/distributions/gentoo"
CONFIG_PROTECT="/etc /var/qmail/control /usr/kde/2/share/config
/usr/kde/3/share/config /usr/X11R6/lib/X11/xkb /usr/kde/3.1/share/config
/usr/share/texmf/tex/generic/config/ /usr/share/texmf/tex/platex/config/
/usr/share/config"
CONFIG_PROTECT_MASK="/etc/gconf /etc/env.d"
PORTDIR="/usr/portage"
DISTDIR="/usr/portage/distfiles"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR_OVERLAY=""
USE="x86 oss 3dnow apm avi crypt encode gif jpeg libg++ mikmod mmx mpeg ncurses
nls pdflib png quicktime spell truetype xml2 xmms xv zlib alsa gdbm berkdb slang
readline arts tetex svga java guile mysql sdl gpm tcpd pam libwww ssl perl
python esd imlib oggvorbis qt kde motif opengl mozilla cdr X gtk gnome cups tcltk"
COMPILER="gcc3"
CHOST="i686-pc-linux-gnu"
CFLAGS="-march=athlon -O3 -pipe"
CXXFLAGS="-O2 -mcpu=i686 -pipe"
ACCEPT_KEYWORDS="x86 ~x86"
MAKEOPTS="-j2"
AUTOCLEAN="yes"
SYNC="rsync://rsync.europe.gentoo.org/gentoo-portage"
FEATURES="sandbox ccache"

Now, xfree uses "xml", but I request "xml2".  Checked the ebuild for why
"xml" is being used and found:

        if [ -n "`use xml`" ]
        then
                echo "#define HasLibxml2 YES" >> config/cf/host.def
        fi

This looks wrong - the check is checking for "xml" (which is specifically
libxml v1), but the reason it appears to be doing it is to say that
libxml v2 exists.

I reckon that this should be "use xml2", not "use xml" (and hence an
updated IUSE as well).

Of course, I may be wrong...

phil

[Comment 1 Seemant Kulleen (RETIRED) 2003-04-21 14:47:35 UTC]

Phil, you are correct.  The thing is, I believe that the xml2 flag is going to die anyway.  Can the UFED people respond and let me know if the "xml2" flag is needed still?

[Comment 2 Fred Van Andel (RETIRED) 2003-04-21 15:23:14 UTC]

Currently the xml and xml2 use flags are being used by the following packages:
xml:
    app-text/htmltidy
    dev-php/mod_php
    dev-php/php
    media-sound/xmms
    net-fs/samba
    x11-base/xfree
    x11-misc/xscreensaver
xml2:
    app-misc/workrave
    app-office/abiword
    dev-libs/commoncpp2
    dev-php/mod_php
    dev-php/php
    media-gfx/imagemagick
    media-gfx/sodipodi
    media-video/vcdimager
There are still a few packages that have specific xml2 support, but I dont know enough about xml vs xml2 to answer your question.

[Comment 3 Robin Johnson 2003-04-21 22:10:24 UTC]

Both of the old libxml and the new libxml2 packages are used by various applications AFAIK. The author of both did advocate that everybody move to libxml2 where they were using libxml previously, but as it required some non-trivial changes in programs to implement, not everybody has moved yet. Therefore I think both of the xml/xml2 flags should still exist. However somebody should go thru the current set of apps that use them and check that they have the correct flag in each. Eg xfree didn't have the right one.

[Comment 4 Seemant Kulleen (RETIRED) 2003-05-30 04:04:05 UTC]

xfree-4.3.0-r3 will make use of the xml2 USE flag instead.

[Comment 5 Phil Richards 2003-06-06 02:37:55 UTC]

Please re-open this bug until it *is* fixed, please.
I fail to see the +xml2 that I would expect, but continue to see -xml.
And the ebuild script confirms that nothing has been changed...

root # emerge -v -uUDp world

These are the packages that I would merge, in order:

Calculating world dependencies ...done!
[ebuild    U ] x11-base/xfree-4.3.0-r3 [4.3.0-r2] -3dfx -sse +mmx +3dnow -xml +truetype +nls -cjk -doc -ipv6
[...]