Page MenuHomeSolus

Inclusion of protonmail-bridge resolves T8937
ClosedPublic

Authored by FriesischScott on Jan 12 2021, 12:21 PM.
Tags
None
Referenced Files
F10806933: D10348.id24981.diff
Tue, May 30, 3:06 AM
F10806295: D10348.id24943.diff
Tue, May 30, 12:27 AM
F10805796: D10348.id24880.diff
Mon, May 29, 10:12 PM
F10805652: D10348.diff
Mon, May 29, 9:27 PM
F10804770: D10348.id25029.diff
Mon, May 29, 5:38 PM
F10708891: D10348.id.diff
Apr 27 2023, 9:01 PM
Subscribers

Details

Summary

Replaces D9816 due to lack of progress.

I have a ProtonMail subscription and would like to maintain this.

ProtonMail Bridge requires either gnome-keyring or pass but will default to pass if both are installed. Should I add a rundep on either?

Test Plan

Installed and configured the bridge. Works flawlessly with Thunderbird.

Diff Detail

Repository
R5138 protonmail-bridge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

JoshStrobl added a subscriber: JoshStrobl.

I imagine gnome-keyring is the more likely password daemon to be used on the system, so let's go with that.

Great work improving on the previous patch. Are all the missing libs intentionally omitted, no longer necessary, or some other reason?

package.yml
17

builddeps should be:

  1. pkgconfigs first, alphabetically ordered 0-9A-Za-z
  2. "explicitly named deps" alphaebetically ordered, like git and golang.
24

Unnecessary empty line.

26

Unnecessary empty line.

This revision now requires changes to proceed.Jan 15 2021, 9:04 AM

Sort builddeps and remove empty lines

Add rundep on gnome-keyring.

The github repository contains two apps. The bridge and an import-export app. The releases have since been split to contain the individual apps prefixed by br- and ie-. I assume the extra libs in D9816 are dependencies of the import-export app.

Something is missing. The build doesn't seem to pick up all necessary Qt5 dependencies. I just tried to install and run my package on a fresh install of Budgie (I was on Plasma before) and it doesn't work. I'll update this once I figured it out.

Add missing rundep on qt5-quickcontrols2

This revision is now accepted and ready to land.Jan 20 2021, 5:04 PM