[flashrom] [PATCH] Makefile: fix overriding of make command line arguments.

Stefan Tauner stefan.tauner at alumni.tuwien.ac.at
Fri Feb 19 09:17:36 CET 2016


On Fri, 19 Feb 2016 01:02:23 +0100
Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net> wrote:

> On 18.02.2016 22:19, Stefan Tauner wrote:
> > Command line arguments given when executing make should not stop us
> > from setting commonly user-configurable variables like CPPFLAGS or
> > LDFLAGS.
> > Without this patch 'make CPPFLAGS="-Ifail"' would fail to find
> > libusb1 headers and abort.
> > Add override statements where we append mandatory options to
> > these variables to make (Debian) packagers more happy.
> >
> > Also, r1920 introduced some explicit calls to pkg-config instead of
> > $(PKG_CONFIG). This patch corrects that as well.
> >
> > Signed-off-by: Stefan Tauner <stefan.tauner at alumni.tuwien.ac.at>
> 
> You missed one instance:
> CFLAGS += -Wno-format
> 
> Even if compiling for DOS is probably not something where default
> packaging is involved, it would make the whole *FLAGS thing more consistent.
> Other than that, it's
> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

Thanks, r1931.
-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner




More information about the flashrom mailing list