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.

Sign up for GitHub

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

Jump to bottom

Simple, more elegant implementation of setting uid/gid on copied files #191

Merged

r-vdp merged 1 commit into numtide:main from srd424:chown-v2

Feb 3, 2025

Contributor

srd424 commented Jan 29, 2025 •

edited

Loading

Simple implementation of setting uid/gid on copied files, more elegant error handing-wise than previous version.

This still does not yet look up user or group names, but it's useful enough to merge I think?

Partial fix for #190.


          Simple, more elegant implementation of setting uid/gid on copied files

cee3aa3

(does not yet look up user or group names.)

zimbatm requested a review from r-vdp

January 30, 2025 10:04

r-vdp merged commit de28d23 into numtide:main

20 checks passed

Member

r-vdp commented Feb 3, 2025

Thanks!

For the user resolution we can use https://docs.rs/nix/latest/nix/unistd/struct.User.html#method.from_name

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet