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 170507
Summary: | Review Request: perl-Mail-Mbox-MessageParser | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Paul Howarth <paul> |
Component: | Package Review | Assignee: | Orion Poplawski <orion> |
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-extras-list |
Target Milestone: | --- | Flags: | j:
fedora-cvs+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://sourceforge.net/projects/m-m-msgparser/ | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2005-10-12 15:34:21 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: | 170509 | ||
Bug Blocks: | 163779, 170506 |
Description
Paul Howarth
2005-10-12 14:23:28 UTC
- Builds clean - License checks out - rpmlint is silent My only suggestion is to perhaps change the Requires to: Requires: %{__grep}, %{__gzip}, %{__bzip2} since that is what you are passing to the makefile. However, since they probably always will be provided by grep, gzip, and bzip2, it's not that big of a deal. Approved. My personal preference is to use package names rather than program pathnames in deps, but I'm open to being convinced that the latter is better if someone can cite a good reason (other than the package names possibly changing in the future, which I'm prepared to deal with in the unlikely event of it happening). Thanks for the (very quick!) reviews by the way. Build on target development succeeded. Paul if you want to own it in EPEL too, then please add yourself. Package Change Request ====================== Package Name: perl-Mail-Mbox-MessageParser New Branches: EL-5 EL-6 Owners: mmaslano ppisar psabata InitialCC: perl-sig Is this needed for something that isn't grepmail? Package Change Request ====================== Package Name: perl-Mail-Mbox-MessageParser New Branches: el4 el5 el6 Owners: mmaslano ppisar psabata pghmcfc InitialCC: perl-sig Git done (by process-git-requests). perl-Mail-Mbox-MessageParser-1.5002-6.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/perl-Mail-Mbox-MessageParser-1.5002-6.el6 perl-Mail-Mbox-MessageParser-1.5002-6.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/perl-Mail-Mbox-MessageParser-1.5002-6.el5 perl-Mail-Mbox-MessageParser-1.5002-6.el4 has been submitted as an update for Fedora EPEL 4. https://admin.fedoraproject.org/updates/perl-Mail-Mbox-MessageParser-1.5002-6.el4 perl-Mail-Mbox-MessageParser-1.5002-6.el5 has been pushed to the Fedora EPEL 5 stable repository. If problems still persist, please make note of it in this bug report. perl-Mail-Mbox-MessageParser-1.5002-6.el4 has been pushed to the Fedora EPEL 4 stable repository. If problems still persist, please make note of it in this bug report. perl-Mail-Mbox-MessageParser-1.5002-6.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report. |