Skip to content

Commit a764175

Browse files
nwc10Nicholas Clarck
authored and
Nicholas Clarck
committed
Two minor Makefile.SH cleanups
Use "$osname" instead of (bare) $osname in all case statements. This was the only one that was different. Delete PATH_SEP, which was never used. This was added in Nov 1995 as part of commit 4633a7c: 5.002 beta 1 with various other macros, and the comment "These variables will be used in a future version to make the make file more portable to non-unix systems." The other macros have been used, but not PATH_SEP. Curiously it also gets a different value if one (re)generates Makefile by running ./Makefile.SH, compared with a Makefile generated by Configure, because the variable that it is set from - $p_ - is not in config.sh
1 parent c693b67 commit a764175

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

Makefile.SH

+1-2
Original file line numberDiff line numberDiff line change
@@ -335,7 +335,6 @@ HOST_EXE_EXT = $_exe
335335
EXE_EXT = $_exe
336336
LIB_EXT = $_a
337337
OBJ_EXT = $_o
338-
PATH_SEP = $p_
339338
340339
# Macros to invoke a copy of miniperl during the build. Targets which
341340
# are built using these macros should depend on \$(MINIPERL_EXE)
@@ -1024,7 +1023,7 @@ $(PERL_EXE): $& $(perlmain_dep) $(LIBPERL) $(static_ext) ext.libs $(PERLEXPORT)
10241023
-@rm -f miniperl.xok
10251024
!NO!SUBS!
10261025

1027-
case $osname in
1026+
case "$osname" in
10281027
# In AmigaOS the Perl executable needs to be linked with -ldl,
10291028
# but none of the other executables should be.
10301029
amigaos) $spitshell >>$Makefile <<'!NO!SUBS!'

0 commit comments

Comments
 (0)