[hackers] [surf] Arrange Makefile || Quentin Rameau

From: <git_AT_suckless.org>
Date: Sun, 30 Apr 2017 23:25:10 +0200 (CEST)

commit c870098b82f1dfc3cd59cd8c90cea580bd01cb47
Author: Quentin Rameau <quinq_AT_fifth.space>
AuthorDate: Sat Apr 29 15:21:49 2017 +0200
Commit: Quentin Rameau <quinq_AT_fifth.space>
CommitDate: Sat Apr 29 16:34:47 2017 +0200

    Arrange Makefile
    
    Let the user specify his own C and LD FLAGS.
    Remove unuseful and non portable CFLAGS like pendantic and Wall.

diff --git a/Makefile b/Makefile
index a9d4d1d..d32d330 100644
--- a/Makefile
+++ b/Makefile
_AT_@ -1,61 +1,66 @@
 # surf - simple browser
 # See LICENSE file for copyright and license details.
+.POSIX:
 
 include config.mk
 
 SRC = surf.c
-OBJ = ${SRC:.c=.o}
+OBJ = $(SRC:.c=.o)
 
 all: options surf
 
 options:
         _AT_echo surf build options:
- _AT_echo "CFLAGS = ${CFLAGS}"
- _AT_echo "LDFLAGS = ${LDFLAGS}"
- _AT_echo "CC = ${CC}"
+ _AT_echo "CFLAGS = $(SURF_CFLAGS)"
+ _AT_echo "LDFLAGS = $(SURF_LDFLAGS)"
+ _AT_echo "CC = $(CC)"
 
 .c.o:
- _AT_echo CC $<
- _AT_${CC} -c ${CFLAGS} $<
+ _AT_echo CC -c $<
+ _AT_$(CC) $(SURF_CFLAGS) -c $<
 
-${OBJ}: config.h config.mk
+$(OBJ): config.h config.mk
 
 config.h:
         _AT_echo creating $@ from config.def.h
         _AT_cp config.def.h $@
 
-surf: ${OBJ}
+surf: $(OBJ)
         _AT_echo CC -o $@
- _AT_${CC} -o $@ surf.o ${LDFLAGS}
+ _AT_$(CC) $(SURF_CFLAGS) -o $@ $(OBJ) $(SURF_LDFLAGS)
 
 clean:
         _AT_echo cleaning
- _AT_rm -f surf ${OBJ} surf-${VERSION}.tar.gz
+ _AT_rm -f surf $(OBJ)
 
-dist: clean
+distclean: clean
+ _AT_echo cleaning dist
+ _AT_rm -f config.h surf-$(VERSION).tar.gz
+
+dist: distclean
         _AT_echo creating dist tarball
- _AT_mkdir -p surf-${VERSION}
+ _AT_mkdir -p surf-$(VERSION)
         _AT_cp -R LICENSE Makefile config.mk config.def.h README \
- surf-open.sh arg.h TODO.md surf.png \
- surf.1 ${SRC} surf-${VERSION}
- _AT_tar -cf surf-${VERSION}.tar surf-${VERSION}
- _AT_gzip surf-${VERSION}.tar
- _AT_rm -rf surf-${VERSION}
+ surf-open.sh arg.h TODO.md surf.png \
+ surf.1 $(SRC) surf-$(VERSION)
+ _AT_tar -cf surf-$(VERSION).tar surf-$(VERSION)
+ _AT_gzip surf-$(VERSION).tar
+ _AT_rm -rf surf-$(VERSION)
 
 install: all
