On Apr 20, 2015, at 3:41 PM, Mark Cave-Ayland wrote:
On 20/04/15 14:40, Programmingkid wrote:
This patch gives the fcode-utils' toke, detok, and romheaders commands an install target. This makes "make install" work.
Changes since v1:
- Added DESTDIR variable
Signed-off-by: John Arbuckle <programmingkidx@gmail.com mailto:programmingkidx@gmail.com>
Index: detok/Makefile
--- detok/Makefile(revision 1334) +++ detok/Makefile(working copy) @@ -23,7 +23,7 @@ #
PROGRAM = detok
+DESTDIR=/usr/local/bin CC ?= gcc STRIP= strip INCLUDES = -I../shared @@ -63,3 +63,5 @@ .c.o: $(CC) -c $(CFLAGS) $(INCLUDES) $< -o $@
+install: +cp $(PROGRAM) $(DESTDIR)/$(PROGRAM) Index: romheaders/Makefile =================================================================== --- romheaders/Makefile(revision 1334) +++ romheaders/Makefile(working copy) @@ -22,6 +22,8 @@ # Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA, 02110-1301 USA #
+PROGRAM=romheaders +DESTDIR=/usr/local/bin CC?= gcc STRIP = strip CFLAGS = -O2 -Wall -Wextra @@ -39,4 +41,7 @@
clean: rm -f *~ -rm -f romheaders +rm -f $(PROGRAM)
+install: +cp $(PROGRAM) $(DESTDIR)/$(PROGRAM) Index: toke/Makefile =================================================================== --- toke/Makefile(revision 1334) +++ toke/Makefile(working copy) @@ -23,7 +23,7 @@ #
PROGRAM = toke
+DESTDIR=/usr/local/bin CC ?= gcc STRIP= strip INCLUDES = -I../shared @@ -73,3 +73,5 @@ .c.o: $(CC) -c $(CFLAGS) $(INCLUDES) $< -o $@
+install: +cp $(PROGRAM) $(DESTDIR)/$(PROGRAM)
This looks much better to me, although I can't really test as it appears in my mail reader as HTML :/
ATB,
Mark.
What mail reader are you using? Are you able to read the raw source of the email? I will just send it again as an attachment with this email.