Skip to content

Conversation

@jkbdco
Copy link

@jkbdco jkbdco commented Dec 25, 2025

Discovered this when trying to update my machine via ssh.

The omarchy-update-without-idle script fails when HYPRLAND_INSTANCE_SIGNATURE is not set (e.g., when running updates via SSH, TTY, or a non-Hyprland session).

Even though the hyprctl output is suppressed, set -e in the parent script
catches the non-zero exit code and triggers the error trap, causing the
update to fail with no visible error message.

Adding || true allows the update to proceed gracefully when the idle
inhibitor can't be set.

@dhh
Copy link
Member

dhh commented Dec 26, 2025

Target dev branch and rebase 👍

@jkbdco
Copy link
Author

jkbdco commented Dec 27, 2025

Target dev branch and rebase 👍

Hey @dhh — when I rebased onto dev, I hit a conflict. Looks like e44e937 already addressed this by moving idle management elsewhere and making this script a no-op. Should I still proceed, or is this PR no longer needed?

@sgruendel
Copy link
Contributor

Target dev branch and rebase 👍

Hey @dhh — when I rebased onto dev, I hit a conflict. Looks like e44e937 already addressed this by moving idle management elsewhere and making this script a no-op. Should I still proceed, or is this PR no longer needed?

Looks like you should now apply your changes to omarchy-update-perform directly.

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