Bug 207548

Summary: perl-Config-Record: missing the dist tag
Product: [Fedora] Fedora Reporter: Jose Pedro Oliveira <jose.p.oliveira.oss>
Component: perl-Config-RecordAssignee: Dennis Gregorovic <dgregor>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: extras-qa
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-09-22 18:17:55 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:
Attachments:
Description Flags
Add dist tag (and minor cleaning) none

Description Jose Pedro Oliveira 2006-09-21 16:06:05 UTC
Description of problem:
This package specfile is missing the dist tag.

Version-Release number of selected component (if applicable):
perl-Config-Record-1.1.1-2

Comment 1 Jose Pedro Oliveira 2006-09-21 16:12:04 UTC
Created attachment 136873 [details]
Add dist tag (and minor cleaning)

Dennis,

This patch add the dist tag and also cleans of lines no longer needed
for fedora distros:
  - the perl_vendorlib macro conditional definition (already supplied by rpm)
  - the perl build requirement (perl is one of the building system packages
    and all fedora distros have perl > 5.8.0)
  - the perl(Test::More) build requirement (Test::More is a perl 5.8 core
package)

jpo

Comment 2 Dennis Gregorovic 2006-09-21 16:39:27 UTC
Thanks!  Patch applied.

Would you like a new build?

Comment 3 Jose Pedro Oliveira 2006-09-21 17:33:14 UTC
(In reply to comment #2)
> ...
> Would you like a new build?

Yes please.

Right now there are only 3 perl packages without a dist tag in the FE devel
repository (and we now have more than 450 packages in the perl- namespace).

Regards,
jpo

Comment 5 Jose Pedro Oliveira 2006-09-21 18:17:21 UTC
Thanks for the new build.

By the way, you can improve the test suite coverage by adding
    BuildRequires:  perl(Test::Pod)
    BuildRequires:  perl(Test::Pod::Coverage)
to the specfile (see the mock build log).

We are now down to 2 perl packages without the dist tag and both of them have
already have bugzilla tickets ;).

Regards,
jpo

Comment 6 Jose Pedro Oliveira 2006-09-22 18:17:55 UTC
Dennis,

Closing as perl-Config-Record-1.1.1-3.fc6.noarch.rpm is already available in the
mirrors.

Thanks,
jpo