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

Problem: siegfried - %defattr directive doesn't work as expected #151

Open
jpellman opened this issue Apr 4, 2018 · 1 comment
Open
Labels

Comments

@jpellman
Copy link

jpellman commented Apr 4, 2018

When installing archivematica-mcp-client with yum under CentOS 7, archivematica-mcp-client pulls in the siegfried package defined here. As this occurs, I'm presented with the following warning messages:

  Installing : siegfried-1.7.6-1.el7.centos.x86_64                                                    115/177
warning: user 1000 does not exist - using root
warning: group 1000 does not exist - using root
warning: user 1000 does not exist - using root
warning: group 1000 does not exist - using root
...
warning: user 1000 does not exist - using root
warning: group 1000 does not exist - using root
warning: user 1000 does not exist - using root
warning: group 1000 does not exist - using root
  Installing : pcre-devel-8.32-17.el7.x86_64

This can be traced back to the %defattr directive on line 33 of the specfile. According to the RPM docs here:

The user and group identifiers must be non-numeric.

If there's a specific user/group that /usr/share/siegfried should belong to, you should use the plain text name apparently. Otherwise, it'll try to change ownership to a user with the string literal '1000' for a name.

(Not tagging this for phase 1 since it's only a warning and thus far, I have not noticed any ill effects that would prevent an end-user from processing data)

@sevein sevein changed the title %defattr directive doesn't work as expected Problem: siegfried - %defattr directive doesn't work as expected Apr 28, 2018
sevein added a commit that referenced this issue Apr 29, 2018
This fixes #151 - we were not using the `%defattr` directive properly. This
commits removes it. It seams reasonable to expect that the user wanting to
update the signature file needs root privileges.
@sevein sevein self-assigned this Apr 29, 2018
sevein added a commit that referenced this issue May 14, 2018
This fixes #151 - we were not using the `%defattr` directive properly. This
commits removes it. It seams reasonable to expect that the user wanting to
update the signature file needs root privileges.
@sevein sevein added the AM17 label May 14, 2018
@scollazo
Copy link
Contributor

In order to get this installed in current 1.7.x systems, we need to change the Release number in the package.spec., so the new package version will be 1.7.6-2. Can you do it?

@scollazo scollazo reopened this May 14, 2018
@sevein sevein removed their assignment May 10, 2019
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