[openssl-commits] Still FAILED build of OpenSSL branch master with options -d --strict-warnings no-dso

2019-01-22 Thread OpenSSL run-checker
Platform and configuration command:

$ uname -a
Linux run 4.4.0-135-generic #161-Ubuntu SMP Mon Aug 27 10:45:01 UTC 2018 x86_64 
x86_64 x86_64 GNU/Linux
$ CC=clang ../openssl/config -d --strict-warnings no-dso

Commit log since last time:

5cae2d349b Build: change remaining $unified_info{install} checks to use 
attributes
13234dd310 Rework build: Windows dependency building fix
5f8257494c Build: pass attributes down to make rule generators
994e86a9ff Build: use attributes to indicate installed script classes
ac6bba6f6e Build: Change all _NO_INST to use attributes instead.
c91f24d4cc Configure: add attributes to end product build.info variables
5d3af25934 Configure: teach the tokenizer to handle other separators than spaces
52bcd4afc8 Update NOTES.ANDROID

Build log ended with (last 100 lines):

../../openssl/test/recipes/30-test_pkey_meth_kdf.t  ok
../../openssl/test/recipes/40-test_rehash.t ... ok
../../openssl/test/recipes/60-test_x509_check_cert_pkey.t . ok
../../openssl/test/recipes/60-test_x509_dup_cert.t  ok
../../openssl/test/recipes/60-test_x509_store.t ... ok
../../openssl/test/recipes/60-test_x509_time.t  ok
../../openssl/test/recipes/70-test_asyncio.t .. ok
../../openssl/test/recipes/70-test_bad_dtls.t . ok
../../openssl/test/recipes/70-test_clienthello.t .. ok
../../openssl/test/recipes/70-test_comp.t . skipped: 
test_comp needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_key_share.t  skipped: 
test_key_share needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_packet.t ... ok
../../openssl/test/recipes/70-test_recordlen.t  ok
../../openssl/test/recipes/70-test_renegotiation.t  skipped: 
test_renegotiation needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_servername.t ... ok
../../openssl/test/recipes/70-test_sslcbcpadding.t  skipped: 
test_sslcbcpadding needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_sslcertstatus.t  skipped: 
test_sslcertstatus needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_sslextension.t . skipped: 
test_sslextension needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_sslmessages.t .. skipped: 
test_sslmessages needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_sslrecords.t ... skipped: 
test_sslrecords needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_sslsessiontick.t ... skipped: 
test_sslsessiontick needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_sslsigalgs.t ... skipped: 
test_sslsigalgs needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_sslsignature.t . skipped: 
test_sslsignature needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_sslskewith0p.t . skipped: 
test_sslskewith0p needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_sslversions.t .. skipped: 
test_sslversions needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_sslvertol.t  skipped: 
test_sslextension needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_tls13alerts.t .. skipped: 
test_tls13alerts needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_tls13cookie.t .. skipped: 
test_tls13cookie needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_tls13downgrade.t ... skipped: 
test_tls13downgrade needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_tls13hrr.t . skipped: 
test_tls13hrr needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_tls13kexmodes.t  skipped: 
test_tls13kexmodes needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_tls13messages.t  skipped: 
test_tls13messages needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_tls13psk.t . skipped: 
test_tls13psk needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_tlsextms.t . skipped: 
test_tlsextms needs the dynamic engine feature enabled
../../openssl/test/recipes/70-test_verify_extra.t . ok
../../openssl/test/recipes/70-test_wpacket.t .. ok
../../openssl/test/recipes/80-test_ca.t ... ok
../../openssl/test/recipes/80-test_cipherbytes.t .. ok
../../openssl/test/recipes/80-test_cipherlist.t ... ok
../../openssl/test/recipes/80-test_ciphername.t ... ok
../../openssl/test/recipes/80-test_cms.t .. ok
../../openssl/te

[openssl-commits] Build completed: openssl master.22053

2019-01-22 Thread AppVeyor


Build openssl master.22053 completed



Commit 5cae2d349b by Richard Levitte on 1/22/2019 3:12 PM:

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


Configure your notification preferences

_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits


[openssl-commits] Build failed: openssl master.22052

2019-01-22 Thread AppVeyor



Build openssl master.22052 failed


Commit ea217473ff by Matt Caswell on 1/22/2019 2:27 PM:

Fix s_client so that it builds on Windows


Configure your notification preferences

_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits


[openssl-commits] [openssl] master update

2019-01-22 Thread Richard Levitte
The branch master has been updated
   via  5cae2d349b561a84dbfc93d6b6abc5fb7263fb7c (commit)
  from  13234dd310511ed2ae1832bb643dd298ddfefb0b (commit)


- Log -
commit 5cae2d349b561a84dbfc93d6b6abc5fb7263fb7c
Author: Richard Levitte 
Date:   Tue Jan 22 15:46:54 2019 +0100

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

Reviewed-by: Matt Caswell 
(Merged from https://github.com/openssl/openssl/pull/8063)

---

Summary of changes:
 Configurations/common.tmpl | 36 
 Configurations/descrip.mms.tmpl| 12 ++--
 Configurations/platform/Unix.pm|  4 +---
 Configurations/platform/VMS.pm |  4 +---
 Configurations/platform/Windows.pm |  4 +---
 5 files changed, 21 insertions(+), 39 deletions(-)

diff --git a/Configurations/common.tmpl b/Configurations/common.tmpl
index 132852c..53384c7 100644
--- a/Configurations/common.tmpl
+++ b/Configurations/common.tmpl
@@ -52,18 +52,6 @@
  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
  # generated source files.  It recurses in case a dependency is also a
  # generated source file.
@@ -132,14 +120,14 @@
  $OUT .= $obj2shlib->(lib => $lib,
   attrs => $unified_info{attributes}->{$lib},
   objs => $unified_info{shared_sources}->{$lib},
-  deps => [ reducedepends(resolvedepends($lib)) ],
-  installed => is_installed($lib));
+  deps => [ reducedepends(resolvedepends($lib)) ]);
  foreach ((@{$unified_info{shared_sources}->{$lib}},
@{$unified_info{sources}->{$lib}})) {
  # If this is somehow a compiled object, take care of it that way
  # Otherwise, it might simply be generated
  if (defined $unified_info{sources}->{$_}) {
- doobj($_, $lib, intent => "shlib", installed => 
is_installed($lib));
+ doobj($_, $lib, intent => "shlib",
+   attrs => $unified_info{attributes}->{$lib});
  } else {
  dogenerate($_, undef, undef, intent => "lib");
  }
@@ -149,7 +137,8 @@
  attrs => $unified_info{attributes}->{$lib},
  objs => [ @{$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;
  }
@@ -163,13 +152,13 @@
  $OUT .= obj2dso(lib => $lib,
  attrs => $unified_info{attributes}->{$lib},
  objs => $unified_info{shared_sources}->{$lib},
- deps => [ resolvedepends($lib) ],
- installed => is_installed($lib));
+ deps => [ resolvedepends($lib) ]);
  foreach (@{$unified_info{shared_sources}->{$lib}}) {
  # If this is somehow a compiled object, take care of it that way
  # Otherwise, it might simply be generated
  if (defined $unified_info{sources}->{$_}) {
- doobj($_, $lib, intent => "dso", installed => is_installed($lib));
+ doobj($_, $lib, intent => "dso",
+   attrs => $unified_info{attributes}->{$lib});
  } else {
  dogenerate($_, undef, $lib, intent => "dso");
  }
@@ -186,10 +175,10 @@
  $OUT .= obj2bin(bin => $bin,
  attrs => $unified_info{attributes}->{$bin},
  objs => [ @{$unified_info{sources}->{$bin}} ],
- deps => $deps,
- installed => is_installed($bin));
+ deps => $deps);
  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;
  }
@@ -201,8 +190,7 @@
  return "" if $cache{$script};
  $OUT .= in2script(script => $script,
attrs => $unified_info{attributes}->{$script},
-   sources => $unified_info{sources}->{$script},
-   installed => is_installed($script));
+   sources => $unifi

[openssl-commits] [openssl] master update

2019-01-22 Thread Richard Levitte
The branch master has been updated
   via  13234dd310511ed2ae1832bb643dd298ddfefb0b (commit)
  from  5f8257494c72ba4ea2a99d693916798517a610e1 (commit)


- Log -
commit 13234dd310511ed2ae1832bb643dd298ddfefb0b
Author: Richard Levitte 
Date:   Tue Jan 22 12:17:36 2019 +0100

Rework build: Windows dependency building fix

One variable misssing

Fixes #8060

Reviewed-by: Matt Caswell 
(Merged from https://github.com/openssl/openssl/pull/8061)

---

Summary of changes:
 Configurations/windows-makefile.tmpl | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Configurations/windows-makefile.tmpl 
b/Configurations/windows-makefile.tmpl
index 872ef4b..0cd1e86 100644
--- a/Configurations/windows-makefile.tmpl
+++ b/Configurations/windows-makefile.tmpl
@@ -613,6 +613,7 @@ $res: $deps
 EOF
  }
  my $obj = platform->obj($args{obj});
+ my $dep = platform->dep($args{obj});
  if ($srcs[0] =~ /\.asm$/) {
  return <<"EOF";
 $obj: $deps
_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits


[openssl-commits] Build failed: openssl master.22049

2019-01-22 Thread AppVeyor



Build openssl master.22049 failed


Commit 5f8257494c by Richard Levitte on 1/22/2019 11:35 AM:

Build: pass attributes down to make rule generators


Configure your notification preferences

_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits


[openssl-commits] Build completed: openssl master.22045

2019-01-22 Thread AppVeyor


Build openssl master.22045 completed



Commit baec8c29b5 by Richard Levitte on 1/22/2019 11:17 AM:

Rework build: Windows dependency building fix


Configure your notification preferences

_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits


[openssl-commits] [openssl] master update

2019-01-22 Thread Richard Levitte
The branch master has been updated
   via  5f8257494c72ba4ea2a99d693916798517a610e1 (commit)
   via  994e86a9ffd4195f08a7b0ce61bf001e3bebf891 (commit)
   via  ac6bba6f6ea328ba22425d6f3f95847452193293 (commit)
   via  c91f24d4cca5862f11876457e0ffb6dd54814814 (commit)
   via  5d3af25934dc5a6850004d6e58af6a89df97e927 (commit)
  from  52bcd4afc84d75f9d22866a3cefaf9ae4e9ff997 (commit)


- Log -
commit 5f8257494c72ba4ea2a99d693916798517a610e1
Author: Richard Levitte 
Date:   Wed Nov 7 11:10:50 2018 +0100

Build: pass attributes down to make rule generators

For good measure, we pass down attributes when calling obj2shlib,
obj2lib, obj2dso, obj2bin, or in2script.  We currently don't use them
in our build file templates, but might as well for future use.

Reviewed-by: Matt Caswell 
(Merged from https://github.com/openssl/openssl/pull/7581)

commit 994e86a9ffd4195f08a7b0ce61bf001e3bebf891
Author: Richard Levitte 
Date:   Wed Nov 7 11:05:17 2018 +0100

Build: use attributes to indicate installed script classes

We have two classes of scripts to be installed, those that are
installed as "normal" programs, and those that are installed as "misc"
scripts.  These classes are installed in different locations, so the
build file templates must pay attention.

Because we didn't have the tools to indicate what scripts go where, we
had these scripts hard coded in the build template files, with the
maintenance issues that may cause.  Now that we have attributes, those
can be used to classify the installed scripts, and have the build file
templates simply check the attributes to know what's what.

Furthermore, the 'tsget.pl' script exists both as 'tsget.pl' and
'tsget', which is done by installing a symbolic link (or copy).  This
link name is now given through an attribute, which results in even
less hard coding in the Unix Makefile template.

Reviewed-by: Matt Caswell 
(Merged from https://github.com/openssl/openssl/pull/7581)

commit ac6bba6f6ea328ba22425d6f3f95847452193293
Author: Richard Levitte 
Date:   Wed Nov 7 11:02:06 2018 +0100

Build: Change all _NO_INST to use attributes instead.

This means that all PROGRAMS_NO_INST, LIBS_NO_INST, ENGINES_NO_INST
and SCRIPTS_NO_INST are changed to be PROGRAM, LIBS, ENGINES and
SCRIPTS with the associated attribute 'noinst'.

Reviewed-by: Matt Caswell 
(Merged from https://github.com/openssl/openssl/pull/7581)

commit c91f24d4cca5862f11876457e0ffb6dd54814814
Author: Richard Levitte 
Date:   Wed Nov 7 10:44:05 2018 +0100

Configure: add attributes to end product build.info variables

Among others, this avoids having special variables like
PROGRAMS_NO_INST.  Instead, we can have something like this:

PROGRAMS{noinst}=foo bar

Configure itself is entirely agnostic to these attributes, they are
simply passed to the build file templates, to be used as they see fit.

Attributes can also have values, for example:

SCRIPTS{linkname=foo}=foo.pl

This could help indicate to build file templates that care that the
perl script 'foo.pl' should also exist with the name 'foo', preferably
as a symbolic link.

Fixes #7568

Reviewed-by: Matt Caswell 
(Merged from https://github.com/openssl/openssl/pull/7581)

commit 5d3af25934dc5a6850004d6e58af6a89df97e927
Author: Richard Levitte 
Date:   Wed Nov 7 10:34:05 2018 +0100

Configure: teach the tokenizer to handle other separators than spaces

Reviewed-by: Matt Caswell 
(Merged from https://github.com/openssl/openssl/pull/7581)

---

Summary of changes:
 Configurations/common.tmpl   |   5 +
 Configurations/descrip.mms.tmpl  |  32 --
 Configurations/unix-Makefile.tmpl|  53 ++---
 Configurations/windows-makefile.tmpl |  63 ---
 Configure| 203 ---
 apps/build.info  |   7 +-
 engines/build.info   |   2 +-
 fuzz/build.info  |  12 +--
 test/build.info  |  34 +++---
 test/ossl_shim/build.info|   2 +-
 util/build.info  |   4 +-
 11 files changed, 266 insertions(+), 151 deletions(-)

diff --git a/Configurations/common.tmpl b/Configurations/common.tmpl
index 9e07a6f..132852c 100644
--- a/Configurations/common.tmpl
+++ b/Configurations/common.tmpl
@@ -130,6 +130,7 @@
  unless ($disabled{shared} || $lib =~ /\.a$/) {
  my $obj2shlib = defined &obj2shlib ? \&obj2shlib : \&libobj2shlib;
  $OUT .= $obj2shlib->(lib => $lib,
+  attrs => $unified_info{attributes}->{$lib},
   objs => $unified_info{sha

[openssl-commits] [openssl] OpenSSL_1_1_1-stable update

2019-01-22 Thread Richard Levitte
The branch OpenSSL_1_1_1-stable has been updated
   via  d0a4e858bb658c098cd267252e1e7cfffe554aff (commit)
  from  8c175d2d4f1fc630b5da8d4a5aeb85b21105a2b5 (commit)


- Log -
commit d0a4e858bb658c098cd267252e1e7cfffe554aff
Author: Matt Eaton 
Date:   Mon Jan 21 20:14:34 2019 -0600

Update NOTES.ANDROID

Minor typo fix to `adjustment` in the line:
"In such case you have to pass matching target
 name to Configure and shouldn't use -D__ANDROID_API__=N. PATH adjustment
 becomes simpler, $ANDROID_NDK/bin:$PATH suffices."

Reviewed-by: Matt Caswell 
Reviewed-by: Richard Levitte 
(Merged from https://github.com/openssl/openssl/pull/8054)

(cherry picked from commit 52bcd4afc84d75f9d22866a3cefaf9ae4e9ff997)

---

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

diff --git a/NOTES.ANDROID b/NOTES.ANDROID
index bbbd8e4..6b4741c 100644
--- a/NOTES.ANDROID
+++ b/NOTES.ANDROID
@@ -54,7 +54,7 @@
  Another option is to create so called "standalone toolchain" tailored
  for single specific platform including Android API level, and assign its
  location to ANDROID_NDK. In such case you have to pass matching target
- name to Configure and shouldn't use -D__ANDROID_API__=N. PATH adjusment
+ name to Configure and shouldn't use -D__ANDROID_API__=N. PATH adjustment
  becomes simpler, $ANDROID_NDK/bin:$PATH suffices.
 
  Running tests (on Linux)
_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits


[openssl-commits] [openssl] master update

2019-01-22 Thread Richard Levitte
The branch master has been updated
   via  52bcd4afc84d75f9d22866a3cefaf9ae4e9ff997 (commit)
  from  f5fb6f0543cafd3db6671cfb987bf475a35f30f6 (commit)


- Log -
commit 52bcd4afc84d75f9d22866a3cefaf9ae4e9ff997
Author: Matt Eaton 
Date:   Mon Jan 21 20:14:34 2019 -0600

Update NOTES.ANDROID

Minor typo fix to `adjustment` in the line:
"In such case you have to pass matching target
 name to Configure and shouldn't use -D__ANDROID_API__=N. PATH adjustment
 becomes simpler, $ANDROID_NDK/bin:$PATH suffices."

Reviewed-by: Matt Caswell 
Reviewed-by: Richard Levitte 
(Merged from https://github.com/openssl/openssl/pull/8054)

---

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

diff --git a/NOTES.ANDROID b/NOTES.ANDROID
index bbbd8e4..6b4741c 100644
--- a/NOTES.ANDROID
+++ b/NOTES.ANDROID
@@ -54,7 +54,7 @@
  Another option is to create so called "standalone toolchain" tailored
  for single specific platform including Android API level, and assign its
  location to ANDROID_NDK. In such case you have to pass matching target
- name to Configure and shouldn't use -D__ANDROID_API__=N. PATH adjusment
+ name to Configure and shouldn't use -D__ANDROID_API__=N. PATH adjustment
  becomes simpler, $ANDROID_NDK/bin:$PATH suffices.
 
  Running tests (on Linux)
_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits


[openssl-commits] Build failed: openssl master.22044

2019-01-22 Thread AppVeyor



Build openssl master.22044 failed


Commit 5fb872c7c6 by Richard Levitte on 1/22/2019 10:05 AM:

Build tls13secretstest correctly


Configure your notification preferences

_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits