-
Notifications
You must be signed in to change notification settings - Fork 34
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
CLI could not see Helidon version #487
Comments
Also the link in my output does not exist: |
Tried with latest CLI version, works good. Output:
|
local settings.xml issue? |
Seems like it gave up without an error message, I'm not too familiar with the code that does this. @tomas-langer the URL has been fixed in the latest release, but it's only going to take effect in the next upgrade. |
I can see that when I run |
This is my output when I connect through VPN:
|
Is the VPN output with the CLI or IDE ? |
CLI |
Identified where we throw away the debug output of the plugins: |
Looks like this variant of the execute method is designed to throw await the output. Usages in Metadata shows that the other variant is used and output shouldn't be thrown away: if (debugPlugin) {
args.add("--debug");
Plugins.execute(PLUGIN_NAME, args, PLUGIN_MAX_WAIT_SECONDS, Metadata::info);
} else {
Plugins.execute(PLUGIN_NAME, args, PLUGIN_MAX_WAIT_SECONDS);
}
The usage of metadata = Metadata.builder()
.url(metadataUrl)
.updateFrequency(config.checkForUpdatesIntervalHours())
.build(); I'm thinking the problem might be that It doesn't call |
- Make Plugins.PluginFailed a checked exception to force error handling - Wrap Plugins.PluginFailed with Metadata.UpdatedFailed as another checked exception to expose metadata update errors - Metadata.UpdateFailed is explicitly thrown by every public methods of Metadata that performs an update check Promote io.helidon.build.cli.codegen.Unchecked to io.helidon.build.util.Unchecke Rename util module from helidon.build.utils to io.helidon.build.util (same as the package name) Fixes helidon-io#487
* Improve Metadata udpate error handling: - Make Plugins.PluginFailed a checked exception to force error handling - Wrap Plugins.PluginFailed with Metadata.UpdatedFailed as another checked exception to expose metadata update errors - Metadata.UpdateFailed is explicitly thrown by every public methods of Metadata that performs an update check - Add a Plugins.PluginFailedUnchecked to propagate errors that are already logged Promote io.helidon.build.cli.codegen.Unchecked to io.helidon.build.util.Unchecke Rename util module from helidon.build.utils to io.helidon.build.util (same as the package name) Fixes #487
Make Plugins.PluginFailed a checked exception to force error handling Wrap Plugins.PluginFailed with Metadata.UpdatedFailed as another checked exception to expose metadata update errors Metadata.UpdateFailed is explicitly thrown by every public methods of Metadata that performs an update check Promote io.helidon.build.cli.codegen.Unchecked to io.helidon.build.util.Unchecke Fixed a bug in Plugin log when Plugin.execute is invoked with a null consumer for output. Fixes helidon-io#487 Fixes helidon-io#507 (forward port of helidon-io#506)
* Improve Metadata update error handling: Make Plugins.PluginFailed a checked exception to force error handling Wrap Plugins.PluginFailed with Metadata.UpdatedFailed as another checked exception to expose metadata update errors Metadata.UpdateFailed is explicitly thrown by every public methods of Metadata that performs an update check Promote io.helidon.build.cli.codegen.Unchecked to io.helidon.build.util.Unchecke Fixed a bug in Plugin log when Plugin.execute is invoked with a null consumer for output. Fixes #487 Fixes #507 (forward port of #506) * Always pass-in a string consumer for Plugins.execute
During HOL session, a user had a problem that CLI could not see version 2.3.2.
Using maven archetype worked just fine.
Worked OK from my own environment with the exact same command (my output below).
Environment:
Dev output with debug:
Dev output with debug on a working machine:
The text was updated successfully, but these errors were encountered: