Bug 322 - configure does not respect CFLAGS variable
configure does not respect CFLAGS variable
Status: RESOLVED FIXED
Product: unbound
Classification: Unclassified
Component: server
1.4.6
Sun other
: P2 normal
Assigned To: unbound team
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-08-18 10:57 CEST by Ihsan Dogan
Modified: 2010-08-18 12:46 CEST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ihsan Dogan 2010-08-18 10:57:17 CEST
In configure.ac is defined:

# for Sun studio 11.                                                                                   
ACX_CHECK_COMPILER_FLAG(xO4, [CFLAGS="$CFLAGS -xO4"])                                                  
ACX_CHECK_COMPILER_FLAG(xtarget=generic, [CFLAGS="$CFLAGS -xtarget=generic"])

This genereates a SPARC32PLUS binary and any CFLAGS are ignored. Instead, the configure script should respect the CFLAGS variables and settings like this should be only available with the --enable-optimisations option.
Comment 1 Wouter Wijngaards 2010-08-18 11:13:29 CEST
Hi Ihsan,  What is it that you want, these checks removed?  Only the xtarget=generic one or also xO4?  It would be nice to enable (some level of) optimisation for the sun studio compiler, by default, I think.

Best regards,   Wouter
Comment 2 Ihsan Dogan 2010-08-18 12:03:09 CEST
(In reply to comment #1)
> Hi Ihsan,  What is it that you want, these checks removed?  Only the
> xtarget=generic one or also xO4?  It would be nice to enable (some level of)
> optimisation for the sun studio compiler, by default, I think.
> 
> Best regards,   Wouter

Hello Wouter,

Thanks for the quick reply. I would prefer to have both removed. Would it be possible, as a compromise, to add the --disable-optimisation flag to the configure script? 



Greetings, Ihsan
Comment 3 Wouter Wijngaards 2010-08-18 12:46:54 CEST
Hi Ihsan,

Commit to svn trunk r2216 has them removed.  I added a note in the README that may encourage people to pass such values to configure on Solaris for sun-cc.

Best regards,
   Wouter