Ticket #9712 (closed defect: fixed)
Make building PolyBoRi depend on GD
| Reported by: | leif | Owned by: | tbd |
|---|---|---|---|
| Priority: | blocker | Milestone: | sage-4.5.3 |
| Component: | packages: standard | Keywords: | |
| Cc: | leif, malb | Work issues: | |
| Report Upstream: | N/A | Reviewers: | Leif Leonhardy |
| Authors: | Mitesh Patel | Merged in: | sage-4.5.3.alpha1 |
| Dependencies: | Stopgaps: |
Description
From #9472:
"Since PolyBoRi also uses GD, $SAGE_ROOT/spkg/standard/deps should be updated (but there seem to have been no issues with that in the past)."
Attachments
Change History
Changed 3 years ago by mpatel
-
attachment
deps.diff
added
Diff of spkg/standard/deps vs. 4.5.2 + #8316.
comment:1 follow-up: ↓ 2 Changed 3 years ago by mpatel
- Status changed from new to needs_review
I've attached a new deps and deps.diff.
I'm not sure whether to put Leif or Martin as the author, or both Leif and Martin. Can someone please update the field?
comment:2 in reply to: ↑ 1 Changed 3 years ago by leif
- Authors set to Mitesh Patel
Replying to mpatel:
I'm not sure whether to put Leif or Martin as the author, or both Leif and Martin. Can someone please update the field?
Can you update the "Reported by:" field? ;-)
From PolyBoRi's SPKG.txt:
... == Dependencies == * Python * Scons * Boost * gd (FIXME/TODO: should be added to deps, I think. Leif, 2010-07-10) * M4RI/libm4ri (already included in deps) * png/libpng12 (accomplished because Python and gd depend on it, too) * libz (accomplished because e.g. libpng depends on it) ...
I must admit I did not check all transitive dependencies, i.e. if some package that PolyBoRi (indirectly) depends on pulls in the GD dependency. Anyway, I think it's always better to add an explicit, perhaps "redundant" dependency rather than to omit it, (not only) since the deps of other packages might change over the time. (Same for explicitly linking against libraries directly used despite other used libraries - currently - already doing so.)
The only package in standard/deps that's listed to directly depend on GD is gdmodule, which in turn is only listed as a (direct) dependency of MatPlotLib (and the Sage library).
comment:3 Changed 3 years ago by leif
- Status changed from needs_review to positive_review
- Reviewers set to Leif Leonhardy
I've only "reviewed" deps.diff though... :)
comment:5 Changed 3 years ago by leif
I'm pretty sure it's just good luck this ever worked. GD is one of the "early" packages, and all of these take little time to build.
I'll perhaps verify that later by forcing GD to be built very late.

Updated spkg/standard/deps. Based on 4.5.2 + #8316.