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

Fix for multiple Call find_package() - fix #326 #343

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nische
Copy link

@nische nische commented Jun 5, 2024

Wrap CMSIS_LD_.... with if to check if allready exsists.

This works for me at the moment.

Wrap CMSIS_LD_.... with if to check if allready exsists
@atsju
Copy link
Collaborator

atsju commented Jun 5, 2024

Nothing obvious has been brocken according to CI tests.
I let @Hish15 review the code

@nische
Copy link
Author

nische commented Jun 7, 2024

I think i miss something. After i clear my builds i have some issues. I think i dont get the .ld files in my other buildfolders. I will do some more checks. but im not sure how this linker magic happens :-D

@Hish15
Copy link
Collaborator

Hish15 commented Sep 9, 2024

Hello @nische, what is the status on this ?

@nische
Copy link
Author

nische commented Sep 9, 2024

Hi,
I have a issue that the LD files are missing in the other "solution".
In my tests it works only because the LD files are already there from the past.
I think that's not the big deal but at the moment I don't have time to fix it. I hope I have time in a few weeks.

BR, Nico

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

Successfully merging this pull request may close these issues.

3 participants