Skip to content

cp: drop unnecessary fn - #16469

Open
oech3 wants to merge 1 commit into
uutils:mainfrom
oech3:cp-symlink
Open

oech3 wants to merge 1 commit into
uutils:mainfrom
oech3:cp-symlink

Conversation

@oech3

@oech3 oech3 commented Nov 14, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@oech3
oech3 force-pushed the cp-symlink branch 2 times, most recently from 3c7eb66 to 35191d7 Compare November 14, 2026 18:25
@sylvestre

Copy link
Copy Markdown
Contributor

thanks for tackling this!
This breaks the build on macOS and the BSDs: rustix is only a cp dependency on linux/android/wasi. Please see the inline comment.

Comment thread src/uu/cp/src/cp.rs
CpError::IoErrContext(
e,
translate!("cp-error-cannot-create-symlink",
#[cfg(not(windows))]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

rustix is only a dependency on linux/android/wasi, this won't build on macos or the BSDs.
please keep std::os::unix::fs::symlink for unix and use rustix only under target_os = "wasi"

This branch has not been deployed

No deployments
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