- _AT_echo installing executable file to ${DESTDIR}${PREFIX}/bin
- _AT_mkdir -p ${DESTDIR}${PREFIX}/bin
- _AT_cp -f surf ${DESTDIR}${PREFIX}/bin
- _AT_chmod 755 ${DESTDIR}${PREFIX}/bin/surf
- _AT_echo installing manual page to ${DESTDIR}${MANPREFIX}/man1
- _AT_mkdir -p ${DESTDIR}${MANPREFIX}/man1
- _AT_sed "s/VERSION/${VERSION}/g" < surf.1 > ${DESTDIR}${MANPREFIX}/man1/surf.1
- _AT_chmod 644 ${DESTDIR}${MANPREFIX}/man1/surf.1
+ _AT_echo installing executable file to $(DESTDIR)$(PREFIX)/bin
+ _AT_mkdir -p $(DESTDIR)$(PREFIX)/bin
+ _AT_cp -f surf $(DESTDIR)$(PREFIX)/bin
+ _AT_chmod 755 $(DESTDIR)$(PREFIX)/bin/surf
+ _AT_echo installing manual page to $(DESTDIR)$(MANPREFIX)/man1
+ _AT_mkdir -p $(DESTDIR)$(MANPREFIX)/man1
+ _AT_sed "s/VERSION/$(VERSION)/g" < surf.1 > $(DESTDIR)$(MANPREFIX)/man1/surf.1
+ _AT_chmod 644 $(DESTDIR)$(MANPREFIX)/man1/surf.1
 
 uninstall:
- _AT_echo removing executable file from ${DESTDIR}${PREFIX}/bin
- _AT_rm -f ${DESTDIR}${PREFIX}/bin/surf
- _AT_echo removing manual page from ${DESTDIR}${MANPREFIX}/man1
- _AT_rm -f ${DESTDIR}${MANPREFIX}/man1/surf.1
+ _AT_echo removing executable file from $(DESTDIR)$(PREFIX)/bin
+ _AT_rm -f $(DESTDIR)$(PREFIX)/bin/surf
+ _AT_echo removing manual page from $(DESTDIR)$(MANPREFIX)/man1
+ _AT_rm -f $(DESTDIR)$(MANPREFIX)/man1/surf.1
 
 .PHONY: all options clean dist install uninstall
diff --git a/config.mk b/config.mk
index df6e812..5bec3e5 100644
--- a/config.mk
+++ b/config.mk
_AT_@ -5,8 +5,8 @@ VERSION = 2.0
 
 # paths
 PREFIX = /usr/local
-MANPREFIX = ${PREFIX}/share/man
-LIBPREFIX = ${PREFIX}/lib/surf
+MANPREFIX = $(PREFIX)/share/man
+LIBPREFIX = $(PREFIX)/lib/surf
 
 X11INC = /usr/X11R6/include
 X11LIB = /usr/X11R6/lib
_AT_@ -15,17 +15,17 @@ GTKINC = `pkg-config --cflags gtk+-3.0 webkit2gtk-4.0`
 GTKLIB = `pkg-config --libs gtk+-3.0 webkit2gtk-4.0`
 
 # includes and libs
-INCS = -I. -I/usr/include -I${X11INC} ${GTKINC}
-LIBS = -L/usr/lib -lc -L${X11LIB} -lX11 ${GTKLIB} -lgthread-2.0
+INCS = -I$(X11INC) $(GTKINC)
+LIBS = -L$(X11LIB) -lX11 $(GTKLIB) -lgthread-2.0
 
 # flags
 CPPFLAGS = -DVERSION=\"${VERSION}\" -DWEBEXTDIR=\"${LIBPREFIX}\" -D_DEFAULT_SOURCE
-CFLAGS = -std=c99 -pedantic -Wall -Os ${INCS} ${CPPFLAGS}
-LDFLAGS = -s ${LIBS}
+SURF_CFLAGS = $(INCS) $(CPPFLAGS) $(CFLAGS)
+SURF_LDFLAGS = $(LIBS) $(LDFLAGS)
 
 # Solaris
-#CFLAGS = -fast ${INCS} -DVERSION=\"${VERSION}\"
-#LDFLAGS = ${LIBS}
+#CFLAGS = -fast $(INCS) -DVERSION=\"$(VERSION)\"
+#LDFLAGS = $(LIBS)
 
 # compiler and linker
-CC = cc
+#CC = c99
Received on Sun Apr 30 2017 - 23:25:10 CEST