Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1862725 - lxqt-wallet-3.2.1 is available
Summary: lxqt-wallet-3.2.1 is available
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: lxqt-wallet
Version: rawhide
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
Assignee: Raphael Groner
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1862744
TreeView+ depends on / blocked
 
Reported: 2020-08-01 21:19 UTC by Upstream Release Monitoring
Modified: 2021-02-24 17:17 UTC (History)
5 users (show)

Fixed In Version: lxqt-wallet-3.2.1-1.fc34
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-02-24 17:17:43 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Upstream Release Monitoring 2020-08-01 21:19:26 UTC
Latest upstream release: 3.2.0
Current version/release in rawhide: 3.1.0-8.fc32
URL: https://github.com/mhogomchungu/lxqt_wallet

Please consult the package updates policy before you issue an update to a stable branch: https://docs.fedoraproject.org/en-US/fesco/Updates_Policy/


More information about the service that created this bug can be found at: https://fedoraproject.org/wiki/Upstream_release_monitoring


Please keep in mind that with any upstream change, there may also be packaging changes that need to be made. Specifically, please remember that it is your responsibility to review the new version to ensure that the licensing is still correct and that no non-free or legally problematic items have been added upstream.


Based on the information from anitya: https://release-monitoring.org/project/11193/

Comment 1 Raphael Groner 2020-08-02 06:43:15 UTC
Fixes for kwallet and updates of translations. Besides commits primarily dedicated to wind*.

Comment 2 Upstream Release Monitoring 2020-08-02 10:46:22 UTC
Latest upstream release: 3.2.1
Current version/release in rawhide: 3.1.0-8.fc32
URL: https://github.com/mhogomchungu/lxqt_wallet

Please consult the package updates policy before you issue an update to a stable branch: https://docs.fedoraproject.org/en-US/fesco/Updates_Policy/


More information about the service that created this bug can be found at: https://fedoraproject.org/wiki/Upstream_release_monitoring


Please keep in mind that with any upstream change, there may also be packaging changes that need to be made. Specifically, please remember that it is your responsibility to review the new version to ensure that the licensing is still correct and that no non-free or legally problematic items have been added upstream.


Based on the information from anitya: https://release-monitoring.org/project/11193/

Comment 3 Gwyn Ciesla 2020-08-03 13:48:04 UTC
3.2.1 is fine for sirikali but zulucrypt fails to build:

/home/gwyn/fedora/git/zulucrypt/zuluCrypt-5.7.1/zuluCrypt-gui/secrets.cpp: In member function 'LXQt::Wallet::Wallet** secrets::internalWallet()':
/home/gwyn/fedora/git/zulucrypt/zuluCrypt-5.7.1/zuluCrypt-gui/secrets.cpp:53:41: error: cannot convert 'std::unique_ptr<LXQt::Wallet::Wallet>' to 'LXQt::Wallet::Wallet*' in assignment
   53 |   m_internalWallet = w::getWalletBackend( w::BackEnd::internal ) ;
      |                      ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~
      |                                         |
      |                                         std::unique_ptr<LXQt::Wallet::Wallet>
/home/gwyn/fedora/git/zulucrypt/zuluCrypt-5.7.1/zuluCrypt-gui/secrets.cpp: In member function 'secrets::wallet secrets::walletBk(LXQt::Wallet::BackEnd)':
/home/gwyn/fedora/git/zulucrypt/zuluCrypt-5.7.1/zuluCrypt-gui/secrets.cpp:67:40: error: could not convert 'LXQt::Wallet::getWalletBackend(LXQt::Wallet::BackEnd)()' from 'std::unique_ptr<LXQt::Wallet::Wallet>' to 'secrets::wallet'
   67 |   return LXQt::Wallet::getWalletBackend( e ) ;
      |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~
      |                                        |
      |                                        std::unique_ptr<LXQt::Wallet::Wallet>

Comment 4 Raphael Groner 2020-08-03 14:35:22 UTC
Thanks for debugging. zulucrypt ships alternatively an embedded version of lxqt-wallet. Will take a closer look into.

Comment 5 Raphael Groner 2020-08-03 14:47:46 UTC
Propably we'd need a new release of zulucrypt as well to continue with unbundling of lxqt-wallet. 
Taking maintainer of zulucrypt in the boat, please can you request upstream to keep compatibility with API as unbundling is explicitly featured (-DINTERNAL_LXQT_WALLET=false).

Comment 6 Gwyn Ciesla 2020-10-01 20:44:43 UTC
Any ETA on upgrading lxqt-wallet?

Comment 7 Raphael Groner 2020-10-12 00:36:28 UTC
(In reply to Gwyn Ciesla from comment #6)
> Any ETA on upgrading lxqt-wallet?

Waiting for clarification about dependencies as a response to my comment #5:
> Propably we'd need a new release of zulucrypt as well to continue with unbundling of lxqt-wallet.

Comment 8 Gwyn Ciesla 2020-10-12 19:39:13 UTC
If they don't provide said feedback and you'd like a hand, let me know.

Comment 9 Raphael Groner 2020-10-13 17:28:16 UTC
As mentioned in comment #3, there seems to be an issue with API of (dropped?) method getWalletBackend but needed for zulucrypt to be able to unbundle. Maybe we can use a downstream patch.

Comment 10 Fabian Affolter 2020-10-13 18:54:24 UTC
I'm all for unbundling but it would require that somebody is providing a patch. At the moment I'm simply trying to keep zulucrypt in Fedora.

Comment 11 Raphael Groner 2021-01-27 18:18:35 UTC
(In reply to Fabian Affolter from comment #10)
> I'm all for unbundling but it would require that somebody is providing a
> patch. At the moment I'm simply trying to keep zulucrypt in Fedora.

What about using the bundled lxqt-wallet in zulucrypt then? It's same upstream so we could assume they know what they do.

Another option is to orphan sirikali as another relevant dependency to care about. Taking that maintainer into the boat, please feel free to assign back if you think there's no issue.

Comment 12 Gwyn Ciesla 2021-01-27 19:17:55 UTC
I would prefer zulucrypt to bundle lxqt-wallet, so sirikali can stay and progress.

Comment 13 Raphael Groner 2021-01-30 19:11:45 UTC
zuluCrypt 6.0.0 available, see bug #1922201

bundled (internal) lxqt-wallet is 3 months old

Comment 14 Fabian Affolter 2021-02-24 13:50:33 UTC
zuluCrypt 6.x requires lxqt-wallet 3.2.0 at least. Thus, it looks like that the lxqt-wallet could come and wouldn't affect sirikali. It was mentioned that sirikali works with 3.2.1

Comment 15 Gwyn Ciesla 2021-02-24 14:14:49 UTC
Correct.


Note You need to log in before you can comment on or make changes to this bug.