Add fix for BSD stat #502
Add fix for BSD stat #502
Conversation
|
Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog. Non-GitHub Verified CommittersThere are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute. |
|
I've verified the commit author as being me with github |
| @@ -96,6 +99,9 @@ def set_attributes(action_handler, path, attributes) | |||
| # Get file attributes { :mode, :owner, :group } | |||
| def get_attributes(path) | |||
| result = transport.execute("stat -c '%a %U %G %n' #{path}", :read_only => true) | |||
| unless result.exitstatus == 0 | |||
| result = transport.execute("stat -f '%OLp %Sg %Su %N' #{path}", :read_only => true) | |||
There was a problem hiding this comment.
Double space stat -f and switch positions of %Sg %Su => %Su %Sg to be consistent with %U %G
Current implementation assumes GNU coreutils version of stat, which fails on BSD.
Try BSD format if GNU coreutils version fails.
fixes #501