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

Support building a static library #446

Closed
wants to merge 1 commit into from
Closed

Conversation

jktjkt
Copy link
Contributor

@jktjkt jktjkt commented Nov 14, 2023

This is literally the bare minimum that one has to do in order to let a properly configured CMake do its thing and produce a static library.

v2: be lazy and rely on CMake's builtin features

I'm deliberately not using CMake's standard way through the BUILD_SHARED_LIBS option because the other libraries in the NETCONF/YANG stack already stick to this custom convention.

This is literally the bare minimum that one has to do in order to let a
properly configured CMake do its thing and produce a static library.
@michalvasko
Copy link
Member

Alright, seems like the right way to support this and we should update other projects this way, too. But please fix the conflicts before merging.

@jktjkt
Copy link
Contributor Author

jktjkt commented Nov 15, 2023

this will need some time on my end because we're currently stuck with the pre-v3/v4/whatever recent push (and we're testing some server-specific mock with actual communication samples, so that might take time)

michalvasko added a commit that referenced this pull request Apr 25, 2024
michalvasko added a commit that referenced this pull request Apr 25, 2024
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.

2 participants