search

estraier / tkrzw

a set of implementations of DBM
Apache License 2.0
164 stars 20 forks source link

configure.in: hardcoded special compiler flags #7

Closed tieugene closed 3 years ago

tieugene commented 3 years ago

As some|many building systems provide they own building plan (verbosity, optimisations, debug info etc) please exclude special flags from defaults. Like this:

-MYCFLAGS="-pthread -Wall -ansi -pedantic -fPIC -fsigned-char -g0 -O2"
+MYCFLAGS="-pthread -fPIC -fsigned-char"
-MYCXXFLAGS="-std=c++17 -pthread -Wall -fPIC -fsigned-char -g0 -O2"
+MYCXXFLAGS="-std=c++17 -pthread -fPIC -fsigned-char"

I propose to add something like '--enable-opt` into configure options instead of hardcoded.

estraier commented 3 years ago

MYCFLAGS and MYCXXFLAGS have been simplified as described above. You can get the previous behavior by setting "--enable-devel" option.