patchGNU Autoconf Archive - Patches: patch #8818, AX_CXX_COMPILE_STDCXX: don't break...

 
 

patch #8818: AX_CXX_COMPILE_STDCXX: don't break "make CXXFLAGS=..."

Submitter:  Joshua Judson Rosen <rozzin>
Submitted:  Wed 02 Dec 2015 09:54:10 PM UTC
   
 
Priority:  5 - Normal Status:  Done
Assigned to:  simons Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 29 Feb 2016 05:24:56 PM UTC, comment #2: 

Thanks, I think that this is a good thing.

Moritz Klammler <moritz_klammler>
Mon 29 Feb 2016 10:42:15 AM UTC, comment #1: 

I've pushed this patch in 9e5ef8a7fa6f32dce37a9a340cef793254d2396b. I've also added previous contributors to give them a chance to comment on the modification.

Peter Simons <simons>
Group administrator
Wed 02 Dec 2015 09:54:10 PM UTC, original submission:  

Make C++ std-selection macros modify $(CXX), not $(CXXFLAGS).

Using $(CXXFLAGS) prevents users from being able to
rebuild with different CXXFLAGS (e.g.: to try out
different optimisations or to change debug levels)
unless they re-run ./configure with their CFLAGS.

This is more like what the mainline Autoconf macros do
(e.g.: AC_PROG_CC_C99, AC_PROG_CC_STD, etc. modify $(CC),
not $(CCFLAGS)), presmuably for much the same reasons:
if you need to specify a standard to the compiler,
it doesn't really make sense to separate that from
$(CC)/$(CXX) since $(CC)/$(CXX) aren't usable without it.

Joshua Judson Rosen <rozzin>

 

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

Attach Files:
   
   
Comment:
   

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by moritz_klammler (Posted a comment)
  • -email is unavailable- added by simons (Posted a comment)
  • -email is unavailable- added by simons (Previous contributors)
  • -email is unavailable- added by simons (Previous contributors)
  • -email is unavailable- added by simons (Previous contributors)
  • -email is unavailable- added by simons (Previous contributors)
  • -email is unavailable- added by simons (Previous contributors)
  • -email is unavailable- added by simons (Previous contributors)
  • -email is unavailable- added by rozzin (Submitted the item)
  •  

    Follow 11 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-03-20 simons StatusReady For Test Done
        Open/ClosedOpen Closed
    2016-02-29 simons StatusNone Ready For Test
        Assigned toNone simons
        Carbon-Copy- Added benjamin kosnik <bkoz@redhat.com>
        Carbon-Copy- Added zack weinberg <zackw@panix.com>
        Carbon-Copy- Added roy stogner <roystgnr@ices.utexas.edu>
        Carbon-Copy- Added alexey sokolov <sokolov@google.com>
        Carbon-Copy- Added paul norman <penorman@mac.com>
        Carbon-Copy- Added moritz klammler <moritz@klammler.eu>
    2015-12-02 rozzin Attached File- Added 0001-AX_CXX_COMPILE_STDCXX-don-t-break-make-CXXFLAGS.patch, #35620

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code