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.

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

configure.ac: add --enable-vanilla-opt configure option #1207

Closed
wants to merge 1 commit into from
Closed

configure.ac: add --enable-vanilla-opt configure option #1207

wants to merge 1 commit into from

Conversation

trofi
Copy link
Contributor

@trofi trofi commented Jan 25, 2017

Usually autotools-based projects allow uses specify
CFLAGS/CXXFLAGS as an environment variable at ./configure time.

Many distributions exploit it to inject distribution-specific
compile-time options like:

  • enable additional debugging (-ggdb3)
  • enable sanitisers (-fsanitize=address|thread|undefined)
  • enable Position Independent Executables (-fPIE)

But nix by default uses it's own set of options mostly to:

  • enable -O3 by default
  • have unoptimised build with 'make OPTIMIZE=0'

This commit leaves nix default but allows falling back
to autotools-style builds by using --enable-vanilla-opt:
$ ./configure --enable-vanilla-opt ...
$ make

Signed-off-by: Sergei Trofimovich siarheit@google.com

Usually autotools-based projects allow uses specify
CFLAGS/CXXFLAGS as an environment variable at ./configure time.

Many distributions exploit it to inject distribution-specific
compile-time options like:
- enable additional debugging (-ggdb3)
- enable sanitisers (-fsanitize=address|thread|undefined)
- enable Position Independent Executables (-fPIE)

But nix by default uses it's own set of options mostly to:
- enable -O3 by default
- have unoptimised build with 'make OPTIMIZE=0'

This commit leaves nix default but allows falling back
to autotools-style builds by using --enable-vanilla-opt:
  $ ./configure --enable-vanilla-opt ...
  $ make

Signed-off-by: Sergei Trofimovich <siarheit@google.com>
@trofi
Copy link
Contributor Author

trofi commented Jun 23, 2018

ping :)

@trofi
Copy link
Contributor Author

trofi commented Jul 3, 2019

Obsoleted by 717e821

@trofi trofi closed this Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants