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

dependencies not well detected when .mak changes #10

Open
fred-r opened this issue Dec 2, 2020 · 1 comment
Open

dependencies not well detected when .mak changes #10

fred-r opened this issue Dec 2, 2020 · 1 comment
Assignees
Labels

Comments

@fred-r
Copy link

fred-r commented Dec 2, 2020

Hi,

I am currently working on adding IAR support for my project with CMSIS Build.
To do so, I have created a "IAR.8.5.0.mak" file.
Of course, at the moment I still need to tune this file.

Unfortunately, when I do changes in the .mak file the build system does not detect it and keeps on working with the very same ".o._cc" files.

To have my updates taken onto account I need to delete the RTE folder manually.

This is a bit similar to the issue I faced with pack updates.
Probably there is a limitation with the dependencies checking ?

Thanks & Regards,
Fred

@brondani brondani self-assigned this Dec 3, 2020
@brondani brondani added the Build label Dec 3, 2020
@brondani
Copy link
Collaborator

brondani commented Dec 3, 2020

Hi Fred,

Thanks for reporting it, as you noted toolchain config files are not set as dependencies, I see that would be helpful in your use case.
For the time being, just for your development purposes, you could add it into the options files recipes in the generated makefile, for example:
$(CC_OPT_FILES): $(MAKEFILE) C:/cbuild/etc/IAR.8.5.0.mak

In such case changes in the C:/cbuild/etc/IAR.8.5.0.mak file will trigger a new build when you call make again. But it's just a temporary solution that will be overwritten once you regenerate the makefile.

Edit: @jkrech has just suggested me you can use cbuild.sh with the rebuild option. In any case there is no need to remove the RTE folder.

Best regards
Daniel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants