Page MenuHomeFreeBSD

Update finance/skrooge to 2.5.0
ClosedPublic

Authored by tcberner on Dec 15 2016, 6:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 18, 4:35 AM
Unknown Object (File)
Dec 22 2024, 9:53 PM
Unknown Object (File)
Dec 22 2024, 7:14 PM
Unknown Object (File)
Dec 10 2024, 10:05 AM
Unknown Object (File)
Dec 2 2024, 7:39 PM
Unknown Object (File)
Nov 25 2024, 10:22 AM
Unknown Object (File)
Nov 25 2024, 1:32 AM
Unknown Object (File)
Nov 24 2024, 11:42 PM
Subscribers
None

Details

Summary
  • Update skrooge to 2.5.0.
  • This also switches the port from using kdelibs4 to using kf5.
  • Add option to use encrypted database via databases/sqlcipher
Test Plan
  • A KDE4 user should probably give it a try.
  • Tested on 9.3@32 and 120@64.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 6264
Build 6507: arc lint + arc unit

Event Timeline

tcberner retitled this revision from to Update finance/skrooge to 2.5.0.
tcberner updated this object.
tcberner edited the test plan for this revision. (Show Details)
tcberner added reviewers: kde, rakuco, mat, adridg.
finance/skrooge/Makefile
43

Is there any problem with enabling sqlcipher unconditionally? From a quick look at the code (but not the documentation) it's not clear if this means Skrooge will be unable to open plan sqlite3 database files. If it is, I don't see why not do that and make Skrooge able to work with both encrypted and unencrypted databases.

finance/skrooge/Makefile
43

When I first created the port databases/sqlcipher conflicted with sqlite3 as far as I remember.

With r376125 this conflict has been removed. So I'll drop the options.

Remove sqlcipher xor sqlite3, as these no longer conflict.

I created in a skrooge-2.5.0 without sqlcipher and opened it in the one compiled
with sqlcipher. Seemed to work as far as I can tell.

rakuco edited edge metadata.

shipit!

finance/skrooge/files/patch-CMakeLists.txt
8

Can you send this patch upstream? The best would be having a proper FindSqlcipher.cmake, but I wouldn't spend too much time on it.

This revision is now accepted and ready to land.Dec 18 2016, 3:13 PM
tcberner added inline comments.
finance/skrooge/files/patch-CMakeLists.txt
8

Upstream review created for the "easy way".

This revision was automatically updated to reflect the committed changes.
tcberner marked an inline comment as done.