Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 184011
Summary: | Review Request: nickle | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Michel Alexandre Salim <michel.salim> |
Component: | Package Review | Assignee: | Gérard Milmeister <gemi> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | gemi |
Target Milestone: | --- | Flags: | michel:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-11-13 06:25:36 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779 |
Description
Michel Alexandre Salim
2006-03-05 00:24:11 UTC
Not a full review, as I am still awaiting sponsorship, but: 1) Description of devel package is ... lacking :) 2) Source0: http://nickle.org/release/nickle-%{version}.tar.gz could be replaced with Source0: http://nickle.org/release/%{name}-%{version}.tar.gz 3) You shouldn't install the INSTALL file as part of %doc - the package is installed, don't confuse users by adding installation instructions :) 4) rpmlint only gives one error about the GPG key (so I think all is fine): rpmlint nickle-2.53-1.src.rpm E: nickle unknown-key GPG#f77eed90 Ah. I wrote the spec before clarifying with upstream what the -devel package is used for, and neglected to added that .. thanks. As for upstream filename, it's common to not replace the name with %{name} - someone might want to make a parallel-installable version of the package, in which case using %{name} will break. Thanks for the note! Spec Name or Url: http://hircus.org/fedora/nickle/nickle.spec SRPM Name or Url: http://hircus.org/fedora/nickle/nickle-2.53-2.src.rpm - /usr/share/nickle/COPYING can be removed, it is already with the doc - I would move the examples to the doc - rpmlint nickle-devel-2.53-2.fc4.i386.rpm: E: nickle-devel description-line-too-long Include files for Nickle, used for bulding external FFI (foreign function interface) libraries (e.g. the Cairo interface for Nickle). Fixed: Spec Name or Url: http://hircus.org/fedora/nickle/nickle.spec SRPM Name or Url: http://hircus.org/fedora/nickle/nickle-2.53-3.src.rpm - why you use %dir %{_includedir}/nickle %{_includedir}/nickle/* instead of simply %{_includedir}/nickle/ - Do not use %ghost to exclude files, simply remove them. Any reason why %ghost is bad? The %dir is just a matter of style, I can change it if you want. %ghost DOES prevent the files from being included in the package. However it still causes a reference to be created in the files list with the attributes etc... This is used for example for log files that may be created during the operation of the program, so that it gets the right permissions. In our case, we simply want the files removed. In the end, here it does not make much difference. There is also another directive, %exclude, that simply does not package the file. ping? pong. http://hircus.org/fedora/nickle/nickle-2.54-1.src.rpm http://hircus.org/fedora/nickle/nickle.spec Sorry for the late response! Changed the %ghost to %exclude and avoided using %dir and then listing everything under it. I think it is enough to have rm -rf $RPM_BUILD_ROOT instead of [ "$RPM_BUILD_ROOT" != "/" ] && rm -rf $RPM_BUILD_ROOT The examples directory appears twice, once in the docdir and once in /usr/share/nickle. I prefer the docdir. The examples directory contains Makefile* files which are not functional. http://hircus.org/fedora/nickle/nickle-2.54-2.src.rpm http://hircus.org/fedora/nickle/nickle.spec Fixed, thanks. Note that the two left-over COPYING files in %{_docdir}/examples are there because they cite different authors than the main COPYING file. * source files match upstream: ffc7b03a830e64ec0547777330ae00b8 nickle-2.54.tar.gz * package meets naming and packaging guidelines * specfile is properly named, is cleanly written and uses macros consistently * dist tag is present * build root is correct * license field matches the actual license * license is open source-compatible, license text included in package * latest version is being packaged * BuildRequires are proper * compiler flags are appropriate * %clean is present * package builds in mock (FC-6, i386) * package installs properly * rpmlint is silent (no doc for -devel package, which is ok) * final provides and requires are sane * no shared libraries are added to the regular linker search paths * owns the directories it creates * doesn't own any directories it shouldn't * no duplicates in %files * file permissions are appropriate * no scriptlets present * code, not content * documentation is small, so no -docs subpackage is necessary * %docs are not necessary for the proper functioning of the package * header files in -devel package * no pkgconfig files * no libtool .la droppings APPROVED Rawhide build done. Thanks for reviewing! Package Change Request ====================== Package Name: nickle New Branches: EL-5 cvs done. |