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.

Sign up for GitHub

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

Jump to bottom

Return process from run_as_root. #336

Merged
merged 2 commits into from
Nov 27, 2023
Merged

Return process from run_as_root. #336

merged 2 commits into from
Nov 27, 2023

Conversation

rick-masters
Copy link
Contributor

Fixes #335

@nanonyme
Copy link
Contributor

Oops, good catch. I wonder why CI didn't fail....

nanonyme
nanonyme reviewed Nov 27, 2023
View reviewed changes
lib/utils.py Outdated Show resolved Hide resolved
@rick-masters
Copy link
Contributor Author

Oops, good catch. I wonder why CI didn't fail....

Unfortunately CI does not test with chroot option.

@fosslinux fosslinux merged commit 1aa0b21 into fosslinux:master Nov 27, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nanonyme nanonyme nanonyme left review comments

@fosslinux fosslinux fosslinux approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

chroot broken by new run_as_root helper
3 participants
@rick-masters @nanonyme @fosslinux