-
Notifications
You must be signed in to change notification settings - Fork 22
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 devel package for linters #288
Conversation
%package devel | ||
Summary: Development package for os-autoinst-scripts-deps | ||
Group: Development/Tools/Other | ||
Requires: %devel_requires | ||
|
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.
I'm not sure if this enough to create a new package in OBS
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.
Good point. I believe so. This should create a new subpackage, like in openQA, but you are likely missing a %description
and %files
sections.
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.
ah right, the description.
I added both now. %files
is just empty, but it's probably needed.
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.
yes, an empty section is needed. Current version looks good now
%package devel | ||
Summary: Development package for os-autoinst-scripts-deps | ||
Group: Development/Tools/Other | ||
Requires: %devel_requires | ||
|
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.
Good point. I believe so. This should create a new subpackage, like in openQA, but you are likely missing a %description
and %files
sections.
Related ticket: https://progress.opensuse.org/issues/153793