Page MenuHomeSolus

Inclusion of protonmail-bridge resolves T8937

Authored by FriesischScott on Jan 12 2021, 12:21 PM.
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
Apr 27 2023, 9:01 PM



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

R5138 protonmail-bridge
Lint Not Applicable
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?


builddeps should be:

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

Unnecessary empty line.


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