The change in the library names comes directly from OpenSSL [2].
+ if (-e "$self->{options}->{openssl}/lib/VC/libssl32MD.lib") Why not using a version-specific logic here?
The version logic is just before that:
+ my ($major, $minor) = $self->GetOpenSSLVersion(); + if ($major == 1 && $minor == 1)
I guess that what you mean is, why testing the 32/64 bits using the libraries instead of using the 'platform'? I try to make it clearer in this version.
+ my ($major, $minor) = $self->GetOpenSSLVersion(); + if ($major == 1 && $minor == 1) + { + print $o "#define HAVE_BIO_GET_DATA 1\n"; + print $o "#define HAVE_BIO_METH_NEW 1\n"; + } I think that you are missing HAVE_OPENSSL_INIT_SSL and HAVE_ASN1_STRING_GET0_DATA here. Please see commit message of bde64eb.
Yes, you are right. Since those do not break the compilation between 1.0.2 and 1.1.0 I did not notice them.