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

[rpm] add logic to ensure 0555 permissions on /usr/bin for rhel >= 6 #4

Merged
merged 1 commit into from
Jan 18, 2017

Conversation

cwjohnston
Copy link

Description

We've found that RHEL 5 ships with mode 0755 on /usr/bin, while RHEL 6
and 7 ship with mode 0555. As a result, installing packages built for RHEL 6 or
7 contain files destined for /usr/bin will fail thusly:

file /usr/bin from install of sensu-1:0.27.0.beta.2-1.el7.x86_64 conflicts with file from package filesystem-3.2-20.el7.x86_64

An solution solution would be to implement an approach which considers the
permissions specified by the system's filesystem package at build time (see
chef#666 (comment)). For now, we'll
kick the can down the road a bit.

Signed-off-by: Cameron Johnston [email protected]

We've found that RHEL 5 ships with mode 0755 on /usr/bin, while RHEL 6
and 7 ship with mode 0555. As a result, installing packages built for RHEL 6 or
7 contain files destined for /usr/bin will fail thusly:

```
file /usr/bin from install of sensu-1:0.27.0.beta.2-1.el7.x86_64 conflicts with file from package filesystem-3.2-20.el7.x86_64
```

An solution solution would be to implement an approach which considers the
permissions specified by the system's `filesystem` package at build time (see
chef#666 (comment)). For now, we'll
kick the can down the road a bit.

Signed-off-by: Cameron Johnston <[email protected]>
@amdprophet amdprophet merged commit 6f4872b into sensu Jan 18, 2017
@amdprophet amdprophet deleted the fix/usr-bin-perms branch January 19, 2017 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants