The branch master has been updated
via 23be743286c0f0a160de33365ef34af39427eac9 (commit)
via 27c4490521261f7f53f494140d8a2b2a239306ef (commit)
via bbb9413b7e530aa25779b33b5ca7a5fd610e66f1 (commit)
from 7731e619fba2f9ea1e888bf906289be37c52e6ac (commit)
- Log -----------------------------------------------------------------
commit 23be743286c0f0a160de33365ef34af39427eac9
Author: Richard Levitte <levi...@openssl.org>
Date: Wed Mar 14 13:24:12 2018 +0100
util/postprocess-makedepend.pl: make an effort to collect dependencies
Instead of just working line by line, we collect all dependencies for
every target and print everything out at the end, with each target
getting a potentially long list of dependencies.
Reviewed-by: Andy Polyakov <ap...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/5591)
commit 27c4490521261f7f53f494140d8a2b2a239306ef
Author: Richard Levitte <levi...@openssl.org>
Date: Wed Mar 14 10:37:26 2018 +0100
util/postprocess-makedepend.pl: For VC, don't include system headers
All dependencies that VC gives us are absolute paths, so we need to
check if some of them are within our source or build tree. We do that
by comparing the start of each dependency with the absolute versions
of our source and build directories.
Reviewed-by: Andy Polyakov <ap...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/5591)
commit bbb9413b7e530aa25779b33b5ca7a5fd610e66f1
Author: Richard Levitte <levi...@openssl.org>
Date: Mon Mar 12 09:18:44 2018 +0100
Streamline dependency generation
It seems that only gcc -MMD produces dependency files that are "sane"
for our needs. For all other methods, some post processing is needed:
- 'makedepend' (Unix) insists that object files are located in the
same spot as the source file.
- 'cl /Zs /showIncludes' (Visual C) has "Note: including file: " where
we'd like to see the object.
- 'CC/DECC' (VMS) insists that the object file is located in the
current directory, i.e. it strips away all directory information.
So far, we've managed this (except for the VMS case) with individual
uncommented perl command lines directly in the build file template.
We're now collecting these diverse hacks into one perl script that
takes an argument to tell what kind of input to expect and that
massages whatever it gets on STDIN and outputs the result on STDOUT.
Reviewed-by: Andy Polyakov <ap...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/5591)
-----------------------------------------------------------------------
Summary of changes:
Configurations/descrip.mms.tmpl | 9 +-
Configurations/unix-Makefile.tmpl | 11 +--
Configurations/windows-makefile.tmpl | 10 +--
util/postprocess-makedepend.pl | 160 +++++++++++++++++++++++++++++++++++
4 files changed, 172 insertions(+), 18 deletions(-)
create mode 100644 util/postprocess-makedepend.pl
diff --git a/Configurations/descrip.mms.tmpl b/Configurations/descrip.mms.tmpl
index 7e7bf71..cf448aa 100644
--- a/Configurations/descrip.mms.tmpl
+++ b/Configurations/descrip.mms.tmpl
@@ -3,6 +3,7 @@
## {- join("\n## ", @autowarntext) -}
{-
use File::Spec::Functions qw/:DEFAULT abs2rel rel2abs/;
+ use File::Basename;
# Our prefix, claimed when speaking with the VSI folks Tuesday
# January 26th 2016
@@ -878,6 +879,9 @@ EOF
my $incs_off = join("\n\t\@ ", @{$incs_cmds[1]}) || '!';
my $depbuild = $disabled{makedepend} ? ""
: " /MMS=(FILE=${objd}${objn}.tmp-D,TARGET=$obj.OBJ)";
+ my $postprocess_makedepend =
+ sourcefile("util", "postprocess-makedepend.pl");
+ my $objdir = dirname($obj);
return <<"EOF"
$obj.OBJ : $deps
@@ -891,9 +895,8 @@ $obj.OBJ : $deps
- PURGE $obj.OBJ
EOF
. ($disabled{makedepend} ? "" : <<"EOF"
- \@ PIPE ( \$(PERL) -e "use File::Compare qw/compare_text/; my \$x =
compare_text(""$obj.D"",""$obj.tmp-D""); exit(0x10000000 + (\$x == 0));" || -
- RENAME $obj.tmp-D $obj.d )
- \@ IF F\$SEARCH("$obj.tmp-D") .NES. "" THEN DELETE $obj.tmp-D;*
+ \$(PERL) $postprocess_makedepend "VMS C" $objdir < $obj.tmp-D > $obj.d
+ - DELETE $obj.tmp-D;*
EOF
);
}
diff --git a/Configurations/unix-Makefile.tmpl
b/Configurations/unix-Makefile.tmpl
index 0be8cb1..e9d112b 100644
--- a/Configurations/unix-Makefile.tmpl
+++ b/Configurations/unix-Makefile.tmpl
@@ -1032,14 +1032,9 @@ $obj$objext: $deps
EOF
if (defined $makedepprog && $makedepprog =~ /\/makedepend/) {
$recipe .= <<"EOF";
- -\$(MAKEDEPEND) -f- -o"|\$\@" -- $incs $cmdflags -- $srcs \\
- >$obj$depext.tmp 2>/dev/null
- -\$(PERL) -i -pe 's/^.*\\|//; s/ \\/(\\\\.|[^ ])*//; \$\$_ = undef if
(/: *\$\$/ || /^(#.*| *)\$\$/); \$\$_.="\\n" unless !defined(\$\$_) or
/\\R\$\$/g;' $obj$depext.tmp
- \@if cmp $obj$depext.tmp $obj$depext > /dev/null 2> /dev/null; then \\
- rm -f $obj$depext.tmp; \\
- else \\
- mv $obj$depext.tmp $obj$depext; \\
- fi
+ \$(MAKEDEPEND) -f- -o"|\$\@" -- $incs $cmdflags -- $srcs 2>/dev/null \\
+ | \$(PERL) \$(SRCDIR)/util/postprocess-makedepend.pl \\
+ 'makedepend' > $obj$depext
EOF
}
}
diff --git a/Configurations/windows-makefile.tmpl
b/Configurations/windows-makefile.tmpl
index b2742e0..8e7d18c 100644
--- a/Configurations/windows-makefile.tmpl
+++ b/Configurations/windows-makefile.tmpl
@@ -599,13 +599,9 @@ EOF
}
return <<"EOF" if (!$disabled{makedepend});
$obj$depext: $deps
- \$(CC) $cflags /Zs /showIncludes $srcs 2>&1 | \\
- "\$(PERL)" -n << > $obj$depext
-chomp;
-s/^Note: including file: *//;
-\$\$collect{\$\$_} = 1;
-END { print '$obj$objext: ',join(" ", sort keys \%collect),"\\n" }
-<<
+ \$(CC) $cflags /Zs /showIncludes $srcs 2>&1 \\
+ | "\$(PERL)" "\$(SRCDIR)\\util\\postprocess-makedepend.pl" \\
+ "VC" "$obj$objext" > $obj$depext
$obj$objext: $obj$depext
\$(CC) $cflags -c \$(COUTFLAG)\$\@ @<<
$srcs
diff --git a/util/postprocess-makedepend.pl b/util/postprocess-makedepend.pl
new file mode 100644
index 0000000..323ce9e
--- /dev/null
+++ b/util/postprocess-makedepend.pl
@@ -0,0 +1,160 @@
+#! /usr/bin/env perl
+# Copyright 2018 The OpenSSL Project Authors. All Rights Reserved.
+#
+# Licensed under the OpenSSL license (the "License"). You may not use
+# this file except in compliance with the License. You can obtain a copy
+# in the file LICENSE in the source distribution or at
+# https://www.openssl.org/source/license.html
+
+use strict;
+use warnings;
+
+use lib '.';
+use configdata;
+
+use File::Spec::Functions qw(canonpath rel2abs);
+
+my $abs_srcdir = rel2abs($config{sourcedir});
+my $abs_blddir = rel2abs($config{builddir});
+
+my $producer = shift @ARGV;
+die "Producer not given\n" unless $producer;
+
+my $procedure = {
+ 'makedepend' =>
+ sub {
+ my $line = shift;
+
+ # makedepend, in its infinite wisdom, wants to have the object file
+ # in the same directory as the source file. This doesn't work too
+ # well with out-of-source-tree builds, so we must resort to tricks
+ # to get things right. The trick is to call makedepend with an
+ # extra suffix that contains the desired object file path, like
+ # this:
+ #
+ # makedepend -f- -o"|dir/foo.o" -- $(CFLAGS) -- ../some/foo.c
+ #
+ # The result will look something like this:
+ #
+ # ../somewhere/foo|dir/foo.o: deps...
+ #
+ # Which is easy to massage by removing everything up to the first |
+
+ # Remove everything up to the first |
+ $line =~ s/^.*\|//;
+ # Also, remove any dependency that starts with a /, because those
+ # are typically system headers
+ $line =~ s/\s+\/(\\.|\S)*//g;
+ # Finally, discard all empty lines or comment lines
+ return undef if $line =~ /:\s*$/ || $line =~ /^(#.*|\s*)$/;
+
+ my ($target, $deps) = $line =~ /^((?:\\.|[^:])*):(.*)/;
+ $deps =~ s/^\s+//;
+ $deps =~ s/\s+$//;
+ return ($target, $deps);
+ },
+ 'VMS C' =>
+ sub {
+ my $line = shift;
+
+ # current versions of DEC / Compaq / HP / VSI C strips away all
+ # directory information from the object file, so we must insert it
+ # back. Just to be safe against future changes, we check that there
+ # really is no directory information.
+ my $directory = shift;
+
+ # The pattern for target and dependencies will always take this
+ # form:
+ #
+ # target SPACE : SPACE deps
+ #
+ # This is so a volume delimiter (a : without any spaces around it)
+ # won't get mixed up with the target / deps delimiter. We use this
+ # fact in the regexp below to make sure we do look at the target.
+ $line =~ s/^/$directory/ unless /^\S+[:>\]]\S+\s+:/;
+
+ # We know that VMS has system header files in text libraries,
+ # extension .TLB. We also know that our header files aren't stored
+ # in text libraries. Finally, we know that VMS C produces exactly
+ # one dependency per line, so we simply discard any line ending
with
+ # .TLB.
+ return undef if /\.TLB\s*$/;
+
+ my ($target, $deps) = $line =~ /^(.*)\s:\s(.*)/;
+ $deps =~ s/^\s+//;
+ $deps =~ s/\s+$//;
+ return ($target, $deps);
+ },
+ 'VC' =>
+ sub {
+ my $line = shift;
+ my $object = shift;
+
+ # For the moment, we only support Visual C on native Windows, or
+ # compatible compilers. With those, the flags /Zs /showIncludes
+ # give us the necessary output to be able to create dependencies
+ # that nmake (or any 'make' implementation) should be able to read,
+ # with a bit of help. The output we're interested in looks like
+ # this (it always starts the same)
+ #
+ # Note: including file: {whatever header file}
+ #
+ # So all we really have to do is to is to replace the start of the
+ # line with an object file specification, given to us as an extra
+ # argument (passed from $ARGV[1]);
+ #
+ # There are also other lines mixed in, for example compiler
+ # warnings, so we simply discard anything that doesn't start with
+ # the Note:
+
+ if (/^Note: including file: */) {
+ (my $tail = $') =~ s/\s*\R$//;
+
+ # VC gives us absolute paths for all include files, so to
+ # remove system header dependencies, we need to check that
+ # they don't match $abs_srcdir or $abs_blddir
+ $tail = canonpath($tail);
+ if ($tail =~ m|^\Q$abs_srcdir\E|i
+ || $tail =~ m|^\Q$abs_blddir\E|i) {
+ return ($object, "\"$tail\"");
+ }
+ }
+
+ return undef;
+ },
+} -> {$producer};
+
+die "Producer unrecognised: $producer\n" unless defined $procedure;
+
+my %collect = ();
+while (<STDIN>) {
+ s|\R$||; # The better chomp
+ my ($target, $deps) = $procedure->($_, @ARGV);
+ $collect{$target}->{$deps} = 1
+ if defined $target;
+}
+
+my $continuation = {
+ 'makedepend' => "\\",
+ 'VMS C' => "-",
+ 'VC' => "\\",
+} -> {$producer};
+
+die "Producer unrecognised: $producer\n" unless defined $continuation;
+
+foreach my $target (sort keys %collect) {
+ my $prefix = $target . ' :';
+ my @deps = sort keys %{$collect{$target}};
+
+ while (@deps) {
+ my $buf = $prefix;
+ $prefix = '';
+
+ while (@deps && ($buf eq '' || length($buf) + length($deps[0]) <= 77))
{
+ $buf .= ' ' . shift @deps;
+ }
+ $buf .= ' '.$continuation if @deps;
+
+ print $buf,"\n";
+ }
+}
_____
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits
