Page MenuHomeSolus

Initial commit for protonvpn-gui, version 1.4.1
ClosedPublic

Authored by GZGavinZhao on Oct 27 2021, 12:11 AM.
Tags
None
Referenced Files
F11052868: D12108.id30917.diff
Thu, Aug 10, 10:14 PM
F11052867: D12108.id.diff
Thu, Aug 10, 10:14 PM
F11052866: D12108.id30901.diff
Thu, Aug 10, 10:14 PM
F11052865: D12108.id30900.diff
Thu, Aug 10, 10:14 PM
F11052864: D12108.id29390.diff
Thu, Aug 10, 10:14 PM
F11052863: D12108.id30921.diff
Thu, Aug 10, 10:13 PM
F11037489: D12108.diff
Wed, Aug 9, 8:42 PM
F10934203: D12108.id.diff
Jul 12 2023, 10:41 AM

Details

Summary

Resolves T9924.

Depends on D12061, D12679, D12680.

Test Plan

Connected to various locations around the world, ran speed and leak
test, and verified log generation and tray icon behavior.

Signed-off-by: Gavin Zhao <gavinzhaojw@protonmail.com>

Diff Detail

Repository
R5438 protonvpn-gui
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

JoshStrobl added a subscriber: JoshStrobl.

builddeps should be cleaned up. If it isn't needed during build time, it shouldn't be added (e.g. a bunch of rundeps not needed during build)

package.yml
14

You're already pulling in libwebkit-gtk-devel so this isn't needed.

58

Can just be install -Dm00644 protonvpn.desktop $installdir/usr/share/applications/protonvpn.desktop

62

Same here re. install

This revision now requires changes to proceed.Dec 25 2021, 10:22 AM

Understood. I'm also doing some refactoring with the Proton Python Client and the ProtonVPN Network Manager library. Before they were packaged into protonvpn-cli, but now since we have two packages using them the versioning and dependencies start to get messy, so I'll separate these two and update everything together.

Fix requested changes and refactor.

GZGavinZhao marked 3 inline comments as done.

Update MAINTAINERS file.

Girtablulu added a subscriber: Girtablulu.

last thing the rest looks good :)

package.yml
7

needs to be SPDX confrom https://spdx.org/licenses/preview/

This revision now requires changes to proceed.Jan 18 2022, 9:54 PM
This revision is now accepted and ready to land.Jan 18 2022, 10:40 PM