Patchwork superiotool: add missing -lz and don't clubber CFLAGS

login
register
about
Submitter Anders Juel Jensen
Date 2010-08-22 02:36:01
Message ID <201008220436.02520.andersjjensen@gmail.com>
Download mbox | patch
Permalink /patch/1784/
State Accepted
Commit r5725
Headers show

Comments

Anders Juel Jensen - 2010-08-22 02:36:01
The LDFLAGS = -lz is needed to compile on slackware.
Clubbering CFLAGS is never a good idea.

Signed-off-by: Anders Juel Jensen <andersjjensen@gmail.com>

Patch

Index: util/superiotool/Makefile
===================================================================
--- util/superiotool/Makefile	(revision 5724)
+++ util/superiotool/Makefile	(working copy)
@@ -29,8 +29,9 @@ 
 SVNDEF := -D'SUPERIOTOOL_VERSION="$(shell svnversion -cn . \
           | sed -e "s/.*://" -e "s/\([0-9]*\).*/\1/")"'
 
-CFLAGS = -O2 -Wall -Werror -Wstrict-prototypes -Wundef -Wstrict-aliasing \
+CFLAGS += -O2 -Wall -Werror -Wstrict-prototypes -Wundef -Wstrict-aliasing \
          -Werror-implicit-function-declaration -ansi -pedantic $(SVNDEF)
+LDFLAGS += -lz
 
 OBJS = superiotool.o ali.o fintek.o ite.o nsc.o nuvoton.o smsc.o winbond.o