- **status**: New --> Fixed
---
** [issues:#5648] Allow user to provide CXXFLAGS on command line**
**Status:** Fixed
**Labels:** Fixed_2_21_0
**Created:** Wed Dec 25, 2019 09:07 PM UTC by Dan Eble
**Last Updated:** Tue Dec 31, 2019 02:33 PM UTC
**Owner:** Dan Eble
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 testlilyissues-a...@lists.sourceforge.net 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
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
