On 05/23/10 04:18 PM, leif wrote:

The more evil thing is that currently *all* warnings in the
compilation of C/C++ Sage library files are suppressed by a *trailing*
"-w", so nobody will see any warning unless he/she edits setup.py

Unfortunately, this is all too common in many components of Sage. I find it very worrying indeed.

'lcalc' had a particularly annoying attempt to cover up warnings from the assembler, as it actually caused the build to break on Solaris, as the option to cover up the warnings was passed directly to the Sun assembler, but it did not accept the option.

make[1]: Entering directory `/export/home/drkirkby/L-1.23/src'
g++ -Wa,-W -O3  -Wno-deprecated  -ffast-math -fPIC  -I../include -c Lglobals.cc
/usr/ccs/bin/as: error: unknown option 'W'
usage: /usr/ccs/bin/as [-V] [-Q{y,n}] [-q] [-s]
        [-S] [-K {pic,PIC}] [-o objfile] [-L] [-T]
        [-P [[-Yc,path] [-Ipath] [-Dname] [-Dname=def] [-Uname]]...]
        [-m [-Ym,path]] [-n] [-ul] [-xF]
        [-m32] [-m64]
[-xarch={v7,v8,v8a,v8plus,v8plusa,v8plusb,v9,v9a,v9b,sparc,sparcvis, sparcvis2,sparcfmaf,sparcima}]
        [-xcode={pic13,pic32}] file.s...
make[1]: *** [Lglobals.o] Error 1

IMHO, the quality of the coding in Sage leaves a lot to be desired in places.

Dave

--
To post to this group, send an email to sage-devel@googlegroups.com
To unsubscribe from this group, send an email to 
sage-devel+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/sage-devel
URL: http://www.sagemath.org

Reply via email to