Bug 165293
Summary: | Review Request: perl-Locale-gettext | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Ralf Corsepius <rc040203> |
Component: | Package Review | Assignee: | Ville Skyttä <scop> |
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://search.cpan.org/~pvandry/gettext-1.05/ | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2005-08-11 04:52:29 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
Ralf Corsepius
2005-08-07 05:38:08 UTC
Not a review yet, nor an objection, but just to make sure you're aware of: We already have perl-libintl in Extras, and porting/patching apps to use it instead of Locale-gettext is often a matter of a trivial s/Locale::gettext/Locale::Messages/ If you'd still like to see L::g in Extras, I can review this. (In reply to comment #1) Well, I had been aware about perl-libintl, but I had not been aware of the "sed". > If you'd still like to see L::g in Extras, I can review this. Let me put it this way, ATM, Locale::gettext to me is just a dependency of another package. I'd prefer to leave the decision on whether to use Locale::gettext or Locale::Messages to upstream perl. Uh... upstream perl? Locale::Message/perl-libintl and Locale::gettext apparently are competing approaches, so unless perl decides to abandon one of each, I don't feel legitimated nor do I think FE is legitmated to abandon on of them. Sure, but neither is in upstream perl, so neither can be abandoned there. You mean CPAN? Anyway, as said, since you're convinced that this is needed, I can review it later today. (In reply to comment #5) > Sure, but neither is in upstream perl, so neither can be abandoned there. You > mean CPAN? Of cause ... nit-picking mode, today? > Anyway, as said, since you're convinced that this is needed Sorry, no, I am not convinced. It's just the most simple way to satisfy another package's dependency, and to avoid having to hack this other package. If _you_ should be convinced Locale::Message is 100% compatible to Locale::gettext, then I'd propose somebody should add a Locale::gettext to perl-libintl - However, too me, this sounds really dangerous. Shrug. REWORK: - BuildRequires: gettext missing -> test suite coverage Cosmetics: - URL tag is subject to bitrot - Release number in latest changelog entry is off sync Updated srpm/spec with Ville's review reflected: Spec Name or Url: ftp://packman.iu-bremen.de/fedora/SRPMS/perl-Locale-gettext.spec SRPM Name or Url: ftp://packman.iu-bremen.de/fedora/SRPMS/perl-Locale-gettext-1.05-2.src.rpm Approved. |