Skip to content
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.

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

add chown() method #195

Open
mjgardner opened this issue Jun 27, 2017 · 3 comments
Open

add chown() method #195

mjgardner opened this issue Jun 27, 2017 · 3 comments
Labels

Comments

@mjgardner
Copy link

Please add a chown method to parallel the existing chmod method.

@xdg
Copy link
Contributor

xdg commented Jun 27, 2017

What advantage does that have over the built-in chown? E.g. why not:

# assuming $file = path("/some/path/to/whatever");
chown 100, 100, $file;

Generally, Path::Tiny methods don't just wrap built-ins. There are exceptions (move, remove) but I view those as legacy mistakes.

@mjgardner
Copy link
Author

Mainly so it can throw the same exceptions on failure as the other Path::Tiny methods, rather than having to make built-in chown throw its own exception.

@martinvonwittich
Copy link

The ability to use user/group names instead of uids/gids like in File::chown would be nice.

@xdg xdg added the wishlist label Oct 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants