Bug 1129493
Summary: | Review Request: pidgin-extras - A Gtk+ based multiprotocol instant messaging client | ||
---|---|---|---|
Product: | [Fedora] Fedora EPEL | Reporter: | Robert Scheck <redhat-bugzilla> |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED DUPLICATE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | epel7 | CC: | mcepl, mcepl, package-review, redhat-bugzilla, vwfoxguru |
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: | 2016-04-08 22:41:04 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Robert Scheck
2014-08-12 22:33:18 UTC
Robert, Thanks for your review submission. Here are my notes from what I can see so far: Reviewing your SPEC file: Since this is only for EL7, you should remove all of the Fedora/RHEL mapping globals in the beginning. If you still need globals, just use ones that cover your scope. There are other long obsoleted references in here, such as gaim. There is a also patch referencing RHEL4, which is probably irrelevant for EL7. Your obsoletes: Obsoletes: pidgin <= 2.7.1-1%{?dist} Pidgin is also not in Requires. It looks like you're actually building a replacement for the pidgin package, rather than an add-on? Doing so will conflict with the CentOS repo pidgin package, so you'll need to split out the extras. The Description describes pidgin generally, not what extras are intended to be provided by this package. Your changelog reflects the pidgin packaging history, not -extras specific. Here's the output from rpmlint: pidgin-extras.src: W: spelling-error Summary(en_US) multiprotocol -> multiprocessor pidgin-extras.src: W: spelling-error %description -l en_US perl -> Perl, peel, perk pidgin-extras.src:422: W: unversioned-explicit-obsoletes gaim-devel pidgin-extras.src:466: W: unversioned-explicit-obsoletes gaim-meanwhile pidgin-extras.src:142: W: mixed-use-of-spaces-and-tabs (spaces: line 27, tab: line 142) pidgin-extras.src: E: specfile-error warning: bogus date in %changelog: Tue Aug 06 2009 Warren Togami <wtogami> 2.6.0-0.11.20090812 pidgin-extras.src: E: specfile-error warning: bogus date in %changelog: Tue Aug 06 2009 Warren Togami <wtogami> 2.6.0-0.10.20090806 pidgin-extras.src: E: specfile-error warning: bogus date in %changelog: Fri May 31 2007 Stu Tomlinson <stu> - 2.0.1-2 pidgin-extras.src: E: specfile-error warning: bogus date in %changelog: Mon Jun 08 2004 Warren Togami <wtogami> 0.78-7 pidgin-extras.src: E: specfile-error warning: bogus date in %changelog: Fri Apr 15 2004 Warren Togami <wtogami> 0.76-6 pidgin-extras.src: E: specfile-error warning: bogus date in %changelog: Fri Apr 14 2004 Warren Togami <wtogami> 0.76-5 pidgin-extras.src: E: specfile-error warning: bogus date in %changelog: Wed Nov 25 2003 Christopher Blizzard <blizzard> 1:0.74-0 1 packages and 0 specfiles checked; 7 errors, 5 warnings. Any of the "E"'s here absolutely need addressed. If you haven't already, please take a look at https://fedoraproject.org/wiki/Packaging:Guidelines Regarding obsoleting the CentOS pidgin, see https://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies: "EPEL packages should only enhance and never disturb the Enterprise Linux distributions they were built for. Thus packages from EPEL should never replace packages from the target base distribution" However, you might still be able to split out the extra addons in a way that doesn't conflict. I'm not absolutely sure if you got the intention of this review request: The pidgin-extras package is actually a copy of the pidgin source RPM that is shipped by RHEL 7, the package thus got a package review before (given the package was taken from Fedora). Further on the changes have been kept as small as possible to hopefully easily rebase on possible pidgin updates in RHEL 7. I'm glad to see that bug #1254778 followed the same intention and got approved for EPEL 7. The "extra" here is to ship "pidgin", the base package, because "pidgin" itself is not in RHEL 7, so %description is right. RHEL 7 only ships some sub-packages of pidgin which makes this a somewhat funny situation. Given "pidgin-epel" is now in EPEL 7 (which is the same approach, but on spec file level slightly different solved), this package review can just be closed. If Matěj is interested, I'm happy to get a co-maintainer, but the package didn't change that much since 2014 as it seems, so less efforts needed :) *** This bug has been marked as a duplicate of bug 1254778 *** Besides, a birdie told me that it is probably we will have /usr/bin/pidgin back in RHEL 7.3. |