|
From: | Auto mailings of changes to Lily Issues via Testlilyissues-auto |
Subject: | [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5648 Allow user to provide CXXFLAGS on command line |
Date: | Wed, 25 Dec 2019 21:07:06 -0000 |
[issues:#5648] Allow user to provide CXXFLAGS on command line
Status: Started
Created: Wed Dec 25, 2019 09:07 PM UTC by Dan Eble
Last Updated: Wed Dec 25, 2019 09:07 PM UTC
Owner: nobody
I've found these changes helpful while experimentally enabling some
optional warnings. I'm submitting these on their own because I'm not
sure how much longer I'll need to finish up the related warning work.
1: move -Woverloaded-virtual from CXXFLAGS to EXTRA_CXXFLAGS
2: move CXXFLAGS to the end of ALL_CXXFLAGS
This allows CXXFLAGS=... on the command line to override flags in
ALL_CXXFLAGS.
3: Stop copying FlexLexer.h
Leave FlexLexer.h where it is, rather than copying it to $(outdir).
As we enable more warnings, we will not want to hear about problems in
this file. g++ automatically suppresses warnings for headers found in
system include directories (specified with -isystem), so the simple
solution is to leave it where it is.
https://codereview.appspot.com/551300043
Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
_______________________________________________ Testlilyissues-auto mailing list address@hidden https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Prev in Thread] | Current Thread | [Next in Thread] |