Skip to content

core/desktopentry: add support for xdg-terminal-exec#638

Open
RGBCube wants to merge 3 commits intoquickshell-mirror:masterfrom
RGBCube:xdg-terminal-exec
Open

core/desktopentry: add support for xdg-terminal-exec#638
RGBCube wants to merge 3 commits intoquickshell-mirror:masterfrom
RGBCube:xdg-terminal-exec

Conversation

@RGBCube
Copy link

@RGBCube RGBCube commented Mar 17, 2026

Adds support for executing Terminal=true desktop entries with DesktopEntry.execute() by reading xdg-terminals.list and scanning for terminals with TerminalEmulator in their categories and valid [X-]TerminalArg* attributes set.

Also adds support for OnlyShowIn, NotShowIn, TryExec to match xdg-terminal-exec's total behaviour.

Another thing fixed in this PR is the usage of XDG_DATA_DIR, the spec says that an empty value should still default to the usr paths. This makes the code respect that.

@RGBCube RGBCube force-pushed the xdg-terminal-exec branch from f3f49f2 to 6095d01 Compare March 17, 2026 01:21
@RGBCube RGBCube changed the title Add support for xdg-terminal-exec core/desktopentry: add support for xdg-terminal-exec Mar 17, 2026
@outfoxxed outfoxxed force-pushed the master branch 4 times, most recently from ca90821 to 3cf65af Compare March 17, 2026 17:10
@RGBCube RGBCube force-pushed the xdg-terminal-exec branch 11 times, most recently from 9fd782a to b9680c5 Compare March 18, 2026 15:54
@RGBCube RGBCube force-pushed the xdg-terminal-exec branch from b9680c5 to 7e42060 Compare March 18, 2026 15:55
RGBCube added 2 commits March 18, 2026 21:54
previously, we defaulted to /usr/local/share:/usr/share if the key was not set
the standard tells you do default if the key is not set _or is empty_

this fixes that
@RGBCube RGBCube force-pushed the xdg-terminal-exec branch from 7e42060 to 9968672 Compare March 18, 2026 18:54
@RGBCube RGBCube marked this pull request as ready for review March 18, 2026 22:08
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.

1 participant