Skip to content

Conversation

@SoumyaRanjanPatnaik
Copy link
Contributor

In order to make sure updating to the latest regolith-session package removes the deprecated regolith-sway-audio-idle-inhibit, added Breaks and Replaces directives. Also specified the minimum version of sway-audio-idle-inhibit package required to ensure the systemd service is available.

Copy link
Member

@kgilmer kgilmer left a comment

Choose a reason for hiding this comment

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

LGTM

wl-clipboard,
xwayland
Recommends:
sway-audio-idle-inhibit,
Copy link
Member

Choose a reason for hiding this comment

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

So we need a changelog version sync to https://github.com/regolith-linux/SwayAudioIdleInhibit/blob/main/debian/changelog#L1. Not clear to me if we are using a gh action to bump changelog or manually pushing commits. Will check

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Even if we do have a action for that, even then I think the bumping needs to happen manually to ensure the major version gets updated, since this is technically a breaking change. Also the upstream version is also 2.0.0, so having 2.x as the version number should make a lot of sense.

Copy link
Member

Choose a reason for hiding this comment

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

Assuming no additional actions required here. Merging PR so I can make a release

@kgilmer kgilmer merged commit f0616c8 into main Nov 23, 2025
4 checks passed
@kgilmer kgilmer deleted the chore/break-replace-regolith-sway-audio-idle-inhibit branch November 23, 2025 19:09
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