Skip to content

fix: use ConditionPathExists for update service checks#14

Merged
RoyalOughtness merged 1 commit intosecureblue:mainfrom
HastD:condition-check
Feb 13, 2026
Merged

fix: use ConditionPathExists for update service checks#14
RoyalOughtness merged 1 commit intosecureblue:mainfrom
HastD:condition-check

Conversation

@HastD
Copy link
Collaborator

@HastD HastD commented Feb 11, 2026

The ConditionFileIsExecutable check doesn't seem to be working properly for some users, and ConditionPathExists should definitely work.

Also use /bin/sh instead of /usr/bin/bash in these services for portability.

Should fix secureblue/secureblue#1929.

@HastD HastD requested a review from RoyalOughtness February 11, 2026 13:25
The ConditionFileIsExecutable check doesn't seem to be working properly
for some users, and ConditionPathExists should definitely work.

Also use `/bin/sh` instead of `/usr/bin/bash` in these services for
portability.
@RoyalOughtness RoyalOughtness merged commit 8720ec9 into secureblue:main Feb 13, 2026
7 checks passed
@HastD HastD deleted the condition-check branch February 13, 2026 03:57
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.

[BUG] brew-update.service was not updating

2 participants