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 <[email protected]>

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 <[email protected]>

Regards,
Carl-Daniel

_______________________________________________
flashrom mailing list
[email protected]
http://www.flashrom.org/mailman/listinfo/flashrom

Reply via email to