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

Fix little incompatibility with busybox #188

Closed
tlaurion opened this issue May 2, 2024 · 1 comment
Closed

Fix little incompatibility with busybox #188

tlaurion opened this issue May 2, 2024 · 1 comment

Comments

@tlaurion
Copy link
Contributor

tlaurion commented May 2, 2024

--- /home/user/wyng.orig 2024-05-01 21:07:11.759288927 -0400
+++ /sbin/wyng 2024-05-01 21:07:54.966291243 -0400
@@ -2576,7 +2576,7 @@
     return

 do_exec([[CP.tar,"-cf","-","--no-recursion","--verbatim-files-from","--files-from", "-"],
-         dest.run_args([dest.cd + "  && tar --no-same-owner -xf -"]
+         dest.run_args([dest.cd + "  && tar -o -xf -"]
                             + [" && mv '"+x+ext+"' '"+x+"'" for x in update_list if ext])
         ], inlines=[x+ext for x in update_list], cwd=lcd)

on busybox, there is no --no-same-owner , but there is -o on both:

man tar:

-o When creating, same as --old-archive. When extracting, same as --no-same-owner.

tar --hel on busybox:

-o Don't restore user:group

@tasket
Copy link
Owner

tasket commented May 2, 2024

OK, please submit as PR in the 08wip branch.

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

No branches or pull requests

2 participants