[hackers] [st] Do not obfuscate what make is doing. || Quentin Rameau

From: <git_AT_suckless.org>
Date: Wed, 12 Jul 2017 15:52:16 +0200 (CEST)

commit fabd4602b3223666165c76c397644a081b9a97e5
Author: Quentin Rameau <quinq_AT_fifth.space>
AuthorDate: Wed Jul 12 00:24:51 2017 +0200
Commit: Quentin Rameau <quinq_AT_fifth.space>
CommitDate: Wed Jul 12 00:28:10 2017 +0200

    Do not obfuscate what make is doing.
    
    Change some styling too while we're at it.

diff --git a/Makefile b/Makefile
index d8595fe..f5b84a0 100644
--- a/Makefile
+++ b/Makefile
_AT_@ -1,63 +1,55 @@
 # st - simple terminal
 # See LICENSE file for copyright and license details.
+.POSIX:
 
 include config.mk
 
 SRC = st.c x.c
-OBJ = ${SRC:.c=.o}
+OBJ = $(SRC:.c=.o)
 
 all: options st
 
 options:
         _AT_echo st build options:
- _AT_echo "CFLAGS = ${CFLAGS}"
- _AT_echo "LDFLAGS = ${LDFLAGS}"
- _AT_echo "CC = ${CC}"
+ _AT_echo "CFLAGS = $(CFLAGS)"
+ _AT_echo "LDFLAGS = $(LDFLAGS)"
+ _AT_echo "CC = $(CC)"
 
 config.h:
         cp config.def.h config.h
 
 .c.o:
- _AT_echo CC $<
- _AT_${CC} -c ${CFLAGS} $<
+ $(CC) $(CFLAGS) -c $<
 
 st.o: config.h st.h win.h
 x.o: arg.h st.h win.h
 
-${OBJ}: config.h config.mk
+$(OBJ): config.h config.mk
 
-st: ${OBJ}
- _AT_echo CC -o $@
- _AT_${CC} -o $@ ${OBJ} ${LDFLAGS}
+st: $(OBJ)
+ $(CC) $(LDFLAGS) -o $_AT_ $(OBJ)
 
 clean:
- _AT_echo cleaning
- _AT_rm -f st ${OBJ} st-${VERSION}.tar.gz
+ rm -f st $(OBJ) st-$(VERSION).tar.gz
 
 dist: clean
- _AT_echo creating dist tarball
- _AT_mkdir -p st-${VERSION}
- _AT_cp -R LICENSE Makefile README config.mk config.def.h st.info st.1 arg.h ${SRC} st-${VERSION}
- _AT_tar -cf st-${VERSION}.tar st-${VERSION}
- _AT_gzip st-${VERSION}.tar
- _AT_rm -rf st-${VERSION}
-
-install: all
- _AT_echo installing executable file to ${DESTDIR}${PREFIX}/bin
- _AT_mkdir -p ${DESTDIR}${PREFIX}/bin
- _AT_cp -f st ${DESTDIR}${PREFIX}/bin
- _AT_chmod 755 ${DESTDIR}${PREFIX}/bin/st
- _AT_echo installing manual page to ${DESTDIR}${MANPREFIX}/man1
- _AT_mkdir -p ${DESTDIR}${MANPREFIX}/man1
- _AT_sed "s/VERSION/${VERSION}/g" < st.1 > ${DESTDIR}${MANPREFIX}/man1/st.1
- _AT_chmod 644 ${DESTDIR}${MANPREFIX}/man1/st.1
+ mkdir -p st-$(VERSION)
+ cp -R LICENSE Makefile README config.mk config.def.h st.info st.1 arg.h $(SRC) st-$(VERSION)
+ tar -cf - st-$(VERSION) | gzip > st-$(VERSION).tar.gz
+ rm -rf st-$(VERSION)
+
+install: st
+ mkdir -p $(DESTDIR)$(PREFIX)/bin
+ cp -f st $(DESTDIR)$(PREFIX)/bin
+ chmod 755 $(DESTDIR)$(PREFIX)/bin/st
+ mkdir -p $(DESTDIR)$(MANPREFIX)/man1
+ sed "s/VERSION/$(VERSION)/g" < st.1 > $(DESTDIR)$(MANPREFIX)/man1/st.1
+ chmod 644 $(DESTDIR)$(MANPREFIX)/man1/st.1
+ tic -sx st.info
         _AT_echo Please see the README file regarding the terminfo entry of st.
- _AT_tic -sx st.info
 
 uninstall:
- _AT_echo removing executable file from ${DESTDIR}${PREFIX}/bin
- _AT_rm -f ${DESTDIR}${PREFIX}/bin/st
- _AT_echo removing manual page from ${DESTDIR}${MANPREFIX}/man1
- _AT_rm -f ${DESTDIR}${MANPREFIX}/man1/st.1
+ rm -f $(DESTDIR)$(PREFIX)/bin/st
+ rm -f $(DESTDIR)$(MANPREFIX)/man1/st.1
 
 .PHONY: all options clean dist install uninstall
diff --git a/config.mk b/config.mk
index c84c5ee..527a64d 100644
--- a/config.mk
+++ b/config.mk
_AT_@ -5,24 +5,24 @@ VERSION = 0.7
 
 # paths
 PREFIX = /usr/local
-MANPREFIX = ${PREFIX}/share/man
+MANPREFIX = $(PREFIX)/share/man
 
 X11INC = /usr/X11R6/include
 X11LIB = /usr/X11R6/lib
 
 # includes and libs
-INCS = -I. -I/usr/include -I${X11INC} \
+INCS = -I$(X11INC) \
        `pkg-config --cflags fontconfig` \
        `pkg-config --cflags freetype2`
-LIBS = -L/usr/lib -lc -L${X11LIB} -lm -lrt -lX11 -lutil -lXft \
- `pkg-config --libs fontconfig` \
+LIBS = -L$(X11LIB) -lm -lrt -lX11 -lutil -lXft \
+ `pkg-config --libs fontconfig` \
        `pkg-config --libs freetype2`
 
 # flags
-CPPFLAGS = -DVERSION=\"${VERSION}\" -D_XOPEN_SOURCE=600
-CFLAGS += -g -std=c99 -pedantic -Wall -Wvariadic-macros -Os ${INCS} ${CPPFLAGS}
-LDFLAGS += -g ${LIBS}
+CPPFLAGS = -DVERSION=\"$(VERSION)\" -D_XOPEN_SOURCE=600
+CFLAGS = -g -std=c99 -pedantic -Wall -Wvariadic-macros -Os $(INCS) $(CPPFLAGS)
+LDFLAGS = -g $(LIBS)
 
 # compiler and linker
-# CC = cc
+# CC = c99
 
Received on Wed Jul 12 2017 - 15:52:16 CEST