-
Notifications
You must be signed in to change notification settings - Fork 28
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
dist: fix RPM specfile groups, call ldconfig directly #330
base: master
Are you sure you want to change the base?
Conversation
@@ -22,7 +22,7 @@ Version: @VERSION@ | |||
Release: 0 | |||
Summary: Library for storage management | |||
License: GPL-2.0 | |||
Group: System/Libraries | |||
Group: Development/Libraries/C and C++ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this change? What are the definitions for those two groups?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
System/Libraries: "Packages in this group should already be installed automatically because of a dependency." libstorage-ng.src.rpm (that is what line 25 touches here) however neither contains a .so file, nor does it get autoinstalled.
More group descriptions are on https://en.opensuse.org/openSUSE:Package_group_guidelines#System .
Note that |
@wfeldt Please get your team mechanics in order. It is maximally annoying to get redirected in circles. OBS says "go use github" and github people say "go use OBS".
|
@jengelh Sorry, wasn't aware of this. Will check. |
Yes, indeed. And you can't override the project settings with per-package settings to turn it off easily for |
@jengelh After discussion with @shundhammer: for the time being, continue patching spec.in. If anything is agreed on the patch will be manually added to obs. For the future, tooling will be adjusted. |
No description provided.