Skip to content

doc: drop rsync dependency from manual build#14824

Merged
Ericson2314 merged 1 commit intoNixOS:masterfrom
roberth:issue-14776
Dec 18, 2025
Merged

doc: drop rsync dependency from manual build#14824
Ericson2314 merged 1 commit intoNixOS:masterfrom
roberth:issue-14776

Conversation

@roberth
Copy link
Member

@roberth roberth commented Dec 18, 2025

Motivation

Context

rsync was only used to copy source files while following symlinks. Replace with tar --dereference, which serves the same purpose. Tried plain cp but couldn't get it to work reliably. tar is already a test dependency.

Add tests/functional/derivation to fileset to include the symlink targets.

Output compares equal


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

rsync was only used to copy source files while following symlinks.
Replace with tar --dereference, which serves the same purpose.
Tried plain cp but couldn't get it to work reliably. tar is already
a test dependency.

Add tests/functional/derivation to fileset to include the symlink
targets.

Fixes NixOS#14776
Copy link
Contributor

@xokdvium xokdvium left a comment

Choose a reason for hiding this comment

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

Neat

@Ericson2314 Ericson2314 added this pull request to the merge queue Dec 18, 2025
Merged via the queue into NixOS:master with commit 4769f3c Dec 18, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Drop rsync dependency

3 participants