Fix use of config-specific libraries for Windows OpenSSL

adunstan · adunstan · commit 5404145c5ad4 · 2018-01-03T15:34:21.000-05:00
Commit 614350a allowed for an different builds of OpenSSL libraries on Windows, but ignored the fact that the alternative builds don't have config-specific libraries. This patch fixes the Solution file to ask for the correct libraries. per offline discussions with Leonardo Cecchi and Marco Nenciarini, Backpatch to all live branches.
diff --git a/src/tools/msvc/Solution.pm b/src/tools/msvc/Solution.pm
@@ -513,10 +513,12 @@ sub AddProject
 		}
 		else
 		{
+			# We don't expect the config-specific library to be here,
+			# so don't ask for it in last parameter
 			$proj->AddLibrary(
-				$self->{options}->{openssl} . '\lib\ssleay32.lib', 1);
+				$self->{options}->{openssl} . '\lib\ssleay32.lib', 0);
 			$proj->AddLibrary(
-				$self->{options}->{openssl} . '\lib\libeay32.lib', 1);
+				$self->{options}->{openssl} . '\lib\libeay32.lib', 0);
 		}
 	}
 	if ($self->{options}->{nls})

Original file line number	Diff line number	Diff line change
`@@ -513,10 +513,12 @@ sub AddProject`
`513`	`513`	`}`
`514`	`514`	`else`
`515`	`515`	`{`
	`516`	`+ # We don't expect the config-specific library to be here,`
	`517`	`+ # so don't ask for it in last parameter`
`516`	`518`	`$proj->AddLibrary(`
`517`		`- $self->{options}->{openssl} . '\lib\ssleay32.lib', 1);`
	`519`	`+ $self->{options}->{openssl} . '\lib\ssleay32.lib', 0);`
`518`	`520`	`$proj->AddLibrary(`
`519`		`- $self->{options}->{openssl} . '\lib\libeay32.lib', 1);`
	`521`	`+ $self->{options}->{openssl} . '\lib\libeay32.lib', 0);`
`520`	`522`	`}`
`521`	`523`	`}`
`522`	`524`	`if ($self->{options}->{nls})`