Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: correct the filename for ConditionPathExists on ucore-signed-autorebase.service #226

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

surdy
Copy link

@surdy surdy commented Dec 25, 2024

Minor fix on the example autorebase butane file. I think that

ConditionPathExists=!/etc/ucore-autorebase/verified

was intended to be

ConditionPathExists=!/etc/ucore-autorebase/signed

Not a big deal since it won't impact the end result as /etc/ucore-autorebase/verified does not exist as well. But /etc/ucore-autorebase/verified would be "more correct" since that is the file being created on successful run.

I noticed this as I took the example and modified it for my custom ucore image and accidentally introduced a bug in my butane file as a result.

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Dec 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant