bugGNU Common Lisp - Bugs: bug #16697, Makefile adds gcc specific...

 
 

bug #16697: Makefile adds gcc specific compiler options when not using gcc.

Submitter:  Dr. David Kirkby <drkirkby>
Submitted:  Mon 29 May 2006 09:25:45 PM UTC
Votes: 100
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 15 Sep 2006 08:10:08 PM UTC, comment #3: 

I've tested this on a solaris machine.  Please let me know if any bad flags remain for you.

Camm Maguire <camm>
Group administrator
Fri 15 Sep 2006 08:08:15 PM UTC, comment #2: 

Greetings!  Fixed in 2.6.8pre and cvs head.
Take care,

Camm Maguire <camm>
Group administrator
Mon 29 May 2006 09:28:44 PM UTC, comment #1: 

I should add -Wall is not the only gcc specific flag added. I don't think either -fomit-frame-pointer or -fsigned-char will not work on Sun's C compiler.

Please, respect compiler flags when the user has indicated what is needed, and don't blindly add gcc specific flags.

Dr. David Kirkby <drkirkby>
Mon 29 May 2006 09:25:45 PM UTC, original submission:  

It seems to me that 40% of the open-source world thinks that gcc is the only compiler, and another 40% that Linux is the only UNIX-like operating system.

Well gcl-2.6.7 has this problem. I'm compiling with the tcsh, and have set the copiler and compiler flags set by the enviroment variables CC and CFLAGS.

sparrow /tmp/gcl-2.6.7 % printenv CC
/opt/SUNWspro/bin/cc
sparrow /tmp/gcl-2.6.7 % printenv CFLAGS
-fast -fsimple=1

So I'm not intending using gcc. The configure script seems to work this out too:

checking whether we are using GNU C... (cached) no
checking whether /opt/SUNWspro/bin/cc accepts -g... (cached) yes
checking how to run the C preprocessor... (cached) /opt/SUNWspro/bin/cc -E

But next one finds that the makefiles have the gcc specific compiler options -Wall

sparrow /tmp/gcl-2.6.7 % gmake
(cd o && gmake ../h/new_decl.h)
gmake[1]: Entering directory `/tmp/gcl-2.6.7/o'
/opt/SUNWspro/bin/cc -c -fast -fsimple=1 -Wall -DVOL=volatile -fsigned-char -O3 -fomit-frame-pointer  -I/tmp/gcl-2.6.7/o -I../h -I../gcl-tk plttest.c
cc: illegal option -Wall
gmake[1]: * [plttest.o] Error 1
gmake[1]: Leaving directory `/tmp/gcl-2.6.7/o'
gmake: * [h/new_decl.h] Error 2

Dr. David Kirkby <drkirkby>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #10074:  con.out added by drkirkby (19KiB - application/octet-stream - The results from the configure script - stderr and stdout are redirceted to this. )

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 100 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

Only logged-in users can vote.

 

Follow 4 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2006-09-15 camm Open/ClosedOpen Closed
2006-09-15 camm StatusNone Fixed
2006-05-29 drkirkby Attached File- Added con.out, #10074
    Carbon-Copy- Added drkirkby

Back to the top

Powered by Savane 3.13-02a9.
Corresponding source code