cqframework / atom_cql_support

Apache License 2.0
4 stars 3 forks source link

Ubuntu 20.04 OpenJDK11 language-cql parser error #25

Closed DrMuir closed 3 years ago

DrMuir commented 3 years ago

Atom language-CQL Java Version parsing error Ubuntu 20.04 includes OpenJDK11 but includes multiple versions:

$ java -version openjdk version "11.0.9.1" 2020-11-04 OpenJDK Runtime Environment (build 11.0.9.1+1-Ubuntu-0ubuntu1.20.04) OpenJDK 64-Bit Server VM (build 11.0.9.1+1-Ubuntu-0ubuntu1.20.04, mixed mode, sharing)

$ which java /usr/bin/java

$ echo $JAVA_HOME /usr/lib/jvm/java-11-openjdk-amd64

I uninstalled packages and Atom and have successfully reinstalled both via Ubuntu software and via Atom.io Latest versions of packages installed successfully for atom-ide-iu, ide-java and language-cql Pop up error as follows:

language-cql requires Java 11 but could not determine your Java version.

Could not parse the Java '--showVersion' output openjdk version "11.0.9.1" 2020-11-04 OpenJDK Runtime Environment (build 11.0.9.1+1-Ubuntu-0ubuntu1.20.04) OpenJDK 64-Bit Server VM (build 11.0.9.1+1-Ubuntu-0ubuntu1.20.04, mixed mode, sharing)

I suspect that the multiple line response is the issue for the language-cql parser. (since Ubuntu 20.04 includes multiple versions of Java) I suspect this may impact others running Linux/Ubuntu 20.04 (MacOs runs ok)

Appreciate your input, Thanks

JPercival commented 3 years ago

The issue is that the version detection logic doesn't handle a 4 part version number (11.0.9.1). Should be an easy fix.

DrMuir commented 3 years ago

Thank you! -- Peter Muir MD E-mail: peter.muir@esacinc.com

On Tue, Dec 8, 2020 at 4:45 PM JPercival notifications@github.com wrote:

The issue is that the version detection logic doesn't handle a 4 part version number (11.0.9.1). Should be an easy fix.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/cqframework/atom_cql_support/issues/25#issuecomment-741082486, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJBBZR3NZOUPAPXNRHQQYXDST2M6FANCNFSM4USQWLFQ .