[build] Avoid hard-coding the path to perl
authorMichael Brown <mcb30@ipxe.org>
Thu, 24 Jun 2010 00:23:00 +0000 (01:23 +0100)
committerStefan Hajnoczi <stefanha@gmail.com>
Wed, 7 Jul 2010 19:22:19 +0000 (20:22 +0100)
The path "/usr/bin/perl" has been hard-coded since Etherboot 5.1, for
no discernible reason.  Use just "perl" instead to fix the
inconsistency and allow building on systems with Perl installed
outside of /usr/bin.

This commit also includes a later fix that removes a dependency on
"perl" which broke builds from fully clean trees.

Reported-by: Gabor Z. Papp <gzp@papp.hu>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
Signed-off-by: Stefan Hajnoczi <stefanha@gmail.com>
src/Makefile
src/Makefile.housekeeping
src/arch/i386/Makefile.pcbios

index 2acde3c..bbd169b 100644 (file)
@@ -20,7 +20,7 @@ MKDIR         := mkdir
 CP             := cp
 ECHO           := echo
 PRINTF         := printf
-PERL           := /usr/bin/perl
+PERL           := perl
 CC             := $(CROSS_COMPILE)gcc
 CPP            := $(CC) -E
 AS             := $(CROSS_COMPILE)as
@@ -31,12 +31,12 @@ RANLIB              := $(CROSS_COMPILE)ranlib
 OBJCOPY                := $(CROSS_COMPILE)objcopy
 NM             := $(CROSS_COMPILE)nm
 OBJDUMP                := $(CROSS_COMPILE)objdump
-PARSEROM       := $(PERL) ./util/parserom.pl
-FIXROM         := $(PERL) ./util/fixrom.pl
-SYMCHECK       := $(PERL) ./util/symcheck.pl
-SORTOBJDUMP    := $(PERL) ./util/sortobjdump.pl
-PADIMG         := $(PERL) ./util/padimg.pl
-LICENCE                := $(PERL) ./util/licence.pl
+PARSEROM       := ./util/parserom.pl
+FIXROM         := ./util/fixrom.pl
+SYMCHECK       := ./util/symcheck.pl
+SORTOBJDUMP    := ./util/sortobjdump.pl
+PADIMG         := ./util/padimg.pl
+LICENCE                := ./util/licence.pl
 NRV2B          := ./util/nrv2b
 ZBIN           := ./util/zbin
 ELF2EFI32      := ./util/elf2efi32
index b6b747e..79a3332 100644 (file)
@@ -486,7 +486,7 @@ define src_template
                 '\n$(2) : $$($(4)_DEPS)\n' \
                 '\nTAGS : $$($(4)_DEPS)\n' \
                >> $(2)
-       @$(PARSEROM) $(1) >> $(2)
+       @$(PERL) $(PARSEROM) $(1) >> $(2)
 
 endef
 
@@ -701,7 +701,7 @@ $(BIN)/%.tmp : $(BLIB) $(MAKEDEPS) $(LDSCRIPT)
        $(QM)$(ECHO) "  [LD] $@"
        $(Q)$(LD) $(LDFLAGS) -T $(LDSCRIPT) $(TGT_LD_FLAGS) $(BLIB) -o $@ \
                --defsym _build_id=`$(BUILD_ID_CMD)` -Map $(BIN)/$*.tmp.map
-       $(Q)$(OBJDUMP) -ht $@ | $(SORTOBJDUMP) >> $(BIN)/$*.tmp.map
+       $(Q)$(OBJDUMP) -ht $@ | $(PERL) $(SORTOBJDUMP) >> $(BIN)/$*.tmp.map
 
 # Keep intermediate object file (useful for debugging)
 .PRECIOUS : $(BIN)/%.tmp
@@ -758,7 +758,7 @@ $(BIN)/%.licence : $(BIN)/%.tmp
                echo "files are missing a licence declaration:" ;\
                echo $(call unlicensed_deps_list,$<);\
                exit 1,\
-               $(LICENCE) $(call licence_list,$<))
+               $(PERL) $(LICENCE) $(call licence_list,$<))
 
 # Extract compression information from intermediate object file
 #
@@ -987,7 +987,7 @@ $(SYMTAB) : $(BLIB)
 CLEANUP        += $(BIN)/symtab
 
 symcheck : $(SYMTAB)
-       $(SYMCHECK) $<
+       $(PERL) $(SYMCHECK) $<
 
 endif # defined(BIN)
 
index 1c1b5a8..1a9cc13 100644 (file)
@@ -23,14 +23,14 @@ MEDIA               += raw
 
 # Padding rules
 #
-PAD_rom                = $(PADIMG) --blksize=512 --byte=0xff $@
+PAD_rom                = $(PERL) $(PADIMG) --blksize=512 --byte=0xff $@
 PAD_mrom       = $(PAD_rom)
-PAD_dsk                = $(PADIMG) --blksize=512 $@
-PAD_hd         = $(PADIMG) --blksize=32768 $@
+PAD_dsk                = $(PERL) $(PADIMG) --blksize=512 $@
+PAD_hd         = $(PERL) $(PADIMG) --blksize=32768 $@
 
 # Finalisation rules
 #
-FINALISE_rom   = $(FIXROM) $@
+FINALISE_rom   = $(PERL) $(FIXROM) $@
 FINALISE_mrom  = $(FINALISE_rom)
 
 # rule to make a non-emulation ISO boot image