Skip to content

Commit

Permalink
Update LDFLAGS to reduce linker warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
sergeyklay committed Sep 27, 2019
1 parent 5fd62ec commit 3c619c0
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
## [Unreleased]
### Fixed
- Fixed underscore only identifiers to allow more than 4 characters
- Update LDFLAGS to reduce linker warnings

## [1.3.1] - 2019-05-01
### Fixed
Expand Down
9 changes: 7 additions & 2 deletions config.m4
Original file line number Diff line number Diff line change
Expand Up @@ -110,11 +110,16 @@ You can disable ccache by setting environment variable CCACHE_DISABLE=1.
CFLAGS=`echo "$CFLAGS" | $SED -e 's/-O[0-9s]*//g'`
CXXFLAGS=`echo "$CXXFLAGS" | $SED -e 's/-O[0-9s]*//g'`
dnl Remove --coverage flag from LDFLAGS
LDFLAGS=`echo "$LDFLAGS" | $SED -e 's/--coverage//g'`
LDFLAGS=`echo "$LDFLAGS" | $SED -e 's/--coverage)//g' -e 's/-fprofile-arcs//g' -e 's/-ftest-coverage//g'`
changequote([,])

dnl Add the special flags
LDFLAGS="$LDFLAGS --coverage"
if test "$($CC --version | head -n 1 | cut -d' ' -f1)" = "Apple"; then
LDFLAGS="$LDFLAGS -fprofile-arcs -ftest-coverage"
else
LDFLAGS="$LDFLAGS -coverage"
fi

CFLAGS="$CFLAGS -O0 -ggdb -fprofile-arcs -ftest-coverage"
CXXFLAGS="$CXXFLAGS -O0 -ggdb -fprofile-arcs -ftest-coverage"

Expand Down

0 comments on commit 3c619c0

Please sign in to comment.