From: Richard Levitte <levitte@openssl.org>
Date: Fri, 2 Feb 2018 11:23:09 +0000 (+0100)
Subject: Refactor the ranlib attribute
X-Git-Tag: OpenSSL_1_1_1-pre2~37
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=5f2e18bc3e8a85fe4ea2d0c892b7399c797c7cd0;p=oweals%2Fopenssl.git

Refactor the ranlib attribute

It was inconsistent to see this specific command have
'$(CROSS_COMPILE)' in its value when no other command did.

Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/5247)
---

diff --git a/Configurations/00-base-templates.conf b/Configurations/00-base-templates.conf
index 99c271e5b1..0350997ff7 100644
--- a/Configurations/00-base-templates.conf
+++ b/Configurations/00-base-templates.conf
@@ -50,8 +50,8 @@ my %targets=(
 	arflags		=> "r",
 	cc		=> "cc",
 	hashbangperl	=> "/usr/bin/env perl",
-	ranlib		=> sub { which("$config{cross_compile_prefix}ranlib") ?
-				     "\$(CROSS_COMPILE)ranlib" : "true"; },
+	ranlib		=> sub { which("$config{cross_compile_prefix}ranlib")
+                                     ? "ranlib" : "" },
 	rc		=> "windres",
 
 	#### THESE WILL BE ENABLED IN OpenSSL 1.2
@@ -94,8 +94,8 @@ my %targets=(
                   && defined($disabled{"zlib-dynamic"})
                   ? "-lz" : () },
         hashbangperl    => "/usr/bin/env perl", # Only Unix actually cares
-        ranlib          => sub { which("$config{cross_compile_prefix}ranlib") ?
-                                     "\$(CROSS_COMPILE)ranlib" : "true"; },
+        ranlib          => sub { which("$config{cross_compile_prefix}ranlib")
+                                     ? "ranlib" : "" },
         rc              => "windres",
 
         build_scheme    => [ "unified", "unix" ],
diff --git a/Configurations/unix-Makefile.tmpl b/Configurations/unix-Makefile.tmpl
index 1f214a2747..2a4d2df332 100644
--- a/Configurations/unix-Makefile.tmpl
+++ b/Configurations/unix-Makefile.tmpl
@@ -218,7 +218,7 @@ PERL={- $config{perl} -}
 
 AR=$(CROSS_COMPILE){- $config{ar} -}
 ARFLAGS= {- join(' ', @{$config{arflags}}) -}
-RANLIB= {- $config{ranlib} -}
+RANLIB={- $config{ranlib} ? "\$(CROSS_COMPILE)$config{ranlib}" : "true"; -}
 RC= $(CROSS_COMPILE){- $target{rc} || "windres" -}
 RCFLAGS={- join(' ', @{$config{rcflags}}) -} {- $target{shared_rcflag} -}
 RM= rm -f