Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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

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

Successfully merging this pull request may close these issues.

3 participants