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 167797
Summary: | perl-DBD-Pg package doesn't Provide: perl(DBD::Pg) | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Paul Howarth <paul> | ||||
Component: | perl-DBD-Pg | Assignee: | Warren Togami <wtogami> | ||||
Status: | CLOSED RAWHIDE | QA Contact: | |||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | CC: | perl-devel | ||||
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: | 2005-11-10 16:20:24 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
Paul Howarth
2005-09-08 10:31:42 UTC
Well, the bigger question is figuring out *why* it no longer auto-provides. Created attachment 118587 [details] Patch to reformat Pg.pm so that perl.prov can interpret it properly (In reply to comment #1) > Well, the bigger question is figuring out *why* it no longer auto-provides. Good point. Patch fixes the formatting of Pg.pm so that auto-provides works again. Ideal fix would be for perl.prov to be able to parse the code as-is. Thanks. Could you please submit this upstream too to make future maintenance easier? While I agree it would be ideal to fix perl.prov, that isn't a solution for older distros unfortunately. =( Reported upstream: http://rt.cpan.org/NoAuth/Bug.html?id=14509 One theoretical point worth noting: Even though providing perl(DBD::Pg) in this package is correct, one could argue that packages in general should not have manually added dependencies on perl(DBD::*) in their specfiles. In the vast majority of use cases, the DBD::* drivers are not explicitly loaded in Perl code, but DBI loads the appropriate driver based on the data source given in connect(). Therefore, manually adding dependencies on perl(DBD::*) is kind of hardcoding assumptions about DBI's internals into specfiles, and in that sense, using the corresponding perl-DBD-* package names in dependencies wouldn't be that bad an alternative. I've added a Provides: for this, should show up tomorrow in the FC-development release. regards, Florian La Roche Ville, to clarify should we add the patch or a virtual provides? As long as the package provides correct things, IMO the implementation is a matter of maintainer preference. I would have personally opted for the Provides too. (BTW, I don't see the change from comment 7 in CVS yet. ) Upstream reports that this is now fixed in CVS and "will appear correctly formatted in 1.44", so the manual Provides: should be removable when that version is released. http://rt.cpan.org/NoAuth/Bug.html?id=14509 |