Bug 1046122 (php-doctrine-collections) - Review Request: php-doctrine-collections - Collections abstraction library
Summary: Review Request: php-doctrine-collections - Collections abstraction library
Keywords:
Status: CLOSED RAWHIDE
Alias: php-doctrine-collections
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Remi Collet
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: php-doctrine-common php-doctrine-orm
TreeView+ depends on / blocked
 
Reported: 2013-12-23 18:33 UTC by Shawn Iwinski
Modified: 2014-01-06 17:43 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-01-06 17:43:35 UTC
Type: ---
Embargoed:
fedora: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)
phpci.log (14.62 KB, text/plain)
2014-01-03 07:59 UTC, Remi Collet
no flags Details
review.txt (7.12 KB, text/plain)
2014-01-03 07:59 UTC, Remi Collet
no flags Details

Description Shawn Iwinski 2013-12-23 18:33:19 UTC
Spec URL: https://raw.github.com/siwinski/rpms/58afbf34df09331c037ce42a9f83d089708ec0ac/php-doctrine-collections.spec

SRPM URL: http://siwinski.fedorapeople.org/SRPMS/php-doctrine-collections-1.1-1.20131221git8198717.fc20.src.rpm

Description:
Collections abstraction library.


Fedora Account System Username: siwinski

Comment 1 Remi Collet 2014-01-03 07:59:18 UTC
Created attachment 844862 [details]
phpci.log

phpcompatinfo version 2.26.0.

Comment 2 Remi Collet 2014-01-03 07:59:44 UTC
Created attachment 844863 [details]
review.txt

Generated by fedora-review 0.5.0 (920221d) last change: 2013-08-30
Command line :/usr/bin/fedora-review -b 1046122

Comment 3 Remi Collet 2014-01-03 08:00:14 UTC
[!]: Dist tag is present (not strictly required in GL).
  Use %{?dist}

[!]: Package does not generate any conflict.
  Conflicts:     php-pear(pear.doctrine-project.org/DoctrineCommon) < 2.4
  Both packages provides Doctrine\Common\Collections
  As /usr/share/pear is before /usr/share/php this will break
  autoloader relying on include_path.
  Also explain this package is a split-off DoctrineCommon

Comment 5 Remi Collet 2014-01-04 06:50:22 UTC
[x]: Dist tag is present (not strictly required in GL).

[x]: Package does not generate any conflict.


All blocker fixed.

== APPROVED ==

Comment 6 Shawn Iwinski 2014-01-04 15:32:55 UTC
THANKS for the review!


New Package SCM Request
=======================
Package Name: php-doctrine-collections
Short Description: Collections abstraction library
Owners: siwinski
Branches: f19 f20 el6
InitialCC:

Comment 7 Gwyn Ciesla 2014-01-06 13:19:26 UTC
Git done (by process-git-requests).


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