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

gtk3: remove check for DISPLAY variable #1379

Merged
merged 3 commits into from
Sep 12, 2020
Merged

Conversation

muelli
Copy link
Contributor

@muelli muelli commented Sep 10, 2020

It appear to work well enough without that check.
With Wayland becoming more pupolar, this check is in
the way of running applications.

Fixes: #9904

Remove this section

Please have a look at our developer documentation before submitting your Pull Request.

Please note that the trac ticket, news fragment, and review submission portions of this process apply to all pull requests, no matter how small; if you don't do them, it's likely that nobody will even notice your PR needs a review.

Contributor Checklist:

It appear to work well enough without that check.
With Wayland becoming more pupolar, this check is in
the way of running applications.

Fixes: twisted#9904
As the code does not depend on this variable any more,
testing for it does not make sense.

There might be more logic to remove, since
4142aee
introduced it.
Copy link
Contributor

@twm twm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks reasonable to me. Thank you!

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.

None yet

2 participants