Skip to content

Conversation

Yanpas
Copy link
Contributor

@Yanpas Yanpas commented Dec 28, 2016

Now there is no need to link each target ot both libs

Now there is no need to link each target ot both libs
@coveralls
Copy link

Coverage Status

Coverage remained the same at 96.231% when pulling 870bb99 on Yanpas:cmake_link into cb44cca on SRombauts:master.

@SRombauts SRombauts self-assigned this Apr 12, 2017
@SRombauts SRombauts merged commit b830e64 into SRombauts:master Apr 12, 2017
@SRombauts
Copy link
Owner

Hi @Yanpas,

Sorry for the very long reaction time, I had to look to documentation, and I added a comment in the CMakeLists.txt file.

It should be easier to use the wrapper now, thanks a lot!

Cheers

@SRombauts
Copy link
Owner

Hi @Yanpas, sorry but I'll have to revert this since it is interacting badly with the new "install step" from #118

SRombauts added a commit that referenced this pull request Apr 12, 2017
@timrae timrae mentioned this pull request Apr 13, 2017
@timrae
Copy link
Contributor

timrae commented Apr 13, 2017

Probably you need to do this inside the if (SQLITECPP_INTERNAL_SQLITE) block... I haven't tested it yet though.

Also pinging @4brunu

@4brunu
Copy link
Contributor

4brunu commented Apr 13, 2017

@tdavis tdavis mentioned this pull request May 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants