[coreboot] [flashrom] r508 - trunk

svn at coreboot.org svn at coreboot.org
Thu May 14 16:17:07 CEST 2009


Author: hailfinger
Date: 2009-05-14 16:17:07 +0200 (Thu, 14 May 2009)
New Revision: 508

Modified:
   trunk/Makefile
Log:
Improve makefile structure a bit.

Signed-off-by: Christian Ruppert <spooky85 at gmail.com>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>


Modified: trunk/Makefile
===================================================================
--- trunk/Makefile	2009-05-14 12:59:36 UTC (rev 507)
+++ trunk/Makefile	2009-05-14 14:17:07 UTC (rev 508)
@@ -8,12 +8,14 @@
 
 CC     ?= gcc
 STRIP	= strip
-INSTALL = /usr/bin/install
-PREFIX  = /usr/local
-#CFLAGS  = -O2 -g -Wall -Werror
-CFLAGS  = -Os -Wall -Werror
-LDFLAGS = 
+INSTALL = install
+PREFIX  ?= /usr/local
+CFLAGS  ?= -Os -Wall -Werror
 
+prefix = $(DESTDIR)$(PREFIX)
+man8dir = $(prefix)/share/man/man8
+sbindir = $(prefix)/sbin
+
 OS_ARCH	= $(shell uname)
 ifneq ($(OS_ARCH), SunOS)
 STRIP_ARGS = -s
@@ -27,7 +29,7 @@
 LDFLAGS += -L/usr/local/lib
 endif
 
-LDFLAGS += -lpci -lz
+LIBS += -lpci -lz
 
 OBJS = chipset_enable.o board_enable.o udelay.o jedec.o stm50flw0x0x.o \
 	sst28sf040.o am29f040b.o mx29f002.o m29f400bt.o \
@@ -45,8 +47,7 @@
           | sed -e "s/.*://" -e "s/\([0-9]*\).*/\1/")"'
 
 $(PROGRAM): $(OBJS)
-	$(CC) -o $(PROGRAM) $(OBJS) $(LDFLAGS)
-	$(STRIP) $(STRIP_ARGS) $(PROGRAM)
+	$(CC) $(LDFLAGS) -o $(PROGRAM) $(OBJS) $(LIBS)
 
 flashrom.o: flashrom.c
 	$(CC) -c $(CFLAGS) $(SVNDEF) $(CPPFLAGS) $< -o $@
@@ -58,15 +59,18 @@
 	rm -f .dependencies
 
 dep:
-	@$(CC) $(SVNDEF) -MM *.c > .dependencies
+	@$(CC) $(CPPFLAGS) $(SVNDEF) -MM *.c > .dependencies
 
+strip: $(PROGRAM)
+	$(STRIP) $(STRIP_ARGS) $(PROGRAM)
+
 pciutils:
 	@echo; printf "Checking for pciutils and zlib... "
 	@$(shell ( echo "#include <pci/pci.h>";		   \
 		   echo "struct pci_access *pacc;";	   \
 		   echo "int main(int argc, char **argv)"; \
 		   echo "{ pacc = pci_alloc(); return 0; }"; ) > .test.c )
-	@$(CC) $(CFLAGS) .test.c -o .test $(LDFLAGS) >/dev/null 2>&1 &&	\
+	@$(CC) $(CFLAGS) $(LDFLAGS) .test.c -o .test $(LIBS) >/dev/null 2>&1 &&	\
 		echo "found." || ( echo "not found."; echo;		\
 		echo "Please install pciutils-devel and zlib-devel.";	\
 		echo "See README for more information."; echo;		\
@@ -74,9 +78,9 @@
 	@rm -f .test.c .test
 
 install: $(PROGRAM)
-	$(INSTALL) $(PROGRAM) $(PREFIX)/sbin
-	mkdir -p $(PREFIX)/share/man/man8
-	$(INSTALL) $(PROGRAM).8 $(PREFIX)/share/man/man8
+	mkdir -p $(sbindir) $(man8dir)
+	$(INSTALL) -m 0755 $(PROGRAM) $(sbindir)
+	$(INSTALL) -m 0644 $(PROGRAM).8 $(man8dir)
 
 .PHONY: all clean distclean dep pciutils
 





More information about the coreboot mailing list