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

Changes to cbuild-pack.yml should trigger cbuild setup #183

Open
ReinhardKeil opened this issue Jan 30, 2025 · 5 comments
Open

Changes to cbuild-pack.yml should trigger cbuild setup #183

ReinhardKeil opened this issue Jan 30, 2025 · 5 comments
Labels
enhancement New feature or request

Comments

@ReinhardKeil
Copy link
Collaborator

To update packs, an effective method is to change the cbuild-pack.yml file.

A change to this file should trigger a cbuild setup to reflect new pack versions.

@ReinhardKeil ReinhardKeil added the enhancement New feature or request label Jan 30, 2025
@jkrech jkrech moved this to Backlog in CMSIS Solution Extension Jan 30, 2025
@jkrech
Copy link
Collaborator

jkrech commented Jan 31, 2025

Since this file is also updated by the tools if a pack changes in other solution input files we need to make sure that we don't run the cbuild setup unnecessarily for a second time.

@JonatanAntoni
Copy link
Member

How can we know? Shall we have a grace period if the file is changed while cbuild is active?

@ReinhardKeil
Copy link
Collaborator Author

I suggest that during execute of cbuild this file watches are disabled.

@jkrech
Copy link
Collaborator

jkrech commented Feb 6, 2025

The safest way would be a dedicated editor view for *.cbuild-pack.yml with a safe button that would trigger a cbuild setup instead of a file watch going forward.

@ReinhardKeil
Copy link
Collaborator Author

Agreed, let's park this ticket for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Backlog
Development

No branches or pull requests

3 participants