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

Allow to fix ownership and permission on files copied from the overla… #1200

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

ocochard
Copy link
Member

…y directory.

@ocochard
Copy link
Member Author

Related to issue #1181

@@ -58,7 +58,8 @@ Options:
the disk image and mount it to \$WRKDIR/world before
installing the contents to the image
-c overlaydir -- The content of the overlay directory will be copied into
the image
the image. Owners and permissions will be overwritten if
an overlaydir.mtree file is found
Copy link
Contributor

Choose a reason for hiding this comment

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

A bit confusing as people might think that the file needs to be strictly named 'overlaydir.mtree'

Copy link
Member Author

Choose a reason for hiding this comment

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

Could using $overlaydir.mtree enough to solve this confusion ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Or .mtree

Copy link
Member Author

Choose a reason for hiding this comment

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

but how the user will know where to put the.mtree file ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Stupid github, that was supposed to be <overlaydir>.mtree

@evadot evadot merged commit 4e7343e into freebsd:master Jan 28, 2025
0 of 6 checks passed
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.

2 participants