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)
Mon, Apr 15, 12:45 PM
Unknown Object (File)
Mon, Apr 15, 12:39 PM
Unknown Object (File)
Mon, Apr 15, 11:07 AM
Unknown Object (File)
Mon, Apr 8, 11:01 PM
Unknown Object (File)
Mon, Apr 8, 9:38 PM
Unknown Object (File)
Thu, Apr 4, 1:59 AM
Unknown Object (File)
Thu, Apr 4, 1:58 AM
Unknown Object (File)
Thu, Apr 4, 1:58 AM
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
Lint Not Applicable
Unit
Tests Not Applicable

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 ↗(On Diff #22959)

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 ↗(On Diff #22959)

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
7 ↗(On Diff #23002)

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
7 ↗(On Diff #23002)

Upstream review created for the "easy way".

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