#7041: GAP purposely unsets CC which screws up Sun Studio build.
--------------------------------------------------------------------+-------
Reporter: drkirkby |
Owner: tbd
Type: defect |
Status: needs_review
Priority: major |
Milestone: sage-5.0
Component: build |
Resolution:
Keywords: gap solaris cc CXX compiler hardcoded hard-coded | Work
issues:
Report Upstream: N/A |
Reviewers:
Authors: Leif Leonhardy |
Merged in:
Dependencies: |
Stopgaps:
--------------------------------------------------------------------+-------
Changes (by {'newvalue': u'Leif Leonhardy', 'oldvalue': ''}):
* cc: rohana (added)
* status: new => needs_review
* author: => Leif Leonhardy
Old description:
> {{{spkg-install}}} for {{{gap-4.4.10.p12}}}:
>
> {{{
> echo "*WARNING*: Unsetting CC since that tends to break GAP building"
> unset CC
> echo "*WARNING*: Unsetting CXX since that tends to break GAP building"
> unset CXX
> }}}
>
> This appears to be done by Michael Abshoff as {{{SPKG.txt}}} shows:
>
> {{{
> === gap-4.4.10.p8 (Michael Abshoff, June 16th, 2008) ===
> * unset CC in spkg-install (work around for #2575)
> }}}
>
> Looking at #2575 I can't help feel there must be a better solution.
New description:
{{{spkg-install}}} for {{{gap-4.4.10.p12}}}:
{{{
echo "*WARNING*: Unsetting CC since that tends to break GAP building"
unset CC
echo "*WARNING*: Unsetting CXX since that tends to break GAP building"
unset CXX
}}}
This appears to be done by Michael Abshoff as {{{SPKG.txt}}} shows:
{{{
=== gap-4.4.10.p8 (Michael Abshoff, June 16th, 2008) ===
* unset CC in spkg-install (work around for #2575)
}}}
Looking at #2575 I can't help feel there must be a better solution.
----
'''New spkg:'''
[http://boxen.math.washington.edu/home/leif/Sage/spkgs/gap-4.4.12.p7.spkg]
'''md5sum:''' `3461058e66fd53d6d53f07acccc8d0c7 gap-4.4.12.p7.spkg`
=== gap-4.4.12.p7 (Leif Leonhardy, March 22nd 2012) ===
* #7041: Only unset `CC` and `CXX` if really necessary (i.e., if they
contain spaces, which is what currently would break the build).
* Clean up `spkg-install`. Some things seem still weird, see comments.
* Added "Special Update/Build Instructions" section above.
--
Comment:
Excerpt from (the modified) `SPKG.txt`:
== Dependencies ==
* readline (according to spkg/standard/deps)
* Sage (? also according to deps, "so that gap_reset_workspace works")
== Special !Update/Build Instructions ==
* TODO:
- Use `patch` instead of copying patched files.
(Then also add `patch` to the dependencies above.)
- "Flatten" (i.e. remove) the `build()` function.
- Perhaps check whether we can fix GAP's configure / build scripts
w.r.t. brokenness of multiple words in `CC` (and `CXX`?) and its
ignorance concerning `CFLAGS`, `CPPFLAGS` and `LDFLAGS`.
(Then also support `SAGE_DEBUG` in `spkg-install`, and probably
set up reasonable default `CFLAGS`.)
* Do we really want to copy everything from the build directory???
(Cf. comment in `spkg-install`.)
...
I'll perhaps address some of the TODOs (I added myself) later, in a p8,
but I'd really like to get this spkg in soon.
The stated (probably obsolete?) dependency on Sage and the last point
should be answered by some of '''you'''... ;-)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7041#comment:6>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
