Re: RFR: 8237944: webview native cl "-m32" unknown option for windows 32-bit build
On Tue, 28 Jan 2020 09:50:27 GMT, Guru Hb wrote: > cl : Command line warning D9002 : ignoring unknown option '-m32' > > post fix for "https://trac.webkit.org/changeset/242724/webkit; makes use of > cross compiling 32 bit JSC in a 64 bit and its holds good only for Linux. > '-m32' flag is gcc specifc and on windows cl.exe (visual studio) doesn't > recognize this flag. All my testing looks good. I added one comment for you to take a look at. modules/javafx.web/src/main/native/Tools/Scripts/webkitdirs.pm line 2300: > 2299: > 2300: if (architecture() eq "x86_64" && shouldBuild32Bit() && (isJava() > && !isCygwin())) { > 2301: # CMAKE_LIBRARY_ARCHITECTURE is needed to get the right .pc This will work, since all we build is the Java platform, but I'm not sure that the `isJava()` test is quite right. It suggests that the logic should be skipped for other platforms (i.e., that the entire test and body of the if block is Java platform-specific), which it isn't. I think this should be: ... && !(isJava() && isCygwin()) - PR: https://git.openjdk.java.net/jfx/pull/97
Re: RFR: 8237944: webview native cl "-m32" unknown option for windows 32-bit build
On Tue, 28 Jan 2020 21:03:01 GMT, Johan Vos wrote: >> cl : Command line warning D9002 : ignoring unknown option '-m32' >> >> post fix for "https://trac.webkit.org/changeset/242724/webkit; makes use of >> cross compiling 32 bit JSC in a 64 bit and its holds good only for Linux. >> '-m32' flag is gcc specifc and on windows cl.exe (visual studio) doesn't >> recognize this flag. > > the -m32 option seems to be ignored by the compiler: > cl : Command line warning D9002 : ignoring unknown option '-m32' > > However, I agree it is better to conditionally remove it. Agreed. It's a good cleanup fix. I'll do a quick test and then review it. - PR: https://git.openjdk.java.net/jfx/pull/97
Re: RFR: 8237944: webview native cl "-m32" unknown option for windows 32-bit build
On Tue, 28 Jan 2020 09:50:27 GMT, Guru Hb wrote: > cl : Command line warning D9002 : ignoring unknown option '-m32' > > post fix for "https://trac.webkit.org/changeset/242724/webkit; makes use of > cross compiling 32 bit JSC in a 64 bit and its holds good only for Linux. > '-m32' flag is gcc specifc and on windows cl.exe (visual studio) doesn't > recognize this flag. the -m32 option seems to be ignored by the compiler: cl : Command line warning D9002 : ignoring unknown option '-m32' However, I agree it is better to conditionally remove it. - PR: https://git.openjdk.java.net/jfx/pull/97