[PATCH] Drop redundant CFLAGS, was already included in CXXFLAGS

classic Classic list List threaded Threaded
2 messages Options
Jed Brown Jed Brown
Reply | Threaded
Open this post in threaded view
|

[PATCH] Drop redundant CFLAGS, was already included in CXXFLAGS

---
 Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

0001-Drop-redundant-CFLAGS-was-already-included-in-CXXFLA.patch (449 bytes) Download Attachment
Carl Worth-2 Carl Worth-2
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Drop redundant CFLAGS, was already included in CXXFLAGS

On Fri, 20 Nov 2009 16:58:15 +0100, Jed Brown <[hidden email]> wrote:
>  %.o: %.cc $(all_deps)
> - $(CXX) -c $(CFLAGS) $(CXXFLAGS) $< -o $@
> + $(CXX) -c $(CXXFLAGS) $< -o $@

Thanks, Jed!

It was nice to get the patch, (which is pushed now).

But it was even nicer to get the encouragement to display text
attachments *and* to include quoted text attachments in replies. I've
often been frustrated by email programs that wouldn't quote patch
attachments in replies for sake of review. So now at least my email
program no longer has that bug.

-Carl

PS. GPG signatures are no longer silently dropped when presenting
messages. We don't yet present the full signature yet, (and in
particular do not yet verify it), but you can at least now see whether a
signature is present.