Skip to content
This repository was archived by the owner on Aug 22, 2025. It is now read-only.

Silent build#17

Open
bregma wants to merge 3 commits intoKamal-Sadek:masterfrom
bregma:silent-build
Open

Silent build#17
bregma wants to merge 3 commits intoKamal-Sadek:masterfrom
bregma:silent-build

Conversation

@bregma
Copy link

@bregma bregma commented Nov 11, 2017

Enable silent building when using an autotools build.

The amount of noise coming from an autotools build is so high that it's hard to see warnings and errors. This PR sets the default to silent builds to help the errors and warning pop.

Noisy builds can be enabled at any time with the --disable-silent-rules option passed to configure or by using make V=1.

Using silent rules in automake builds makes build errors just pop instead of
getting lost in the noise.
Build noise was being generated because of a redefinition of VERSION_STRING when
building using autotools.  Silenced it by ifdefing the Windows-specific
hardcoded version string definition if the autotools are in use.
The variable $CSSFLAGS is reserved for user use and should not be forced by the
build machinery (like the autotools).  This prevents using fancy-schamncy things
like -Wall from the command line.

This change moves a few things around so that the flags required by the build
machinery (eg. -std=c++11) use the AM_CXXFLAGS reserved for that purpose and
leaves CXXFLAGS free for appropriate use.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant