-
Notifications
You must be signed in to change notification settings - Fork 26
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 ?] "executes:" not handled correctly #168
Comments
@saperlot do you apply Forget the request, the information is there. |
I've just tried it again within cproject.yml. But here it will fail when the batch file does not return Status OK. But it still only execute without --contex builds. |
As documented here please note The |
@saperlot I have removed the critical label for now. Given Daniel's link to the documentation, do you still believe the behavior of the tools is different from what has been documented? |
@jkrech I do not have a for-context directive in the executes config. What i mean with "But it still only execute without --contex builds." is that when i issue a build command from vsocode cmsis extensions build button, then it only builds the current active context with |
Thank for raising this issues. We improved the description https://github.com/ReinhardKeil/cmsis-toolbox/blob/main/docs/YML-Input-Format.md#executes. The We will work on better messaging for the user to indicate when execute: commands are performed. |
Can this issue be closed? |
Improved description for executes: node addresses issues: - #168 - Open-CMSIS-Pack/devtools#1043 - Open-CMSIS-Pack/devtools#1735 - Open-CMSIS-Pack/devtools#1561 Added new cpackget options Open-CMSIS-Pack/cpackget#324 Added to cbuild-idx.yml apis, component file status, messages Improve documentation for Linker Script generation
This issue Open-CMSIS-Pack/cbuild2cmake#185 will probably address your concern. |
Is there an existing issue for this?
Unusable feature
Post Build scripts are not executed corretly.
Current Behavior
For example:
Included in csolution.yml:
cbuild *** --context-set --context ***.Debug***
is issued, the script wont execute.Now if we remove the for-context in executes: config:
Now if we set the for-context to - .Release in executes: config:
--context ***-Release+***
, error appears:error csolution: context ".Debug+" referenced by access sequence 'hex' does not exist or is not selected
--> So executes is not usable, which is a neccessary feature, therefore i've set it as a blocker
Another issue is, that build script execution direction is tmp/ I would assume that base directory to be on root.
Expected Behavior
Executes handled correctly.
Steps To Reproduce
--> see current behavior
Affected tools
Affected operating systems
Environment
Windows 11
cmsis-toolbox: 2.5.0
The text was updated successfully, but these errors were encountered: