Opened 14 years ago

Closed 11 years ago

Last modified 11 years ago

#2999 closed defect (duplicate)

Some packages don't respect the CC environment variable

Reported by: dfdeshom Owned by: Michael Abshoff
Priority: critical Milestone: sage-duplicate/invalid/wontfix
Component: build Keywords:
Cc: Michael Orlitzky Merged in:
Authors: Reviewers: Michael Orlitzky, R. Andrew Ohana
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by Leif Leonhardy)

Packages which seem to not honor the CC environment variable (they use "gcc"):

flint-1.06.p2
atlas-3.8.1.p1
f2c-20070816.p0
symmetrica-2.0.p2
polybori-0.3.1.p1
rubiks-20070912.p5
zn_poly-0.8.p0
sage-3.0.rc1
gap-4.4.10.p7 // guava3.4
tachyon-0.98beta.p5
palp-1.1.p1

Still not fixed:

  • ratpoints-2.1.3.p2 (see #12682)
  • zn_poly-0.9.p5 (see #12433)

Attachments (1)

pbori-custom_py.patch (394 bytes) - added by PolyBoRi team 14 years ago.
CC/CXX patch

Download all attachments as: .zip

Change History (23)

comment:1 Changed 14 years ago by PolyBoRi team

For PolyBoRi? see the patch for the custom.py file. Feel free to add additional Variables there

Good Night,

Alexander

Changed 14 years ago by PolyBoRi team

Attachment: pbori-custom_py.patch added

CC/CXX patch

comment:2 Changed 14 years ago by Michael Abshoff

Status: newassigned

comment:3 Changed 13 years ago by Minh Van Nguyen

Ticket #6437 has an updated spkg so that polybori can be built under Solaris.

comment:4 Changed 13 years ago by Dan Drake

There are other spkgs which also fail to respect CC: from http://groups.google.com/group/sage-devel/msg/a9192a6b51a74d22 this thread, I see the following spkgs which are not listed above:

  • cliquer-1.2
  • symmetrica-2.0.p4
  • ratpoints-2.1.2.p2

comment:5 Changed 13 years ago by Minh Van Nguyen

Cliquer should respect the CC environment variable now. See ticket #6681.

comment:6 Changed 11 years ago by Michael Orlitzky

Cc: Michael Orlitzky added
Description: modified (diff)
Report Upstream: N/A

The f2c package is fixed at #7027, so I've removed it from the list.

comment:7 Changed 11 years ago by Michael Orlitzky

Description: modified (diff)

symmetrica fixed at #10719, so removing that too.

comment:8 Changed 11 years ago by Michael Orlitzky

Description: modified (diff)

flint and zn_poly have their own tickets at #7024 and #12433 respectively, so there's no need to duplicate them here.

comment:9 Changed 11 years ago by Michael Orlitzky

Description: modified (diff)

Polybori also respects $CC now, although I can't pin down the ticket where it went from doesn't-work-at-all to something else.

comment:10 Changed 11 years ago by Michael Orlitzky

Description: modified (diff)

Tachyon should have been fixed by #9379 and #10681. Waiting on #7069 to confirm on Solaris.

comment:11 Changed 11 years ago by Michael Orlitzky

Description: modified (diff)

And gap was fixed at #2575 and #4161...

comment:12 Changed 11 years ago by Michael Orlitzky

Description: modified (diff)

Rubiks fixed at #7036.

comment:13 Changed 11 years ago by Michael Orlitzky

Description: modified (diff)

Working on palp at #7071.

comment:14 Changed 11 years ago by Volker Braun

ATLAS supports CC since atlas-3.8.3.p18.

comment:15 Changed 11 years ago by Michael Orlitzky

Description: modified (diff)
Milestone: sage-5.0sage-duplicate/invalid/wontfix
Status: newneeds_review

Ok, atlas was fixed in #10226 and we have a ticket for the sage library at #12443. I replaced the initial list for review.

comment:16 Changed 11 years ago by R. Andrew Ohana

Authors: Michael Orlitzky
Reviewers: R. Andrew Ohana
Status: needs_reviewpositive_review

Yup, there are plenty of tickets regarding all of these packages -- some from me with the clang port, some from David Kirby with the Sun CC port.

comment:17 Changed 11 years ago by Michael Orlitzky

Thanks, I did the same thing with the $CXX ticket at #3000.

comment:18 in reply to:  9 Changed 11 years ago by Alexander Dreyer

Replying to mjo:

Polybori also respects $CC now, although I can't pin down the ticket where it went from doesn't-work-at-all to something else.

That was #6437 as mentioned above.

comment:19 Changed 11 years ago by Jeroen Demeyer

Authors: Michael Orlitzky
Resolution: duplicate
Reviewers: R. Andrew OhanaMichael Orlitzky, R. Andrew Ohana
Status: positive_reviewclosed

comment:20 in reply to:  16 ; Changed 11 years ago by Leif Leonhardy

Description: modified (diff)

Replying to ohanar:

Yup, there are plenty of tickets regarding all of these packages -- some from me with the clang port, some from David Kirby with the Sun CC port.

AFAIK at least ratpoints doesn't [yet] have its own ticket; I would have left this ticket open as a meta-ticket until all issues (or spkgs) have really been fixed.

comment:21 in reply to:  11 Changed 11 years ago by Leif Leonhardy

Replying to mjo:

And gap was fixed at #2575 and #4161...

Aha. I knew CC was "intentionally" unset in GAP's spkg-install for a long time (which was annoying anyway), but now I still get:

gcc version 4.6.3 (GCC) 
****************************************************
*WARNING*: Unsetting CC since that tends to break GAP building
*WARNING*: Unsetting CXX since that tends to break GAP building
checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-unknown-linux-gnu
checking target system type... x86_64-unknown-linux-gnu
checking for gcc... gcc
checking for C compiler default output file name... 
configure: error: C compiler cannot create executables
See `config.log' for more details.
Configuration of GAP failed.

real	0m0.793s
user	0m0.160s
sys	0m0.050s
************************************************************************
Error installing package gap-4.4.12.p6
************************************************************************

So if there's been an issue with CC and CXX set, it might have been fixed upstream (I believe so), but it isn't fixed in Sage.

[The problem here simply is that the "default" gcc, which is 4.4.3, doesn't understand some of the options I pass in CFLAGS. GCC 4.6.3, specified in CC, of course does understand them.]

comment:22 in reply to:  20 Changed 11 years ago by Leif Leonhardy

Description: modified (diff)

Replying to leif:

Replying to ohanar:

Yup, there are plenty of tickets regarding all of these packages -- some from me with the clang port, some from David Kirby with the Sun CC port.

AFAIK at least ratpoints doesn't [yet] have its own ticket [...]

This (ratpoints) is now #12682 (needing review).

Note: See TracTickets for help on using tickets.