Skip to content

Conversation

@wjt
Copy link
Contributor

@wjt wjt commented Oct 21, 2025

Since godotengine/godot#108704, the sentence about Flatpak is fully obsolete:

  1. All standard Flatpak runtimes contain libdbus;
  2. With the addition of support for the XDG Inhibit portal, Godot is able to inhibit idle without any sandboxing adjustments.

More generally, I don't think we need to mention that D-Bus is required to inhibit idle, even outside Flatpak: practically speaking, all desktop Linux systems have had libdbus available for 20+ years.

Remove the whole paragraph.

Since godotengine/godot#108704, the sentence
about Flatpak is fully obsolete:

1. All standard Flatpak runtimes contain libdbus;
2. With the addition of support for the XDG Inhibit portal, Godot is
   able to inhibit idle without any sandboxing adjustments.

More generally, I don't think we need to mention that D-Bus is required
to inhibit idle, even outside Flatpak: practically speaking, all desktop
Linux systems have had libdbus available for 20+ years.

Remove the whole paragraph.
@AThousandShips AThousandShips requested a review from a team October 21, 2025 09:25
@AThousandShips AThousandShips marked this pull request as ready for review October 21, 2025 09:25
@AThousandShips AThousandShips marked this pull request as draft October 21, 2025 09:25
@Calinou Calinou added enhancement area:manual Issues and PRs related to the Manual/Tutorials section of the documentation waiting on pr merge PRs that can't be merged until an engine PR is merged first labels Oct 21, 2025
@skyace65 skyace65 removed the waiting on pr merge PRs that can't be merged until an engine PR is merged first label Nov 28, 2025
@skyace65 skyace65 added this to the 4.6 milestone Nov 28, 2025
@skyace65 skyace65 marked this pull request as ready for review November 28, 2025 03:19
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Headless distros/containers won't have dbus installed, but they don't display a window in the first place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:manual Issues and PRs related to the Manual/Tutorials section of the documentation enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants