Overview
Request 810469 superseded
- Created by badshah400
- In state superseded
- Superseded by 811083
- Open review for openSUSE:Factory:Staging:adi:66
Request History
badshah400 created request
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
licensedigger accepted review
ok
maxlin_factory added as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:66"
maxlin_factory accepted review
Picked "openSUSE:Factory:Staging:adi:66"
dimstar declined review
35-Requires(post): info
36+Requires(post): info
37 Requires(preun): info
This is legacy - the post/postun scriptlets for info files are no longer needed, as they are handled by file triggers
26-BuildRequires: pkg-config
27+BuildRequires: pkgconfig
why? the package has always been called pkg-config for openSUSE (the replacement pkgconf-pkg-config provides the symbol, but should not explicitly be used)
dimstar declined request
35-Requires(post): info
36+Requires(post): info
37 Requires(preun): info
This is legacy - the post/postun scriptlets for info files are no longer needed, as they are handled by file triggers
26-BuildRequires: pkg-config
27+BuildRequires: pkgconfig
why? the package has always been called pkg-config for openSUSE (the replacement pkgconf-pkg-config provides the symbol, but should not explicitly be used)
Thanks for the review. IIRC the
pkg-config->pkgconfigchange was done by spec-cleaner@dimstar: as to why pkg-config -> pkgconfig: it's a stupidity of spec-cleaner