Overview
Request 593316 accepted
- Fix grammatical error in description
- Fix SRPM group.
- Update descriptions.
- update to 1.1.0:
* add handling of font index encoding
* add fontconfig dependency
* add freetype dependency
* add common variables LIB_INSTALL_DIR, BIN_INSTALL_DIR,
INCLUDE_INSTALL_DIR to set install directories
- Use %license for LICENSE
- Update to release 1.0.1, bump soversion (API break in 1.0.0)
- initial package version
- Created by StefanBruens
- In state accepted
- Supersedes 591759
Group: line of the SRPM/main package cannot be System/Libraries. Only the shlib package libemf2svg1 is allowed to have it.
The main package is empty, so this is fine
When I'm making a comment, it's not fine.
So, please where in the packaging guidelines can this be found?
https://en.opensuse.org/openSUSE:Package_group_guidelines """System/Libraries is intended for packages providing the part of libraries necessary to run applications. Packages in this group should already be installed automatically because of a dependency. Therefore, this group ought to be never used for the main Group: line of a .spec file (since the libraries ought to be in a subpackage, see SLPP). Neither users nor developers should need to search for packages in this group."""
Which exactly proves my interpretation: - there is no main package, so this is not violated - the library is packaged according to the SLPP and has the right group set
"Ought to be never used" is a weak statement which is justified by the fact the library is not in the main package, but other stuff is. The Group for the main package is guided by its contents.
What is the right main group if the spec builds only a library subpackage? What if it also builds a devel subpackage?
The guidelines are meant for build packages only, the whole reasoning is about packages which are installed.
If there were any policy about SRPMs, this should be clearly stated in the guidelines. But whats the correct Group for a SRPM creating library, documentation, devel and example subpackages?
What is the right main group if the spec builds only a library subpackage?
Easy: the topic/field in which the library operates. Same document,
"""If a library can be attributed to a specific RPM topic, use it. For example, the ntl library focuses on specific aspects of number theory, and so [is] under Productivity/Scientific/Math, while a general-purpose library like libHX is in the fallback group Development/Libraries/C and C++. For packages that fall inbetween two groups, pick the one that is less abstract, e.g. pick Math over Libs/C++."""
Care to cite the whole section ...?
It is about build packages - "This section [Development] has been created to help software developers. Normal users should never need to select packages from this group manually." ... "The following groups are intended for packages that allow developing with a library."
If a SRPM is meant for sw developers, all SRPMs should go into a Development subgroup. As this is clearly not intended, it does not apply to SRPMs. SRPMs are not for "developing with a library".
"The -devel subpackage is generally put into the group Development/Libraries/(Subgroup), depending on language. The subpackage providing the shared libraries located are generally marked as System/Libraries."
If you want the guidelines to be interpreted in a specific way, make the wording clear, specific and unambigous - after putting it up to discussion on the mailing list.
SRPMS are developy stuff, but if we made the Group a Devel/Something, then it would be wrong on the main BRPM too, if and when such is created. So the main Group: tag has to be the "topic" (Scientific/xyz, Graphics/abc, etc) out of necessity. This aspect gets reviewd in factory for over a year already.
If you feel the wiki is not adequately expressing already-applied practices, feel free to augment it, and if you need more detail, you can always ask about it.
Request History
StefanBruens created request
- Fix grammatical error in description
- Fix SRPM group.
- Update descriptions.
- update to 1.1.0:
* add handling of font index encoding
* add fontconfig dependency
* add freetype dependency
* add common variables LIB_INSTALL_DIR, BIN_INSTALL_DIR,
INCLUDE_INSTALL_DIR to set install directories
- Use %license for LICENSE
- Update to release 1.0.1, bump soversion (API break in 1.0.0)
- initial package version
licensedigger accepted review
ok
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto added repo-checker as a reviewer
Please review build success
factory-auto accepted review
Check script succeeded
staging-bot added openSUSE:Factory:Staging:adi:15 as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:15"
staging-bot accepted review
Picked openSUSE:Factory:Staging:adi:15
repo-checker accepted review
cycle and install check passed
mrdocs accepted review
ok
staging-bot accepted review
ready to accept
staging-bot approved review
ready to accept
maxlin_factory accepted request
Accept to openSUSE:Factory