-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Refactor CMakeLists, targetify everything #3068
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
Conversation
|
I forgot to test packaging...will do tonight |
|
Friendly bump :) |
|
I'm lost with cmake files, I haven't got the knowledge to review this. Let's wait some days for a possible review by @MKleusberg. Otherwise, I suppose it can be merged, if nobody opposes it. |
|
How's the progress on this? |
|
Ok, nobody opposed after a long time. Thanks for your contribution. |
|
By the way, have you checked if this affect documentation? Wiki and BUILDING.md |
|
Everything should stay the same as before, I'm just rewriting some parts to utilize targets. |
|
Perfect, thanks. |
at least this isn't working anymore: https://github.com/sqlitebrowser/sqlitebrowser/wiki/Compiling-DB4S-on-Ubuntu-To-Use-A-Pre-Installed-Version-of-SQLite i am unable to compile with a current sqlite version. It stays at 3.15.2 |
|
Oh I should have noticed that. But did you install SQLite to a non-standard prefix? I'm pretty sure CMake also looks up |
|
just installed it with
(kubuntu 21.04) //running the latest .appimage it does show the included sqlite 3.38.5 in about and with |
|
Hmm okay. During configuration, there should be a "Found SQLite3: ..." somewhere in the log. Does it point to the correct library file? There might be a leftover SQLite3 installation somewhere. |
|
i was unable to find anything in the log with i added the for future reference. this was working for me: |
|
I'll add this to the wiki. Glad it helped! |
|
Been trying to get our Windows nightly builds working again today, as they broke when this was merged. 😉 The problem seems to be it's just not finding the SQLCipher directory, regardless of what value I pass it. 😦 We used to point to the SQLCipher directory using a Also setting Note that it's pointing to the
Anyone have ideas how to get this working again? |
|
As a data point, our macOS nightly builds are broken too. Not sure why, but it seems to be picking up the SQLite library included in macOS (that doesn't have Kind of need sleep atm, so I'll take a look at this again tomorrow. 😄 |
|
Hmm I'll have to look at the Makefile to see how it works. I rarely work with hand written Makefiles nowadays. For the macOS problem, did you add |
Fix Windows build after PR #3068
I've tested this with:
sqlite3,sqlcipher,qscintilla,qcustomplotsqlite3,qscintilla,qhexedit,qcustomplotAll builds checked with
lddto ensure the correct libraries are linked against.Only thing left is to test it on Windows...