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

fix:Do not set GDK_BACKEND to x11 in terminals. #1911

Merged
merged 1 commit into from
Oct 1, 2021

Conversation

Davidy22
Copy link
Collaborator

@Davidy22 Davidy22 commented Sep 24, 2021

Do not set GDK_BACKEND to x11 in terminals.

Don't automatically inherit env variables and manually pass modified env

Fixes #1871

@Davidy22 Davidy22 changed the title Do not set GDK_BACKEND to x11 in terminals. fix:Do not set GDK_BACKEND to x11 in terminals. Sep 24, 2021
@Davidy22 Davidy22 force-pushed the gdk_backend branch 2 times, most recently from c469460 to 3176b24 Compare September 27, 2021 06:05
@@ -121,6 +121,10 @@ def __init__(self, guake):

self.setup_drag_and_drop()

exclude = {"GDK_BACKEND"}
Copy link
Collaborator

Choose a reason for hiding this comment

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

we may put this as a constant?

self.ENVV_EXCLUDE_LIST = ["GDK_BACKEND"]

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Alright, sure, could be useful at some point.

guake/terminal.py Outdated Show resolved Hide resolved
guake/terminal.py Show resolved Hide resolved
Copy link
Collaborator

@mlouielu mlouielu left a comment

Choose a reason for hiding this comment

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

Function test pass, LGTM.

Only a small point, please help remove duplicate commit title in commit message.

@Davidy22
Copy link
Collaborator Author

Davidy22 commented Oct 1, 2021

Done.

mlouielu
mlouielu previously approved these changes Oct 1, 2021
Copy link
Collaborator

@mlouielu mlouielu left a comment

Choose a reason for hiding this comment

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

LGTM, but I forgot one thing (I thought it is like GitLab...), please also put Fixed: #1871 in commit message!

Then it is good to merge.

Don't automatically inherit env variables and manually pass modified env

Fixes Guake#1871
@Davidy22
Copy link
Collaborator Author

Davidy22 commented Oct 1, 2021

eh alright done

Copy link
Collaborator

@mlouielu mlouielu left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

@mlouielu mlouielu merged commit 0cedfd6 into Guake:master Oct 1, 2021
@mlouielu
Copy link
Collaborator

mlouielu commented Oct 1, 2021

I just find out that merge commit will be editable after I click merge..., so basically my last request can be added in merge commit next time 💯

@Davidy22 Davidy22 deleted the gdk_backend branch October 1, 2021 12:51
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.

GDK_BACKEND is propagated to the shell in terminal
2 participants