Fix use of config-specific libraries for Windows OpenSSL

adunstan · adunstan · commit 99d5a3ffb9fe · 2018-01-03T15:36:54.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
@@ -535,10 +535,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
`@@ -535,10 +535,12 @@ sub AddProject`
`535`	`535`	`}`
`536`	`536`	`else`
`537`	`537`	`{`
	`538`	`+ # We don't expect the config-specific library to be here,`
	`539`	`+ # so don't ask for it in last parameter`
`538`	`540`	`$proj->AddLibrary(`
`539`		`- $self->{options}->{openssl} . '\lib\ssleay32.lib', 1);`
	`541`	`+ $self->{options}->{openssl} . '\lib\ssleay32.lib', 0);`
`540`	`542`	`$proj->AddLibrary(`
`541`		`- $self->{options}->{openssl} . '\lib\libeay32.lib', 1);`
	`543`	`+ $self->{options}->{openssl} . '\lib\libeay32.lib', 0);`
`542`	`544`	`}`
`543`	`545`	`}`
`544`	`546`	`if ($self->{options}->{nls})`