Build: change remaining $unified_info{install} checks to use attributes

Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/8063)
This commit is contained in:
Richard Levitte 2019-01-22 15:46:54 +01:00
parent 13234dd310
commit 5cae2d349b
5 changed files with 21 additions and 39 deletions

View File

@ -52,18 +52,6 @@
map { $replace{$_} // $_; } @newlist; map { $replace{$_} // $_; } @newlist;
} }
# is_installed checks if a given file will be installed (i.e. they are
# not defined _NO_INST in build.info)
sub is_installed {
my $product = shift;
if (grep { $product eq $_ }
map { (@{$unified_info{install}->{$_}}) }
keys %{$unified_info{install}}) {
return 1;
}
return 0;
}
# dogenerate is responsible for producing all the recipes that build # dogenerate is responsible for producing all the recipes that build
# generated source files. It recurses in case a dependency is also a # generated source files. It recurses in case a dependency is also a
# generated source file. # generated source file.
@ -132,14 +120,14 @@
$OUT .= $obj2shlib->(lib => $lib, $OUT .= $obj2shlib->(lib => $lib,
attrs => $unified_info{attributes}->{$lib}, attrs => $unified_info{attributes}->{$lib},
objs => $unified_info{shared_sources}->{$lib}, objs => $unified_info{shared_sources}->{$lib},
deps => [ reducedepends(resolvedepends($lib)) ], deps => [ reducedepends(resolvedepends($lib)) ]);
installed => is_installed($lib));
foreach ((@{$unified_info{shared_sources}->{$lib}}, foreach ((@{$unified_info{shared_sources}->{$lib}},
@{$unified_info{sources}->{$lib}})) { @{$unified_info{sources}->{$lib}})) {
# If this is somehow a compiled object, take care of it that way # If this is somehow a compiled object, take care of it that way
# Otherwise, it might simply be generated # Otherwise, it might simply be generated
if (defined $unified_info{sources}->{$_}) { if (defined $unified_info{sources}->{$_}) {
doobj($_, $lib, intent => "shlib", installed => is_installed($lib)); doobj($_, $lib, intent => "shlib",
attrs => $unified_info{attributes}->{$lib});
} else { } else {
dogenerate($_, undef, undef, intent => "lib"); dogenerate($_, undef, undef, intent => "lib");
} }
@ -149,7 +137,8 @@
attrs => $unified_info{attributes}->{$lib}, attrs => $unified_info{attributes}->{$lib},
objs => [ @{$unified_info{sources}->{$lib}} ]); objs => [ @{$unified_info{sources}->{$lib}} ]);
foreach (@{$unified_info{sources}->{$lib}}) { foreach (@{$unified_info{sources}->{$lib}}) {
doobj($_, $lib, intent => "lib", installed => is_installed($lib)); doobj($_, $lib, intent => "lib",
attrs => $unified_info{attributes}->{$lib});
} }
$cache{$lib} = 1; $cache{$lib} = 1;
} }
@ -163,13 +152,13 @@
$OUT .= obj2dso(lib => $lib, $OUT .= obj2dso(lib => $lib,
attrs => $unified_info{attributes}->{$lib}, attrs => $unified_info{attributes}->{$lib},
objs => $unified_info{shared_sources}->{$lib}, objs => $unified_info{shared_sources}->{$lib},
deps => [ resolvedepends($lib) ], deps => [ resolvedepends($lib) ]);
installed => is_installed($lib));
foreach (@{$unified_info{shared_sources}->{$lib}}) { foreach (@{$unified_info{shared_sources}->{$lib}}) {
# If this is somehow a compiled object, take care of it that way # If this is somehow a compiled object, take care of it that way
# Otherwise, it might simply be generated # Otherwise, it might simply be generated
if (defined $unified_info{sources}->{$_}) { if (defined $unified_info{sources}->{$_}) {
doobj($_, $lib, intent => "dso", installed => is_installed($lib)); doobj($_, $lib, intent => "dso",
attrs => $unified_info{attributes}->{$lib});
} else { } else {
dogenerate($_, undef, $lib, intent => "dso"); dogenerate($_, undef, $lib, intent => "dso");
} }
@ -186,10 +175,10 @@
$OUT .= obj2bin(bin => $bin, $OUT .= obj2bin(bin => $bin,
attrs => $unified_info{attributes}->{$bin}, attrs => $unified_info{attributes}->{$bin},
objs => [ @{$unified_info{sources}->{$bin}} ], objs => [ @{$unified_info{sources}->{$bin}} ],
deps => $deps, deps => $deps);
installed => is_installed($bin));
foreach (@{$unified_info{sources}->{$bin}}) { foreach (@{$unified_info{sources}->{$bin}}) {
doobj($_, $bin, intent => "bin", installed => is_installed($bin)); doobj($_, $bin, intent => "bin",
attrs => $unified_info{attributes}->{$bin});
} }
$cache{$bin} = 1; $cache{$bin} = 1;
} }
@ -201,8 +190,7 @@
return "" if $cache{$script}; return "" if $cache{$script};
$OUT .= in2script(script => $script, $OUT .= in2script(script => $script,
attrs => $unified_info{attributes}->{$script}, attrs => $unified_info{attributes}->{$script},
sources => $unified_info{sources}->{$script}, sources => $unified_info{sources}->{$script});
installed => is_installed($script));
$cache{$script} = 1; $cache{$script} = 1;
} }

View File

@ -877,16 +877,16 @@ EOF
} }
my $cflags; my $cflags;
if ($args{installed}) { if ($args{attrs}->{noinst}) {
$cflags = { shlib => '$(LIB_CFLAGS)',
lib => '$(LIB_CFLAGS)',
dso => '$(DSO_CFLAGS)',
bin => '$(BIN_CFLAGS)' } -> {$args{intent}};
} else {
$cflags = { shlib => '$(NO_INST_LIB_CFLAGS)', $cflags = { shlib => '$(NO_INST_LIB_CFLAGS)',
lib => '$(NO_INST_LIB_CFLAGS)', lib => '$(NO_INST_LIB_CFLAGS)',
dso => '$(NO_INST_DSO_CFLAGS)', dso => '$(NO_INST_DSO_CFLAGS)',
bin => '$(NO_INST_BIN_CFLAGS)' } -> {$args{intent}}; bin => '$(NO_INST_BIN_CFLAGS)' } -> {$args{intent}};
} else {
$cflags = { shlib => '$(LIB_CFLAGS)',
lib => '$(LIB_CFLAGS)',
dso => '$(DSO_CFLAGS)',
bin => '$(BIN_CFLAGS)' } -> {$args{intent}};
} }
$cflags .= { shlib => '$(LIB_CPPFLAGS)', $cflags .= { shlib => '$(LIB_CPPFLAGS)',
lib => '$(LIB_CPPFLAGS)', lib => '$(LIB_CPPFLAGS)',

View File

@ -44,9 +44,7 @@ sub staticname {
# Non-installed libraries are *always* static, and their names remain # Non-installed libraries are *always* static, and their names remain
# the same, except for the mandatory extension # the same, except for the mandatory extension
my $in_libname = platform::BASE->staticname($_[1]); my $in_libname = platform::BASE->staticname($_[1]);
return $in_libname return $in_libname if $unified_info{attributes}->{$_[1]}->{noinst};
unless ( grep { platform::BASE->staticname($_) eq $in_libname }
@{$unified_info{install}->{libraries}} );
# We currently return the same name anyway... but we might choose to # We currently return the same name anyway... but we might choose to
# append '_static' or '_a' some time in the future. # append '_static' or '_a' some time in the future.

View File

@ -39,9 +39,7 @@ sub staticname {
# Non-installed libraries are *always* static, and their names remain # Non-installed libraries are *always* static, and their names remain
# the same, except for the mandatory extension # the same, except for the mandatory extension
my $in_libname = platform::BASE->staticname($_[1]); my $in_libname = platform::BASE->staticname($_[1]);
return $in_libname return $in_libname if $unified_info{attributes}->{$_[1]}->{noinst};
unless ( grep { platform::BASE->staticname($_) eq $in_libname }
@{$unified_info{install}->{libraries}} );
return platform::BASE::__concat($_[0]->osslprefix(), return platform::BASE::__concat($_[0]->osslprefix(),
platform::BASE->staticname($_[1]), platform::BASE->staticname($_[1]),

View File

@ -30,9 +30,7 @@ sub staticname {
# Non-installed libraries are *always* static, and their names remain # Non-installed libraries are *always* static, and their names remain
# the same, except for the mandatory extension # the same, except for the mandatory extension
my $in_libname = platform::BASE->staticname($_[1]); my $in_libname = platform::BASE->staticname($_[1]);
return $in_libname return $in_libname if $unified_info{attributes}->{$_[1]}->{noinst};
unless ( grep { platform::BASE->staticname($_) eq $in_libname }
@{$unified_info{install}->{libraries}} );
# To make sure not to clash with an import library, we make the static # To make sure not to clash with an import library, we make the static
# variant of our installed libraries get '_static' added to their names. # variant of our installed libraries get '_static' added to their names.