If you get it working do come back and contribute. We did recompile the netbeans.exe launchers for NetBeans 9 (and we are reusing them for NetBeans 10 too), but I don't remember a discussion about some other exe to recompile.
Note that the NetBeans installer is just being donated (there's a PR about it on GitHub) but I see that nbi has been donated since the start. So... just to be safe I reopened https://issues.apache.org/jira/browse/NETBEANS-1157 , perhaps there's something missing. --emi http://coolbeans.xyz/ - CoolBeans: An IDE for Java, JavaEE, PHP and more! On Sat, Jan 5, 2019 at 9:55 PM Michael Huang <[email protected]> wrote: > > Hi, > > Thanks for the update. > > Yes, I think that may be the case (it was not recompiled). I didn't find > nlw.exe from the Netbeans10 source and now I remember that I populated that > from Netbeans 9.0.... including the problematic source. It's good that it's > now fixed. > > I will try to find recompiled nlw.exe as I'm having trouble recompiling a > working one using VisualStudio 2017 (the project was using Visual Studio > 2005). > > Regards, > > MH > ----------------------------------------------- > SPISim: [www.spisim.com] > EDA in Signal, Power Integrity and SIMulation > ----------------------------------------------- > > > On Sat, Jan 5, 2019 at 11:43 AM Emilian Bold <[email protected]> wrote: > > > Maybe we are looking at different versions of JavaUtils.c but to me it > > looks like it has a small parser and it expressly supports major > > versions with more digits: > > > > https://github.com/apache/incubator-netbeans/blob/master/nbi/engine/native/launcher/windows/src/JavaUtils.c#L138 > > > > This has been fixed since Aug 21st by Andreas Hauffe in > > NETBEANS-1157, see > > > > https://github.com/apache/incubator-netbeans/commit/57cb8c469fb6024de3a3b9702fa468302acb4039#diff-f0f864825c4c1d5461c1a84cc9aa9e00 > > > > So you are saying the exe file from NetBeans 10 is not good? Maybe we > > never recompiled it? > > > > --emi > > > > http://coolbeans.xyz/ - CoolBeans: An IDE for Java, JavaEE, PHP and more! > > > > On Sat, Jan 5, 2019 at 9:26 PM Michael Huang <[email protected]> > > wrote: > > > > > > Hi, > > > > > > I think there is a bug in nbi's JavaUtils.c, line 125 > > > (Function getJavaVersionFromString), when it's used to unpack JVM of > > > version 10 or above such as OpenJDK11. This will affect netbeans launcher > > > on windows (nlw.exe) and thus need to have it rebuild after code fix. > > > > > > The current codes' assumption is that Java version (java -version) show > > > MAJOR.MINOR and the MAJOR only has one digit (e.g. 1.8, 1.7 etc). Codes > > use > > > string[1] to check "." first. For OpenJDK11, the java -version will show > > > 11.X... and thus the dot "." will not appear at the string[1]. As a > > result, > > > result code of "ERROR_OK" will not be assigned and JVM unpacking will > > fail. > > > > > > I have never committed codes to netbeans myself and thus will leave > > this > > > for someone more familiar with the code committing process to review > > first > > > and then commit if necessary. Hope this will help those whom may need > > > NetBeans launcher to unpack JVM during installation process on windows. > > > > > > Thanks and regards, > > > > > > MH > > > ----------------------------------------------- > > > SPISim: [www.spisim.com] > > > EDA in Signal, Power Integrity and SIMulation > > > ----------------------------------------------- > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > > For further information about the NetBeans mailing lists, visit: > > https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists > > > > > > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
