Skip to content
This repository has been archived by the owner on Jan 17, 2024. It is now read-only.

Add libnotify support, update (some) dependencies #50

Merged
merged 2 commits into from Jun 27, 2022
Merged

Add libnotify support, update (some) dependencies #50

merged 2 commits into from Jun 27, 2022

Conversation

ingemarberg
Copy link
Contributor

This pull request adds libnotify support, which should close #42. I've built and tested it locally and it seems to work as expected. It also updates some dependencies. It seems as requests most be updated before charset-normalizer can be bumped to version 2.1.0, please see psf/requests#6169. I've not been able to build the newest version of PyQt5 locally. It seems as the Flathub builder has the same problem.

@flathubbot
Copy link
Contributor

Started test build 97791

@flathubbot
Copy link
Contributor

Build 97791 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/95521/com.github.micahflee.torbrowser-launcher.flatpakref

@JakobDev
Copy link
Contributor

I've not been able to build the newest version of PyQt5 locally.

You don't need to build PyQt5. Just use com.riverbankcomputing.PyQt.BaseApp.

@AsciiWolf
Copy link
Collaborator

It looks like it built fine. Thanks a lot! :-) It was in my TODO, but I sadly do not have much free time lately. I will keep the PyQt5 module for now, we can switch to the PyQt.BaseApp later if desired.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Native notification doesn't work
4 participants