On Sun, Oct 29, 2017 at 07:32:39PM +0800, Christopher Li wrote: > On Thu, Oct 26, 2017 at 8:11 PM, Jeff Layton <jlayton@xxxxxxxxxx> wrote: > > > > Perhaps instead of describing it, you could roll up an alternative patch > > that shows what you would prefer to see here? > > Here it is the patch. > > I push it on https://git.kernel.org/pub/scm/devel/sparse/chrisl/sparse.git/log/?h=cflags > > Chris > > From 9fc170ec5e5814ae858922496dea668bb80dbf34 Mon Sep 17 00:00:00 2001 > From: Christopher Li <sparse@xxxxxxxxxxx> > Date: Sun, 29 Oct 2017 19:16:44 +0800 > Subject: [PATCH] Makefile: provide CFLAGS for command line override. > > Avoid assign to CFLAGS in Makefile. > Rename BASIC_CFLAGS to COMMON_CFLAGS. > Use PKG_CFLAGS to store external package related cflags. The goal of the initial patch, as stated by Jeff, was: "The fedora packaging tools want to *override* $CFLAGS when building sparse" This version doesn't allow to override CFLAGS. See what happens with 'make CFLAGS=-O3' for example. -- Luc -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
