Add gpgmepp
AbandonedPublic

Authored by wojnilowicz on May 9 2018, 9:55 AM.

Details

Reviewers
vonreth

Diff Detail

Repository
R877 Craft Blueprints for KDE
Lint
Lint Skipped
Unit
Unit Tests Skipped
wojnilowicz requested review of this revision.May 9 2018, 9:55 AM
wojnilowicz created this revision.
This revision now requires changes to proceed.May 9 2018, 12:46 PM

Cool, but what about MSVC? On Linux I have two packages gpgmepp and KF5-gpgmepp. For KMyMoney, the first one is searched and the second one is a fallback. This patch adds KF5-gpgmepp, so it should work on MSVC as well. Is it OK for you?

gpgmepp was once a KDE project but it was merged into gpgme.
So for now there is no way to get gpgme besides providing a cmake patch to gpgmepp

(the KDE version is dead)

(the KDE version is dead)

It's dead but it is better than nothing. Modern Linux distros provide KF5-gpgmepp and I can build it against KMyMoney, so maybe we should allow it as intermittent solution for MSVC.

As far as I can remember, we had several issues with the old gpgmepp

As far as I can remember, we had several issues with the old gpgmepp

Yes, but we don't loose nothing by trying. In case, it wouldn't work, we can remove it. Is it ok to commit it into that place?
As I can see, your gpgmepp has issues as well :)

  1. What about MinGW build? It fails constantly on your gpgmepp. Do I have to provide a option to make it compile right?
  1. What about this review? Are you OK with me commiting it to the location I suggested?

For the error you get in kmymoney.
We need to patch the cmake config to be relocatable and to not include msys install paths, if it gets to complicated a postInstall rutine to patch the file would be ok.

For this patch, it doesn't even build with msvc so I don't accept the patch.

wojnilowicz abandoned this revision.May 15 2018, 5:00 PM

It seems, that it has never been designed to work on MSVC. It fails on inclusion of sys/time.h.