Skip to content

[3.7] bpo-40767: Allow pure Wayland to get default XDG web browser (GH-20382) #20609

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

Merged
merged 1 commit into from
Jun 3, 2020

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jun 3, 2020

Would be nice to backport to python 3.7+. I don't think it's worth the hassle to backport this all the way down to 3.10. But I'll let the maintainers decide.

This is hard to test because the test setup already includes this environment variable

Let me know if something doesn't match the PR guidelines. This is my first PR in the python source code.
(cherry picked from commit c822efe)

Co-authored-by: Jeremy Attali [email protected]

https://bugs.python.org/issue40767

Automerge-Triggered-By: @orsenthil

…H-20382)

Would be nice to backport to python 3.7+. I don't think it's worth the hassle to backport this all the way down to 3.10. But I'll let the maintainers decide.

This is hard to test because the test setup already includes this [environment variable](https://github.com/python/cpython/blob/master/Lib/test/pythoninfo.pyGH-L292)

Let me know if something doesn't match the PR guidelines. This is my first PR in the python source code.
(cherry picked from commit c822efe)

Co-authored-by: Jeremy Attali <[email protected]>
@miss-islington
Copy link
Contributor Author

@jtheoof: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 911c35d into python:3.7 Jun 3, 2020
@miss-islington miss-islington deleted the backport-c822efe-3.7 branch June 3, 2020 13:01
@miss-islington
Copy link
Contributor Author

@jtheoof: Status check is done, and it's a success ✅ .

@jtheoof
Copy link
Contributor

jtheoof commented Jun 3, 2020

Thanks @miss-islington for the merge and the backports. ❤️

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.

5 participants