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

Spectre CommandRuntimeException #34

Closed
itzexor opened this issue Jan 15, 2023 · 4 comments
Closed

Spectre CommandRuntimeException #34

itzexor opened this issue Jan 15, 2023 · 4 comments

Comments

@itzexor
Copy link

itzexor commented Jan 15, 2023

Hello, I've just gotten the new release to test the fixed issues, but I seem to be running into a new one...

$ tone-0.1.3-linux-x64/tone 
Spectre.Console.Cli.CommandRuntimeException: Could not get settings type for command of type 'tone.Commands.DumpCommand'.
  at ConfiguredCommand Spectre.Console.Cli.ConfiguredCommand.FromType<TCommand>(string , bool )
  at ICommandConfigurator Spectre.Console.Cli.Configurator.AddCommand<TCommand>(string )       
  at void Program.<>c__DisplayClass0_0.<<Main>$>b__8(IConfigurator )                           
  at async Task<int> Program.<Main>$(string[] )

It's happening both on my linux desktop (above) and the musl version within docker (below)

# tone
Spectre.Console.Cli.CommandRuntimeException: Could not get settings type for command of type 'tone.Commands.DumpCommand'.
  at ConfiguredCommand Spectre.Console.Cli.ConfiguredCommand.FromType<TCommand>(string , bool )
  at ICommandConfigurator Spectre.Console.Cli.Configurator.AddCommand<TCommand>(string )       
  at void Program.<>c__DisplayClass0_0.<<Main>$>b__8(IConfigurator )                           
  at async Task<int> Program.<Main>$(string[] )

Is there something I've done wrong with the new version? 0.1.2 works as expected thru direct extraction & execution.

Thanks in advance

@sandreas
Copy link
Owner

Mmh this is strange, but thank you for reporting. I'll check the release version, maybe it has to do something with the net7.0 switch I made.

@sandreas
Copy link
Owner

Confirmed. Switched back to SDK 6.0 and re-released, now it seems to work again. Could you please check it is working for you?

@itzexor
Copy link
Author

itzexor commented Jan 15, 2023

Yes both versions, musl and not, work properly now :) 👍

@itzexor itzexor closed this as completed Jan 15, 2023
@sandreas
Copy link
Owner

Yes both versions, musl and not, work properly now :) +1

Thanks for testing. Seems that net7.0 is just not yet ready for action... this is something I would have never expected.

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

2 participants