jonathanpeppers / boots

boots is a .NET global tool for "bootstrapping" vsix & pkg files. Just "boots https://url/to/your/package"!
MIT License
86 stars 9 forks source link

Bad error message on Linux #89

Open jonathanpeppers opened 3 years ago

jonathanpeppers commented 3 years ago

Running on Linux:

$ boots --preview Mono
* Automatic URL resolving is a new feature. File issues at: https://github.com/jonathanpeppers/boots/issues
Unhandled exception: System.NotImplementedException: Value for product not implemented on Windows: Mono
   at Boots.Core.WindowsUrlResolver.GetProductId(Product product) in /Users/runner/work/1/s/Boots.Core/WindowsUrlResolver.cs:line 97
   at Boots.Core.WindowsUrlResolver.Resolve(ReleaseChannel channel, Product product, CancellationToken token) in /Users/runner/work/1/s/Boots.Core/WindowsUrlResolver.cs:line 23
   at Boots.Core.Bootstrapper.Install(CancellationToken token) in /Users/runner/work/1/s/Boots.Core/Bootstrapper.cs:line 36
   at Boots.Program.Run(String url, String stable, String preview, Nullable`1 fileType) in /Users/runner/work/1/s/Boots/Program.cs:line 86
   at System.CommandLine.Invocation.CommandHandler.GetResultCodeAsync(Object value, InvocationContext context)
   at System.CommandLine.Invocation.ModelBindingCommandHandler.InvokeAsync(InvocationContext context)
   at System.CommandLine.Invocation.InvocationPipeline.<>c__DisplayClass4_0.<<BuildInvocationChain>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<UseParseErrorReporting>b__21_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass16_0.<<UseHelp>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass25_0.<<UseVersionOption>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass23_0.<<UseTypoCorrections>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<UseSuggestDirective>b__22_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<UseParseDirective>b__20_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<UseDebugDirective>b__11_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<RegisterWithDotnetSuggest>b__10_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass14_0.<<UseExceptionHandler>b__0>d.MoveNext()

It's definitely not Windows, so some logic is wrong here.