Bug 704529

Summary: graphviz fails to build with guile-2.0.1
Product: [Fedora] Fedora Reporter: Miroslav Lichvar <mlichvar>
Component: graphvizAssignee: Jaroslav Škarvada <jskarvad>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: alex, jima, john.ellson, jskarvad, pbrobinson, tremble
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-02-15 16:59:15 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 706140    
Bug Blocks: 678238    
Attachments:
Description Flags
Proposed fix
none
Proposed fix none

Description Miroslav Lichvar 2011-05-13 14:12:35 UTC
The guile package in rawhide is planned for update to a new major release.
In a test rebuild of all packages depending on guile, graphviz has failed to build.

Rebuild results:
http://mlichvar.fedorapeople.org/tmp/guile/rebuild/

New guile packages:
http://mlichvar.fedorapeople.org/tmp/guile/

Please close this bug if a new upstream version of graphviz which works
with the new guile is available or if you think the current version can be
easily fixed. It doesn't have to be commited in fedora git, we are just trying
to find out if we need to delay the guile update (or make a guile-compat
package).

Comment 1 Jaroslav Škarvada 2011-05-19 15:40:23 UTC
Created attachment 499879 [details]
Proposed fix

It builds fine with proposed fix and patched swig (bug 706140).

Comment 2 Jaroslav Škarvada 2011-05-19 17:54:29 UTC
Upstream ticket:
http://www.graphviz.org/mantisbt/view.php?id=2091

Comment 3 John Ellson 2011-05-19 18:37:56 UTC
Many thanks for pushing this bug upstream.  (Can bugzilla be trained to do that automatically? )

Patch has been applied to graphviz-2.29 development series.

Can I assume that you will apply this as a fedora-distro patch for graphviz-2.28 when guile-2 arrives?

Comment 4 Jaroslav Škarvada 2011-05-20 07:14:11 UTC
(In reply to comment #3)
> Many thanks for pushing this bug upstream.  (Can bugzilla be trained to do that
> automatically? )
>
That would be my dream bugzilla :)

> Patch has been applied to graphviz-2.29 development series.
> 
Thanks.

> Can I assume that you will apply this as a fedora-distro patch for
> graphviz-2.28 when guile-2 arrives?
>
Sure.

Comment 5 Jaroslav Škarvada 2011-05-20 07:26:19 UTC
> Can I assume that you will apply this as a fedora-distro patch for
> graphviz-2.28 when guile-2 arrives?
I will probably use the patch attached to this bugzilla not the one send upstream, because we currently do not use the autogen.sh when packaging graphviz, but the principle behind the patch is the same.

Comment 6 Peter Robinson 2013-02-11 23:06:04 UTC
Reopening. guile 2.0 has landed (finally) in rawhide and graphviz doesn't build with it

Comment 7 John Ellson 2013-02-13 03:31:40 UTC
Yes, sadly, the recently released "stable" graphviz-2.30.0 fails to build with the even-more recently released changes to guile (and also php and ruby) on rawhide.    

These are fixed in the graphviz-2.31 development series (fixes primarily in configure.ac), and we expect to do a graphiz-2.30.1 point release in a week or so to include these, and a few windows fixes.

Comment 8 Jaroslav Škarvada 2013-02-13 09:02:14 UTC
(In reply to comment #7)
I had this issue on my (pretty long) todo list :) Really thanks for the upstream fix, I will wait for the 2.30.1 release.

Comment 9 John Ellson 2013-02-14 19:02:14 UTC
graphviz-2.30.1 (stable release) is now available from http://www.graphviz.org/

It contains a fix for this problem.

Comment 10 Jaroslav Škarvada 2013-02-15 10:56:17 UTC
Created attachment 697673 [details]
Proposed fix

(In reply to comment #9)
> graphviz-2.30.1 (stable release) is now available from
> http://www.graphviz.org/
> 
> It contains a fix for this problem.

Thanks.

To build with latest guile (2.0.7) I had to apply the attached patch.