Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Prefer system rsync #243

Merged
merged 1 commit into from
Nov 2, 2023
Merged

Prefer system rsync #243

merged 1 commit into from
Nov 2, 2023

Conversation

vancluever
Copy link
Contributor

This sets up the package derivation to prefer the system rsync in the same way the system SSH is preferred, to prevent issues that may arise (e.g. crashes) that can arise from mixing the system SSH with the derivation-provided rsync.

Fixes #242.

This sets up the package derivation to prefer the system rsync in the
same way the system SSH is preferred, to prevent issues that may arise
(e.g. crashes) that can arise from mixing the system SSH with the
derivation-provided rsync.

Fixes #242.
@vancluever
Copy link
Contributor Author

PS: Not putting the updated lockfile in this one, but I'd imagine that this should also be shipped to nixpkgs even if it doesn't result in an actual version bump.

@Mic92
Copy link
Member

Mic92 commented Nov 2, 2023

@mergify queue

Copy link
Contributor

mergify bot commented Nov 2, 2023

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at bea2862

@mergify mergify bot merged commit bea2862 into nix-community:main Nov 2, 2023
@Mic92
Copy link
Member

Mic92 commented Nov 3, 2023

Are we sure that this has something to do with rsync/ssh not being compatible and not with this regression? NixOS/nixpkgs#265014

@nyawox
Copy link

nyawox commented Nov 3, 2023

Seems like nix profile install nixpkgs/nixos-23.05#rsync on local machine running unstable works as temporary easy workaround
Installed successful now

@vancluever
Copy link
Contributor Author

Hmm, yeah interesting! Admittedly for sure my only real need for rsync recently has been for nixos-anywhere, so if this is indeed the issue this can just be reverted - that'd fix the nixpkgs invocation too and the flake lockfile here could just be fast-forwarded when the build is fixed in nixpkgs/nixos-unstable.

henrirosten added a commit to tiiuae/ghaf-infra that referenced this pull request Nov 6, 2023
- Pin nixos-anywhere to prevent possible issues caused by upstream PR:
  nix-community/nixos-anywhere#243.
  Read more from the below nixpkgs issue:
  NixOS/nixpkgs#265014.
- Fix the task.py usage instructions in README.md after
  #8.
- Make `nix flake check` pre-push check less verbose.

Signed-off-by: Henri Rosten <henri.rosten@unikie.com>
@Mic92
Copy link
Member

Mic92 commented Nov 6, 2023

I am currently working on a proper fix: #234 to fix rsync itself.

henrirosten added a commit to tiiuae/ghaf-infra that referenced this pull request Nov 6, 2023
- Pin nixos-anywhere to prevent possible issues caused by upstream PR:
  nix-community/nixos-anywhere#243.
  Read more from the below nixpkgs issue:
  NixOS/nixpkgs#265014.
- Fix the task.py usage instructions in README.md after
  #8.
- Make `nix flake check` pre-push check less verbose.

Signed-off-by: Henri Rosten <henri.rosten@unikie.com>
fayadf pushed a commit to fayadf/ghaf-infra that referenced this pull request Oct 2, 2024
- Pin nixos-anywhere to prevent possible issues caused by upstream PR:
  nix-community/nixos-anywhere#243.
  Read more from the below nixpkgs issue:
  NixOS/nixpkgs#265014.
- Fix the task.py usage instructions in README.md after
  tiiuae#8.
- Make `nix flake check` pre-push check less verbose.

Signed-off-by: Henri Rosten <henri.rosten@unikie.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using --extra-files on a non-NixOS source machine causes crash with newer lockfile
3 participants