Bug 337791 - Review Request: php-channel-symfony - Adds symfony project channel to PEAR
Review Request: php-channel-symfony - Adds symfony project channel to PEAR
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jason Tibbitts
Fedora Extras Quality Assurance
:
Depends On:
Blocks: 337801 351441
  Show dependency treegraph
 
Reported: 2007-10-18 08:08 EDT by Alexander Kahl
Modified: 2010-05-28 10:54 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-10-30 05:19:24 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
tibbs: fedora‑review+
tibbs: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Alexander Kahl 2007-10-18 08:08:01 EDT
Spec URL: http://prelive.iconmobile.com/dev31/fedora-icm-repo/Fedora/7/SPECS/php-channel-symfony.spec
SRPM URL: http://prelive.iconmobile.com/dev31/fedora-icm-repo/Fedora/7/SRPMS/php-channel-symfony-1.0.0-1.fc7.src.rpm
Description: This package adds the symfony channel which allows
PEAR packages from this channel to be installed.
Comment 1 Jason Tibbitts 2007-10-23 20:17:55 EDT
There's not much to these channel packages.

rpmlint says:
  php-channel-symfony.noarch: W: no-documentation
which is true, but not a blocker.

I'm not sure your URL: is correct.  Shouldn't it point to symfony-project.com? 
phing.info seems to be the page for a different project.

I'm not even sure the channel.xml contains copyrightable content, but given that
the actual symfony-project software is all under the MIT license, are you sure
that LGPLv2 is proper?  Where do you find that documented?

* source file matches upstream:
   b5e1ea8f15922ce174428fc9b264a08434a81ee97a327fd84692bf042a218533  channel.xml
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
? license field matches the actual license.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* rpmlint output is OK.
* final provides and requires are sane:
   php-channel(pear.symfony-project.com)
   php-channel-symfony = 1.0.0-1.fc8
  =
   /bin/sh
   /usr/bin/pear
   php-cli
   php-pear(PEAR)
* doesn't own any directories it shouldn't.
* no duplicates in %files (only one file!)
* file permissions are appropriate.
* scriptlets are OK (PEAR channel registration)
Comment 2 Alexander Kahl 2007-10-24 04:13:40 EDT
(In reply to comment #1)
[...]
> I'm not sure your URL: is correct.  Shouldn't it point to symfony-project.com? 
> phing.info seems to be the page for a different project.
> 
> I'm not even sure the channel.xml contains copyrightable content, but given that
> the actual symfony-project software is all under the MIT license, are you sure
> that LGPLv2 is proper?  Where do you find that documented?
[...]

Both issues you point out are mistakes I've made when copying the phing channel
spec file as a template for this one and not making the double check.

This should fix it:
Updated Spec URL:
http://prelive.iconmobile.com/dev31/fedora-icm-repo/Fedora/7/SPECS/php-channel-symfony.spec
Updated SRPM URL:
http://prelive.iconmobile.com/dev31/fedora-icm-repo/Fedora/7/SRPMS/php-channel-symfony-1.0.0-2.fc7.src.rpm
Comment 3 Jason Tibbitts 2007-10-25 12:49:52 EDT
OK, the license looks good to me and the URL is correct.

APPROVED
Comment 4 Alexander Kahl 2007-10-25 18:50:25 EDT
New Package CVS Request
=======================
Package Name: php-channel-symfony
Short Description: Adds symfony project channel to PEAR
Owners: akahl
Branches: F-7 F-8
InitialCC: 
Cvsextras Commits: yes
Comment 5 Alexander Kahl 2007-10-30 05:19:24 EDT
All builds successful.
Comment 6 Christof Damian 2010-05-27 14:25:24 EDT
Package Change Request
======================
Package Name: pkgname
New Branches: EL-6
Owners: remi
Comment 7 Jason Tibbitts 2010-05-27 18:18:27 EDT
This is a bit confusing.

"pkgname" obviously isn't what you intended to put there; I can guess, but as these are processed by a script, please do try to make proper requests.

I see the owner has recently changed so that you own this package.  However, since you won't own the EL-6 branch, we need an ack from remi.
Comment 8 Christof Damian 2010-05-28 01:25:35 EDT
I am sorry, here is the correct one. Remi asked me to create the branch, because he needs the channel for another package he has for EL-6. Would it help if I put myself also in as additional owner for EL-6 ?


Package Change Request
======================
Package Name: php-channel-symfony 
New Branches: EL-6
Owners: remi
Comment 9 Remi Collet 2010-05-28 01:49:24 EDT
@Jason : my RFE for this branch in EL-6 is bug #592528

And yes, I'm ok to maintain it in EPEL.
Comment 10 Jason Tibbitts 2010-05-28 10:31:32 EDT
Great, although it would have been simpler to just raise the CVS flag in that bug as the explanation is all there.

CVS done.
Comment 11 Christof Damian 2010-05-28 10:48:46 EDT
(In reply to comment #10)
> Great, although it would have been simpler to just raise the CVS flag in that
> bug as the explanation is all there.
> 
> CVS done.    

I will remember for the next time. I was under the impression that they always should go on the review request. But the other bug makes obviously more sense.
Comment 12 Jason Tibbitts 2010-05-28 10:54:42 EDT
We will find any ticket with the fedora-cvs flag set to '?'.  The intent is to keep folks from opening separate tickets just to request CVS, and the package review ticket works for this.  But if you have an existing ticket that actually contains useful context regarding the CVS request you're making, it of course makes sense to raise the fedora-cvs flag there.

Note You need to log in before you can comment on or make changes to this bug.