Skip to content

system.conf: Remove absolute keyring path#187

Merged
leon-anavi merged 1 commit into
rauc:masterfrom
tchx84:fix-keyring-path
Jun 9, 2026
Merged

system.conf: Remove absolute keyring path#187
leon-anavi merged 1 commit into
rauc:masterfrom
tchx84:fix-keyring-path

Conversation

@tchx84

@tchx84 tchx84 commented May 13, 2026

Copy link
Copy Markdown
Contributor

This has been updated in meta-rauc to prepare it for hermetic /usr/ [1].

[1] rauc/meta-rauc@01f7e1b

@tchx84
tchx84 force-pushed the fix-keyring-path branch from 8013e34 to 4adb25d Compare May 13, 2026 15:42
@tchx84 tchx84 changed the title meta-rauc-raspberrypi: Update keyring path to /usr/lib system.conf: Update keyring path to /usr/lib May 13, 2026
@tchx84

tchx84 commented May 13, 2026

Copy link
Copy Markdown
Contributor Author

Perhaps would be better if it doesn't specify the path, so it would work with both /etc and /usr/lib?

@ejoerns

ejoerns commented May 27, 2026

Copy link
Copy Markdown
Member

Perhaps would be better if it doesn't specify the path, so it would work with both /etc and /usr/lib?

Of course this depends on the setup and if you install the keyring to a non-default location, but I'd agree that for this case (where both the system.conf and the keyring are expected to be in the same directory), omitting the absolute paths might be better, yes.

This has been updated in meta-rauc to prepare it for hermetic /usr/ [1] but,
instead of updating the current absolute path to  /usr/lib, it is better to
remove the absolute path so it works for both /etc and /usr/lib.

[1] rauc/meta-rauc@01f7e1b

Signed-off-by: Martin Abente Lahaye <mabente@igalia.com>
@tchx84
tchx84 force-pushed the fix-keyring-path branch from 4adb25d to 51ddc70 Compare May 29, 2026 13:05
@tchx84

tchx84 commented May 31, 2026

Copy link
Copy Markdown
Contributor Author

Perhaps would be better if it doesn't specify the path, so it would work with both /etc and /usr/lib?

Of course this depends on the setup and if you install the keyring to a non-default location, but I'd agree that for this case (where both the system.conf and the keyring are expected to be in the same directory), omitting the absolute paths might be better, yes.

@ejoerns done !

@tchx84 tchx84 changed the title system.conf: Update keyring path to /usr/lib system.conf: Remove absolute keyring path May 31, 2026
@leon-anavi
leon-anavi merged commit e3cd8b9 into rauc:master Jun 9, 2026
2 of 4 checks passed
@leon-anavi

Copy link
Copy Markdown
Collaborator

@tchx84 thank you for your contribution! I've merged the GitHub pull request.

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