Skip to content

tests: fix xattr build errors - #14738

Merged
sylvestre merged 1 commit into
uutils:mainfrom
xtqqczze:xattr-cfg
Sep 27, 2026
Merged

sylvestre merged 1 commit into
uutils:mainfrom
xtqqczze:xattr-cfg

Conversation

@xtqqczze

Copy link
Copy Markdown
Collaborator

Follow-up to #14652.

@xtqqczze

This comment was marked as outdated.

@xtqqczze
xtqqczze force-pushed the xattr-cfg branch 2 times, most recently from 894d7fc to 95ca386 Compare September 20, 2026 12:18
Comment thread tests/by-util/test_cp.rs
#[cfg(all(
unix,
not(any(target_vendor = "apple", target_os = "android", target_os = "openbsd"))
#[cfg(any(

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.

same os list copy-pasted in 5 places now, could we use a cfg alias (cfg_aliases in build.rs) instead?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I would like to do something with cfg_aliases eventually, #11016 was my last attempt.

Comment thread tests/uutests/Cargo.toml Outdated
auto-merge was automatically disabled September 26, 2026 12:51

Head branch was pushed to by a user without write access

@xtqqczze
xtqqczze requested a review from sylvestre September 27, 2026 08:53
@sylvestre
sylvestre merged commit c6d0aed into uutils:main Sep 27, 2026
88 checks passed
@xtqqczze
xtqqczze deleted the xattr-cfg branch September 27, 2026 22:22
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