In perl.git, the branch smoke-me/no-LDLIBPTH-for-CC has been updated

<http://perl5.git.perl.org/perl.git/commitdiff/d18d9e4686db9938e6d30c8b7c4186952fb5ed24?hp=1927e203ff157957a4c28ff5a935927a92e8b48d>

- Log -----------------------------------------------------------------
commit d18d9e4686db9938e6d30c8b7c4186952fb5ed24
Author: Nicholas Clark <[email protected]>
Date:   Fri Apr 27 17:56:34 2012 +0200

    No need to prefix the $(CC) invocation that links miniperl with $(LDLIBPTH).
    
    $(LDLIBPTH) is used to prefix the appropriate LD_LIBRARY_PATH=... before
    commands in the Makefile when perl is build with a shared perl library so
    that the uninstalled ./perl will be able to find it.
    
    Commit c4f23d77f4b3486a (in May 1998) added the code for LDLIBPTH, but
    also added it to the default command line used to invoke $(CC) to link
    miniperl. This command doesn't need to run ./perl, hence the use of
    $(LDLIBPTH) is superfluous here. Removing it makes the code simpler.
-----------------------------------------------------------------------

Summary of changes:
 Makefile.SH | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile.SH b/Makefile.SH
index ef30479..c018872 100755
--- a/Makefile.SH
+++ b/Makefile.SH
@@ -900,7 +900,7 @@ lib/buildcustomize.pl: $& $(mini_obj) 
write_buildcustomize.pl
                $spitshell >>$Makefile <<'!NO!SUBS!'
 lib/buildcustomize.pl: $& $(mini_obj) write_buildcustomize.pl
        -@rm -f miniperl.xok
-       $(LDLIBPTH) $(CC) $(CLDFLAGS) -o $(MINIPERL_EXE) \
+       $(CC) $(CLDFLAGS) -o $(MINIPERL_EXE) \
            $(mini_obj) $(libs)
        $(LDLIBPTH) $(RUN) ./miniperl$(HOST_EXE_EXT) -w -Ilib 
-Idist/Exporter/lib -MExporter -e '<?>' || sh -c 'echo >&2 Failed to build 
miniperl.  Please run make minitest; exit 1'
        $(MINIPERL) -f write_buildcustomize.pl

--
Perl5 Master Repository

Reply via email to