Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bring in C++ configuration change to mirror flux-core #270

Merged
merged 4 commits into from
Sep 27, 2017
Merged

Bring in C++ configuration change to mirror flux-core #270

merged 4 commits into from
Sep 27, 2017

Commits on Sep 27, 2017

  1. config: Convert @GCCWARN@ to $(WARNING_CFLAGS) 

    Change the name of GCCWARN to WARNING_CFLAGS to eliminate
the gcc-specificity from the name.  Granted it is unlikely, but
WARNING_CFLAGS might potentially have different values depending
on the detected compiler, without requiring changes to each
Makefile.am.  The name is also now in keeping with our other
*_CFLAGS named variables.

Also, we change from @GCCWARN@ to the more common variable usage
$(WARNING_CFLAGS).

This will help a little bit with coming C++ flags.
    @morrone
    morrone committed Sep 27, 2017
    Configuration menu
    Copy the full SHA
    c258209 View commit details
    Browse the repository at this point in the history

  2. config: Update to a newer ax_code_coverage.m4 

    Update ax_code_coverage.m4 to the latest from
 https://www.gnu.org/software/autoconf-archive/ax_code_coverage.html.

This new version supports C++ with a CODE_COVERAGE_CXXFLAGS variable.

Rename CODE_COVERAGE_LDFLAGS to CODE_COVERAGE_LIBS since the newer
m4 file deprecates the former.
    @morrone
    morrone committed Sep 27, 2017
    Configuration menu
    Copy the full SHA
    360e284 View commit details
    Browse the repository at this point in the history

  3. config: Introduce WARNING_CXXFLAGS 

    Introduce the WARNING_CXXFLAGS variable.
    @morrone
    morrone committed Sep 27, 2017
    Configuration menu
    Copy the full SHA
    e2efec5 View commit details
    Browse the repository at this point in the history

  4. config: Require that C++ use the C++11 standard 

    Introduce the AX_CXX_COMPILE_STDCXX macro from
 https://www.gnu.org/software/autoconf-archive/ax_cxx_compile_stdcxx.html.

Employ the macro to set the C++ compiler to use C++11.
    @morrone
    morrone committed Sep 27, 2017
    Configuration menu
    Copy the full SHA
    4c21db9 View commit details
    Browse the repository at this point in the history