Skip to content

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

Merged
mlouielu merged 1 commit into
Guake:masterfrom
Davidy22:gdk_backend
Oct 1, 2021
Merged

fix:Do not set GDK_BACKEND to x11 in terminals.#1911
mlouielu merged 1 commit into
Guake:masterfrom
Davidy22:gdk_backend

Conversation

@Davidy22

@Davidy22 Davidy22 commented Sep 24, 2021

Copy link
Copy Markdown
Collaborator

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
Comment thread guake/terminal.py Outdated

self.setup_drag_and_drop()

exclude = {"GDK_BACKEND"}

Copy link
Copy Markdown
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
Copy Markdown
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.

Comment thread guake/terminal.py Outdated
Comment thread guake/terminal.py

@mlouielu mlouielu left a comment

Copy link
Copy Markdown
Collaborator

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

Davidy22 commented Oct 1, 2021

Copy link
Copy Markdown
Collaborator Author

Done.

mlouielu
mlouielu previously approved these changes Oct 1, 2021

@mlouielu mlouielu left a comment

Copy link
Copy Markdown
Collaborator

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

Davidy22 commented Oct 1, 2021

Copy link
Copy Markdown
Collaborator Author

eh alright done

@mlouielu mlouielu left a comment

Copy link
Copy Markdown
Collaborator

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

mlouielu commented Oct 1, 2021

Copy link
Copy Markdown
Collaborator

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