bookmenu icon indicating copy to clipboard operation
bookmenu copied to clipboard

Published 20 hours ago • verified publisher icon BrodieRobertson

Add Makefile

Open crian opened this issue 4 years ago • 2 comments

Use a Makefile. Remove install script.

crian avatar Apr 08 '20 18:04 crian

The ?= syntax is non-POSIX, why not just PREFIX=/usr/local? That way you still could change the prefix on the command line via make PREFIX=/usr. Also, the install and uninstall targets should be phony targets and I’d put a POSIX: at the beginning, which guarantees POSIX Makefiles to work in POSIX compliant make(1) implementations. make(1) is designed to print out all the commands, so why mute them?

Humm42 avatar Apr 11 '20 08:04 Humm42

ok! i'm not using PREFIX=/usr/local because /usr/... is hardcoded in the script.

crian avatar Apr 11 '20 12:04 crian