-
Notifications
You must be signed in to change notification settings - Fork 132
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
RFE: support reflinking directories #618
Labels
Comments
intelfx
added a commit
to intelfx/rmlint
that referenced
this issue
Mar 10, 2023
…tory pairs) - modify rm_util_link_type() to specifically detect directory pairs - modify sh output generator to emit cp_reflink for directory pairs - modify sh template to support reflinking directories and extend stampfile code to preserve attributes recursively Fixes sahib#618.
intelfx
added a commit
to intelfx/rmlint
that referenced
this issue
Mar 10, 2023
- modify rm_util_link_type() to specifically detect directory pairs - modify sh output generator to emit cp_reflink for directory pairs - modify sh template to support reflinking directories and extend stampfile code to preserve attributes recursively Fixes sahib#618.
intelfx
added a commit
to intelfx/rmlint
that referenced
this issue
Mar 27, 2023
- modify rm_util_link_type() to specifically detect directory pairs - modify sh output generator to emit cp_reflink for directory pairs - modify sh template to support reflinking directories and extend stampfile code to preserve attributes recursively Fixes sahib#618.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
existing behavior
Currently,
rmlint -T df -Dj -o sh:rmlint.sh -c sh:handler=reflink
will not do anything for any duplicate directories:(Note the "Unexpected return code 3", which stands for
RM_LINK_NOT_FILE
.)suggested behavior
In the shell script, reflinks are created with
cp -a --reflink=always
. There is nothing preventing the same command from being used to reflink-copy an entire directory. It might make sense to support that.The text was updated successfully, but these errors were encountered: