Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Eclipse preference Dart SDK location validation always false - wrong compare string #179

Closed
Nawakland opened this issue Nov 5, 2020 · 4 comments

Comments

@Nawakland
Copy link
Contributor

Nawakland commented Nov 5, 2020

  • Test on both Windows 7 and Debian 10 OS
  • Eclipse v4.14
  • Dart v2.10.3

Today, the command dart --version return Dart SDK version: 2.10.3 [...]
So the compare string return version.startsWith("Dart VM version"); is always false/not valid line 136 from file org.eclipse.dartboard/src/org/eclipse/dartboard/preferences/DartSDKLocationFieldEditor.java

I suggest to test if the keywords "Dart" and "version" appear in the compare string (and better compare both in lower case to avoid mismatch).

Tell me when it will be available in nighty build to start working with.

Best regards,
Nawakland.

dartboard_issue

@vogella
Copy link
Contributor

vogella commented Nov 5, 2020

@Nawakland thanks for the report. Could you send a PR for the fix?

Nawakland added a commit to Nawakland/dartboard that referenced this issue Nov 6, 2020
Correct wrong string compare for Dart SDK location validation in Eclipse
preference view.

Signed-off-by: Bryan Claude Alain BERTHOU <bryan25.berthou@laposte.net>
@vogella vogella closed this as completed in 09b1b48 Nov 6, 2020
@Nawakland
Copy link
Contributor Author

@vogella What about nightly build ? Seems there is no update from commits.

@vogella
Copy link
Contributor

vogella commented Nov 10, 2020

@Nawakland sorry for this. I look into this on Friday, the next two days are already really busy.

@harshchokshi
Copy link

Any solution available.

Not a valid SDK Location?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants