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

[Bug]: csolution list configs command crashes under macOS and Windows on Arm64 #188

Open
4 of 13 tasks
KeilChris opened this issue Oct 29, 2024 · 2 comments
Open
4 of 13 tasks
Assignees
Labels
bug Something isn't working

Comments

@KeilChris
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Please check the attached project. I opened it in VS Code on macOS and Windows on Arm64. I did a
cbuild setup MyName.csolution.yml --context-set --context MyName.Debug+A31G112 --packs --update-rte
and afterwards a
csolution list configs
The tool cashes on both operating systems.
csolutionCrash.zip

Expected Behavior

It should list something and not crash.

Steps To Reproduce

See above

Affected tools

  • cbuild
  • cbuild2cmake
  • cpackget
  • csolution
  • packchk
  • svdconv

Affected operating systems

  • Windows amd64
  • Windows arm64
  • Linux amd64
  • Linux arm64
  • MacOS amd64
  • MacOS arm64

Environment

macOS 15.0.1 and Windows 11 in a Parallels VM on macOS.

@KeilChris KeilChris added the bug Something isn't working label Oct 29, 2024
@jkrech jkrech moved this to Backlog in CMSIS-Toolbox 2.7.0 Oct 29, 2024
@jkrech
Copy link
Member

jkrech commented Oct 29, 2024

Just for the sake of clarity:
You are neither specifying the csolution.yml file nor a context or context-set for listing the configs:
I had expected something like:

  csolution list configs MyName.csolution.yml [--context-set]

Would that command "crash" for you as well?

On windows I get:

csolutionCrash>csolution list configs

csolutionCrash>echo %ERRORLEVEL%
-1073741819

which does not right.

However when I run:

csolutionCrash>csolution list configs MyName.csolution.yml -S
csolutionCrash/MyName.cbuild-set.yml - info csolution: file is already up-to-date
../RTE/Device/A31G112/[email protected] (up to date) from ABOV::Device:[email protected]
../RTE/Device/A31G112/[email protected] (up to date) from ABOV::Device:[email protected]
../RTE/Device/A31G112/[email protected] (up to date) from ABOV::Device:[email protected]
../RTE/Device/A31G112/[email protected] (up to date) from ABOV::Device:[email protected]
../RTE/Device/A31G112/[email protected] (up to date) from ABOV::Device:[email protected]

In my opinion the command line parser must throw an error if no *.csolution.yml is specified and the usage/help must make *.csolution.yml mandatory.

@KeilChris
Copy link
Contributor Author

I tried your proposal, and it works. I was just using the "short form" as I read it in the documentation (I will fix that). Still, as you say, it should throw an error and not crash silently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Backlog
Development

No branches or pull requests

3 participants