Bug 512727
| Summary: | Review Request: perl-Async-MergePoint - Resynchronise diverged control flow | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Nicolas Chauvet (kwizart) <kwizart> |
| Component: | Package Review | Assignee: | Marcela Mašláňová <mmaslano> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | fedora-package-review, notting |
| Target Milestone: | --- | Flags: | mmaslano:
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: | 2009-07-29 10:03:02 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: | 512729 | ||
|
Description
Nicolas Chauvet (kwizart)
2009-07-20 15:09:00 UTC
OK rpmlint on every package.
OK name of package accords to the Naming Guidelines.
OK spec file name match the base package %{name}.
OK package meet the Packaging Guidelines.
OK package has a good license.
OK spec file is written in American English.
OK spec file for the package must be legible.
OK The sources used to build the package must match the upstream source.
f4a6f9a9dc0a1748f503c01893d544cc
OK successful koji compilation.
http://koji.fedoraproject.org/koji/taskinfo?taskID=1497144
OK correct BuildRequires, Requires.
OK proper use of %find_lang macro.
OK shared library must call ldconfig in %post and %postun.
OK relocatable package must state this fact.
OK package must own their directories.
OK permissions on files must be set properly.
OK %clean section with rm -rf %{buildroot}.
OK consistent use of macros.
OK package contains code, or permissable content.
OK large documentation go in a -doc subpackage.
OK %doc must not affect the runtime of the application.
OK header files must be in a -devel package.
OK static libraries must be in a -static package.
OK pkgconfig(.pc) files must 'Requires: pkgconfig'.
OK library files with a suffix (.so) must go in -devel.
OK usually devel packages must require the base package.
OK Remove .la libtool archives.
OK GUI applications must include a %{name}.desktop file.
OK %install section starts with rm -rf %{buildroot}.
Probably you can also change (same comment as in bz512725) files section. From %{perl_vendorlib}/* %exclude %dir %{perl_vendorarch}/auto/ To %{perl_vendorarch}/Async %{perl_vendorarch}/auto/Async If you want you can generate spec by cpanspec and we can go through review again, but I don't think that will be necessary. Thx for the review, I will adopt the suggested %files section New Package CVS Request ======================= Package Name: perl-Async-MergePoint Short Description: Resynchronise diverged control flow Owners: kwizart Branches: F-11 F-10 EL-5 InitialCC: perl-sig I suppose you forgot set cvs flag to ? http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure I hope you don't mind if I set it. Thx. I've missed it, indeed. cvs done. |