Hi Damjan,

I added 2 trailing "/" and now it works... ;-)
Maybe Patricia can confirm it?

I will now start a build from trunk with Cygwin64 and put the results here:
https://home.apache.org/~mseidel/AOO-builds/AOO-420-Test-Cygwin64/

Regards, Matthias


Am 14.11.2017 um 03:06 schrieb Damjan Jovanovic:
> Please test whether the attached patch helps?
>
> Regards
> Damjan
>
> On Sat, Nov 11, 2017 at 8:02 PM, Matthias Seidel
> <[email protected] <mailto:[email protected]>> wrote:
>
>     Hi Damjan,
>
>     Am 10.11.2017 um 06:27 schrieb Damjan Jovanovic:
>     > Hi
>     >
>     > I've committed a few small small fixes to oowintool, mostly for
>     JDK 8
>     > detection and better logging, but I've confirmed that it already
>     works
>     > well, and --with-frame-home --with-psdk-home --with-midl-path
>     > --with-jdk-home --with-csc-path and --with-cl-home options to
>     ./configure
>     > are all unnecessary.
>     >
>     > As for 64 bit Cygwin I believe the problem with oowintool is
>     that it needs
>     > to look under the Wow6432node subkey in the registry too,
>     something that
>     > should be easy to add. Is anyone available to test 64 bit Cygwin?
>
>     I just installed Cygwin64 on Windows 10 and ./configure stops with:
>
>     Can't find MS Visual Studio / VC++ at ./oowintool line 236.
>     configure: error: oowintool failed to copy CRT
>
>     Regards, Matthias
>
>     >
>     > Damjan
>     >
>
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]

Index: oowintool
===================================================================
--- oowintool	(revision 1815058)
+++ oowintool	(working copy)
@@ -217,6 +217,11 @@
 	    $ver->{'product_dir'} = $install;
 	    return $ver;
 	}
+	$install = reg_get_value ("HKEY_LOCAL_MACHINE/SOFTWARE/Wow6432Node/" . $ver->{'key'});
+	if (defined $install && $install ne '') {
+	    $ver->{'product_dir'} = $install;
+	    return $ver;
+	}
     }
     die "Can't find MS Visual Studio / VC++";
 }
@@ -232,6 +237,11 @@
 	    $ver->{'product_dir'} = $install;
 	    return $ver;
 	}
+	$install = reg_get_value("HKEY_LOCAL_MACHINE/SOFTWARE/Wow6432Node/" . $ver->{'key'});
+	if (defined $install && $install ne '') {
+	    $ver->{'product_dir'} = $install;
+	    return $ver;
+	}
     }
     die "Can't find MS Visual Studio / VC++";
 }

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

Reply via email to