-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
test_cp_parents_2_dirs
fails on android
#4079
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
Comments
Looking into this. |
Looks like the problem is in attributes, if I remove this line here - https://github.com/uutils/coreutils/blob/main/src/uu/cp/src/cp.rs#L739, it passes. |
After a bit more digging, looks like |
Here is a simpler test case that you can run in an Android emulator in Termux:
This is probably because of selinux attributes, but I'm not sure. |
Something else that is interesting. With GNU cp, when we specify |
I can think of three solutions here:
What do you think? |
Interesting! I think we should match GNU behaviour as much as possible. One thing I'm curious about, is it possible too build cp with xattr support and what is the behaviour then? |
@tertsdiepraam looking at https://github.com/coreutils/coreutils/blob/master/src/copy.c#L1427-L1432:
If we select I will try building GNU cp with xattr enabled on android. |
Again, this is because of #3477. |
test introduced in:
ac3fcca
cc @jfinkels
The text was updated successfully, but these errors were encountered: