Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 284689 - stable request for media-gfx/graphviz-2.24.0-r2
Summary: stable request for media-gfx/graphviz-2.24.0-r2
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo Graphics Project
URL:
Whiteboard:
Keywords: STABLEREQ
Depends on:
Blocks: built_with_use
  Show dependency tree
 
Reported: 2009-09-12 12:51 UTC by Petteri Räty (RETIRED)
Modified: 2010-01-14 06:54 UTC (History)
6 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Petteri Räty (RETIRED) gentoo-dev 2009-09-12 12:51:22 UTC
maekke: ready for stable in order to nuke built_with_use?
Comment 1 Markus Meier gentoo-dev 2009-09-12 13:13:17 UTC
pleas do, thanks.
Comment 2 Markus Meier gentoo-dev 2009-09-14 21:21:44 UTC
amd64/arm/x86 stable
Comment 3 emerald 2009-09-15 06:47:37 UTC
Sorry to ask here, but is there a special reason that graphviz depends on gd[-fontconfig]? Till now it seemed to be working with gd with USE=fontconfig. And no changelog or ebuild comment i could find explained about the forced disabled use flag.
Comment 4 Andy Crook 2009-09-15 08:12:41 UTC
Yes, after the stabilization, we have the mess with fontconfig USE flag

I think ordinary users will be confused with such uninformative error output:

emerge: there are no ebuilds built with USE flags to satisfy ">=media-libs/gd-2.0.28[-fontconfig,jpeg,png]".
!!! One of the following packages is required to complete your request:
- media-libs/gd-2.0.35 (Change USE: -fontconfig)
(dependency required by "media-gfx/graphviz-2.24.0" [ebuild])
(dependency required by "media-gfx/imagemagick-6.5.2.9" [installed])
(dependency required by "gnome-extra/libgsf-1.14.11" [installed])
(dependency required by "gnome-base/librsvg-2.22.3" [installed])
(dependency required by "gnome-base/nautilus-2.24.2-r3" [installed])
(dependency required by "gnome-base/gnome-mount-0.8" [installed])
(dependency required by "gnome-base/gnome-vfs-2.24.1" [installed])
(dependency required by "gnome-base/libgnomeui-2.24.1" [installed])
(dependency required by "gnome-base/eel-2.24.1" [installed])

IMHO the ebuild for 2.24.0 version needs some modification

so, what's the reason for
	>=media-libs/gd-2.0.28[-fontconfig,jpeg,png]
