Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement QtSingleApplication #243

Merged
merged 3 commits into from
Apr 7, 2019

Conversation

Hofer-Julian
Copy link
Collaborator

This implements similar functionalites as QtSingleApplication (adapted from https://stackoverflow.com/a/12712362/11038610) which unfortunately is not available for Python.

It solves part of #132 and enables work on #233.

In theory it should work on all Linux distributions, macOS and Windows.

@Hofer-Julian
Copy link
Collaborator Author

@m3nu if this works on macOS it should be okay to merge this pull request :)

@m3nu
Copy link
Contributor

m3nu commented Apr 7, 2019

Fully agree. I just take some more time testing new PRs after the flood of hotfixes I had to do recently.

@Hofer-Julian
Copy link
Collaborator Author

Completely understandable. Can I do more to help, apart from have it running on my machine?

(I am also currently looking into writing tests for the other functions I implemented)

@m3nu m3nu force-pushed the Implement-QtSingleApplication branch from 5e1ee46 to 29e18f9 Compare April 7, 2019 07:22
@m3nu
Copy link
Contributor

m3nu commented Apr 7, 2019

I can confirm that your change works as promised on macOS and Ubuntu 18.04.

Also thanks for the offer. Since you're already maintainer, you can just jump on anything you see. Like better tests. Some issues are just dependent on the environment and hard to test.

@m3nu m3nu merged commit ab04d84 into borgbase:master Apr 7, 2019
@Hofer-Julian
Copy link
Collaborator Author

Some issues are just dependent on the environment and hard to test.

Some issues seemed to be KDE specific. I also noticed that vorta sometimes responds differently under Wayland than it does under X. Is it be possible to set up two Linux environments under Travis? Like Kubuntu 18.04 with KDE and X and Fedora 29 with GNOME and Wayland?

@Hofer-Julian Hofer-Julian deleted the Implement-QtSingleApplication branch April 7, 2019 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants