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

dissect: Use COPY_MERGE #35956

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

Conversation

DaanDeMeyer
Copy link
Contributor

When copying a directory from or to an image, let's always merge with existing directories instead of failing with "File Exists".

Fixes systemd/mkosi#3342.

When copying a directory from or to an image, let's always merge
with existing directories instead of failing with "File Exists".

Fixes systemd/mkosi#3342.
@github-actions github-actions bot added dissect please-review PR is ready for (re-)review by a maintainer labels Jan 10, 2025
@bluca bluca added good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed and removed please-review PR is ready for (re-)review by a maintainer labels Jan 10, 2025
@septatrix
Copy link
Contributor

Maybe add a small note to the man page?

- If the source path in the image refers to a directory, it is copied to the destination path, recursively with all containing files and directories.
+ If the source path in the image refers to a directory, it is copied to the destination path, recursively with all containing files and directories, merging into existing directories and updating already existing files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dissect good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed
Development

Successfully merging this pull request may close these issues.

If the main image includes multiple base images, a common file in base images causes build failure
3 participants