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

CMake install target not generating cmake config #394

Open
DraconPern opened this issue Sep 18, 2015 · 6 comments · May be fixed by #1118
Open

CMake install target not generating cmake config #394

DraconPern opened this issue Sep 18, 2015 · 6 comments · May be fixed by #1118
Labels

Comments

@DraconPern
Copy link
Contributor

there's no way to include soci in another cmake project because the sociconfig.cmake file is missing from the installed directory. Attempt to use FindSoci.cmake doesn't work either because the lib is decorated e.g. libsoci_core_3_2.lib

@DraconPern
Copy link
Contributor Author

Basically, it doesn't work if one uses find_package in config mode nor module mode.

@vadz vadz added the CMake label Sep 18, 2015
@mloskot mloskot added this to the 4.0.0 milestone Sep 18, 2015
@mloskot
Copy link
Contributor

mloskot commented Sep 18, 2015

@DraconPern TBH, I have not managed to wrap my head around the CMake project.config magic. Last time I tried I gave up on unclear documentation and lack of complete sample for a project that is more complex than basic 'hello world'.

CMake-experienced contributors strongly welcome :)

@DraconPern
Copy link
Contributor Author

i'll work on some modifications to get it building on appveyor on windows. I noticed that it's got switches for both shared and static build. I think they should be mutually exclusive options? boost does it that way, and most of the projects I have worked on also do the same.

@snikulov
Copy link
Member

@DraconPern I prefer it should be exclusive and controlled by standard variable BUILD_SHARED_LIBS as pointed out here http://www.slideshare.net/DanielPfeifer1/cmake-48475415 slide 28
@mloskot What do you think?

@mloskot
Copy link
Contributor

mloskot commented Sep 23, 2015

@snikulov I agree. It was first time I read through Daniel Pfeifer's presentation. It's great resource, I wish I knew it before I created CMake setup for SOCI :)

@Krzmbrzl
Copy link
Contributor

Krzmbrzl commented Feb 2, 2025

This should be resolved by #1118

@Krzmbrzl Krzmbrzl linked a pull request Feb 2, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants