Skip to content

Remove Fcitx5 autostart desktop entry#4752

Merged
dhh merged 1 commit intobasecamp:devfrom
pomartel:fcitx5-autostart
Feb 26, 2026
Merged

Remove Fcitx5 autostart desktop entry#4752
dhh merged 1 commit intobasecamp:devfrom
pomartel:fcitx5-autostart

Conversation

@pomartel
Copy link
Contributor

Turns out we are starting fcitx5 twice. This PR removes the autostart desktop entry.

Copilot AI review requested due to automatic review settings February 26, 2026 16:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR resolves a duplicate startup issue where Fcitx5 (an input method framework) was being launched twice: once through the XDG autostart mechanism via /etc/xdg/autostart/org.fcitx.Fcitx5.desktop and once explicitly through the Hyprland autostart configuration. The PR removes the XDG autostart desktop entry to prevent the duplication, keeping only the explicit Hyprland startup.

Changes:

  • Added a migration script to remove the Fcitx5 XDG autostart desktop entry for existing installations
  • Created a new configuration script that performs the removal operation
  • Integrated the removal script into the main configuration pipeline for fresh installations

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
migrations/1772120972.sh Migration script that executes the Fcitx5 autostart removal for existing systems
install/config/remove-fcitx5-autostart.sh Configuration script that removes the XDG autostart desktop entry file
install/config/all.sh Adds the removal script to the main configuration pipeline

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dhh dhh merged commit a613f4c into basecamp:dev Feb 26, 2026
3 of 4 checks passed
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.

3 participants