?
Comment 5 Samuli Suominen (RETIRED) gentoo-dev 2009-09-15 08:29:24 UTC
(In reply to comment #4)
> Yes, after the stabilization, we have the mess with fontconfig USE flag
> 
> I think ordinary users will be confused with such uninformative error output:
> 
> emerge: there are no ebuilds built with USE flags to satisfy
> ">=media-libs/gd-2.0.28[-fontconfig,jpeg,png]".
> !!! One of the following packages is required to complete your request:
> - media-libs/gd-2.0.35 (Change USE: -fontconfig)

I don't know how "Change USE: -fontconfig" could be any more clear than that (unless we are talking about handholding here :-)
Comment 6 emerald 2009-09-15 09:18:03 UTC
...end after remerging gd with USE=-fontconfig it still links against fontconfig, which makes the whole fontconfig use-flag-play.....
..... confusing? useless? ????
For me it makes no sense anymore.

ren ~ # emerge -pq gd
[ebuild   R   ] media-libs/gd-2.0.35  USE="jpeg png truetype xpm -fontconfig"


ren ~ # ldd /usr/lib64/libgd.so.2.0.0
        linux-vdso.so.1 =>  (0x00007fff143ff000)
        libXpm.so.4 => /usr/lib/libXpm.so.4 (0x00007f300cc18000)
        libX11.so.6 => /usr/lib/libX11.so.6 (0x00007f300c8dd000)
        libjpeg.so.7 => /usr/lib/libjpeg.so.7 (0x00007f300c6a4000)
        libfreetype.so.6 => /usr/lib/libfreetype.so.6 (0x00007f300c40d000)
        libpng12.so.0 => /usr/lib/libpng12.so.0 (0x00007f300c1e6000)
        libz.so.1 => /lib/libz.so.1 (0x00007f300bfcf000)
        libm.so.6 => /lib/libm.so.6 (0x00007f300bd4c000)
        libc.so.6 => /lib/libc.so.6 (0x00007f300b9da000)
        libxcb.so.1 => /usr/lib/libxcb.so.1 (0x00007f300b7bb000)
        libXau.so.6 => /usr/lib/libXau.so.6 (0x00007f300b5b7000)
        libXdmcp.so.6 => /usr/lib/libXdmcp.so.6 (0x00007f300b3b1000)
        libdl.so.2 => /lib/libdl.so.2 (0x00007f300b1ac000)
        /lib64/ld-linux-x86-64.so.2 (0x00007f300d0a3000)
Comment 7 Andy Crook 2009-09-15 09:29:04 UTC
(In reply to comment #5)

> I don't know how "Change USE: -fontconfig" could be any more clear than that
> (unless we are talking about handholding here :-)

You know what I mean :)
global USE -fontconfig in make.conf is not acceptable for many people
so they should write the according string in their package.use fike
IMHO it's bad practice for a _stable_ package

PS. It's only IMHO :)
Comment 8 SpanKY gentoo-dev 2009-09-15 12:46:29 UTC
you havent given any reason as to why the gd[-fontconfig] is there.  the question is not "what does it mean", it's "wth does graphviz require gd be built with USE=-fontconfig".

if you're working around a bug instead of actually fixing it (bug 267373 ?), then this version shouldnt be going stable
Comment 9 SpanKY gentoo-dev 2009-09-15 12:48:05 UTC
especially considering `emerge graphviz --nodeps` worked on my system with gd built with USE=fontconfig.  but perhaps there is a runtime failure (not that i have any way of checking -> no docs as to why this requirement exists).
Comment 10 None 2009-09-15 13:58:18 UTC
(In reply to comment #8)
> you havent given any reason as to why the gd[-fontconfig] is there.  the
> question is not "what does it mean", it's "wth does graphviz require gd be
> built with USE=-fontconfig".
> 
> if you're working around a bug instead of actually fixing it (bug 267373 ?),
> then this version shouldnt be going stable
> 
I second this.  Some kind of comment in the ebuild explaining why -fontconfig is necessary would be nice.
Comment 11 Markus Meier gentoo-dev 2009-09-15 22:30:04 UTC
looks like I had a look at bug #267373 some time ago (and forgot to remove the change), which is only present in graphviz-2.22*. I apologize for this.
Could you guys verify that media-gfx/graphviz-2.24.0-r1 works ok?

*graphviz-2.24.0-r1 (15 Sep 2009)

  15 Sep 2009; Markus Meier <maekke@gentoo.org> -graphviz-2.24.0.ebuild,
  +graphviz-2.24.0-r1.ebuild:
  revision bump, remove false dep on media-libs/gd[-fontconfig], remove
  broken ebuild
Comment 12 Nick Bowler 2009-09-18 16:47:11 UTC
This revision bump appears to have removed the stable keywords for amd64/arm/x86 that were present on 2.24.0, forcing a downgrade.
Comment 13 SpanKY gentoo-dev 2009-09-18 17:00:59 UTC
that was by design.  the version wasnt really ready for stable yet.  once bug 267373 is resolved (it seems to be already), and the new version doesnt cause any other bugs, we will stabilize the new -r1.
Comment 14 Nick Bowler 2009-09-18 17:31:32 UTC
In the future, it'd be nice to have a short note that in the changelog when removing the most recent version of a stable package from the tree.  Something like "all stable users are recommended to downgrade to the previous version".  Otherwise, those of us with perfectly working graphviz installs are left wondering if someone just made a mistake and whether or not we should be worried about it.
Comment 15 Markus Meier gentoo-dev 2009-09-27 14:29:00 UTC
just added a new revision (wrt bug #285750), let's give it a few days before re-requesting stabilization.
Comment 16 Petteri Räty (RETIRED) gentoo-dev 2009-10-03 11:21:45 UTC
(In reply to comment #15)
> just added a new revision (wrt bug #285750), let's give it a few days before
> re-requesting stabilization.
> 

Time to do this?
Comment 17 Markus Meier gentoo-dev 2009-10-03 16:30:57 UTC
arches, please do you magic, thanks.
Comment 18 Tobias Klausmann (RETIRED) gentoo-dev 2009-10-03 18:22:08 UTC
Stable on alpha.
Comment 19 Markus Meier gentoo-dev 2009-10-15 08:10:41 UTC
amd64/x86 stable
Comment 20 Markus Meier gentoo-dev 2009-10-15 11:40:55 UTC
arm stable
Comment 21 Brent Baude (RETIRED) gentoo-dev 2009-10-31 14:31:09 UTC
ppc64 done
Comment 22 Raúl Porcel (RETIRED) gentoo-dev 2009-11-04 17:22:07 UTC
ia64/s390/sh/sparc stable
Comment 23 Jeroen Roovers (RETIRED) gentoo-dev 2010-01-14 06:54:44 UTC
Stable for HPPA and closing.