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

Export targets for external use #11

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Blast545
Copy link

@Blast545 Blast545 commented Jul 7, 2020

Allowing to use the library in other CMake projects.

Signed-off-by: Jorge Perez <jjperez@ekumenlabs.com>
Signed-off-by: Jorge Perez <jjperez@ekumenlabs.com>
Signed-off-by: Jorge Perez <jjperez@ekumenlabs.com>
Signed-off-by: Jorge Perez <jjperez@ekumenlabs.com>
@hidmic hidmic mentioned this pull request Jul 8, 2020
@Snaipe
Copy link
Owner

Snaipe commented Jul 28, 2020

This looks fine as-is, but isn't the convention for cmake config files PascalCase?

@grasmanek94
Copy link

I think this PR would be a good place to fix #28 ?

@Blast545
Copy link
Author

Blast545 commented Nov 4, 2021

I remember I didn't push any further on this because I ended up working on something else. But as it is, I think it can be merged. @grasmanek94 Feel free to test this branch to see if it solves your issue.

but isn't the convention for cmake config files PascalCase?

I used the convention I've been using for my projects, I honestly don't know if there's a particular convention for cmake config files. @Snaipe If you give me any particular suggestions to the PR I can make amendments.

ziyao233 pushed a commit to ziyao233/Mimick that referenced this pull request Feb 9, 2024
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
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