[BUG SUB] silc toolkit contains gcc-isims and CFLAGS when it shouldn\'t - silc toolkit

[Pekka Riikonen]

: Software: silc toolkit
: Version: 1.1
: Operating System: Solaris
: Installation: source
: Severity: non-critical
: 
: First, CFLAGS has -g -O appended to it no matter what I set. This is 
: broken. CFLAGS should be left blank and set by the user if they want 
: them set. I already set some debug and optimization flags and this 
: overrides them.
: 
The -g is used with all compilers, not just with GCC, so that even 
non-debug version can be debugged for bugs if needed.  The CFLAGS can be 
defined by user but the ./configure does remove all -O options from it 
because we want to control optimizations (it should preserve other user 
specified options).  Some parts of the code are compiled with different 
optimization levels.  If these options don't do with your compiler what we 
expect them to do, that is the real bug, and I'd appreciate knowing what 
compiler/OS you were using (what configure said you were using) so that we 
can set specific case for that.

: Second, MATH_CFLAGS is set and used in some locations and its gcc 
: specific. That should either be an option or env variable to set at 
: configure time.
: 
That's one of the parts we want to compile and optimize differently, hence 
the MATH_CFLAGS.  Even if I'd add support for MATH_CFLAGS environment 
variable we'd still set our default, mandatory, options for all compilers.

	Pekka
________________________________________________________________________
 Pekka Riikonen                                 priikone at silcnet.org
 Secure Internet Live Conferencing (SILC)       http://silcnet.org/