First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 173676
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: media-gfx herd <graphics@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Axel Dyks <XL@XLsigned.net>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
graphviz-2.12-parallel-build.patch patches "./tclpkg/gv/Makefile.am" patch Axel Dyks 2007-04-07 21:34 0000 2.98 KB Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 173676 depends on: Show dependency tree
Bug 173676 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2007-04-07 17:03 0000
Unfortunately bug #167484 has been closed without reason,
although a "parallel build issue" occurs in folder "./tclpkg/gv"
which is caused my invalid "mutiple target rules" in the Makefile.
The following rules (given as an example. The "ruby", "java", ...
rules are broken as well)

-- original ------------------------------------------
gv_perl.cpp $(PERL_hdr): gv.i
    $(SWIG) -c++ -perl -o gv_perl.cpp $(srcdir)/gv.i
gv_python.cpp $(PYTHON_hdr): gv.i
    $(SWIG) -c++ -python -o gv_python.cpp $(srcdir)/gv.i
------------------------------------------------------

should be rewritten as

-- modified ------------------------------------------
$(PERL_hdr): gv_perl.cpp
gv_perl.cpp: gv.i
    $(SWIG) -c++ -perl -o gv_perl.cpp $(srcdir)/gv.i
gv_php.cpp $(PHP_hdr): gv.i
    $(SWIG) -c++ -php -o gv_php.cpp $(srcdir)/gv.i
$(PYTHON_hdr): gv_python.cpp
gv_python.cpp: gv.i
    $(SWIG) -c++ -python -o gv_python.cpp $(srcdir)/gv.i
------------------------------------------------------

This not only prevents swig from being run _twice_, but also
prevents swig to overwrite gv_python.cpp, gc_perl.cpp on
the second run when building gv.py, pv.pm which might occur
(-jN) while gv_python.cpp, gc_perl.cpp are already being
compiled.

Actually this should better be fixed upstream and/or I
should have attached a patch for Makefile.am, but maybe
I'll do this later ...

Cheers
Axel

------- Comment #1 From Jakub Moc (RETIRED) 2007-04-07 17:05:44 0000 -------
*** Bug 167484 has been marked as a duplicate of this bug. ***

------- Comment #2 From Axel Dyks 2007-04-07 21:34:01 0000 -------
Created an attachment (id=115683) [edit]
patches "./tclpkg/gv/Makefile.am"

... so that all but the IO and LUA multi-target rules that call "swig"
are split into separate rules which avoids duplicate envocations of
"swig" and parallel build issues.
// Both IO_hdr and LUA_hdr are empty, so there is no need to
// modify those rules

Axel

------- Comment #3 From Axel Dyks 2007-04-10 01:02:44 0000 -------
bug #141132 might be a duplicate of this bug

------- Comment #4 From Axel Dyks 2007-04-11 03:54:52 0000 -------
I've forwarded the bug report as well as the patch to upstream's
bug tracking system:

  http://www.graphviz.org/bugs/b1111.html

Axel

------- Comment #5 From Axel Dyks 2007-04-21 05:38:19 0000 -------
It appears that neither upstream nor media-gfx herd are taking
further steps on this matter ...

... which is a little hard to understand, because according to
Murphy's Law "emerge --update ... world" (or any other command
that pulls in the graphviz ebuild ) will typically fail a few
minutes after leaving it unattended and going to bed.     

:-( Axel

------- Comment #6 From Jakub Moc (RETIRED) 2007-06-13 16:26:36 0000 -------
*** Bug 181925 has been marked as a duplicate of this bug. ***

------- Comment #7 From Markus Ullmann 2007-06-13 16:31:23 0000 -------
*** Bug 141132 has been marked as a duplicate of this bug. ***

------- Comment #8 From Markus Ullmann 2007-07-24 22:06:45 0000 -------
Patch committed as it has been accepted upstream

First Last Prev Next    No search results available      Search page      Enter new bug