[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-07-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #19 from Fedora Update System  ---
php-symfony2-Routing-2.0.15-3.el6 has been pushed to the Fedora EPEL 6 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-07-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #18 from Fedora Update System  ---
php-symfony2-Routing-2.0.15-3.fc17 has been pushed to the Fedora 17 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-07-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2012-07-12 14:56:12

--- Comment #17 from Fedora Update System  ---
php-symfony2-Routing-2.0.15-3.fc16 has been pushed to the Fedora 16 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-07-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #16 from Fedora Update System  ---
php-symfony2-Routing-2.0.15-3.fc16 has been pushed to the Fedora 16 testing
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-07-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #15 from Fedora Update System  ---
php-symfony2-Routing-2.0.15-3.el6 has been submitted as an update for Fedora
EPEL 6.
https://admin.fedoraproject.org/updates/php-symfony2-Routing-2.0.15-3.el6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-07-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #14 from Fedora Update System  ---
php-symfony2-Routing-2.0.15-3.fc16 has been submitted as an update for Fedora
16.
https://admin.fedoraproject.org/updates/php-symfony2-Routing-2.0.15-3.fc16

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-07-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #13 from Fedora Update System  ---
php-symfony2-Routing-2.0.15-3.fc17 has been submitted as an update for Fedora
17.
https://admin.fedoraproject.org/updates/php-symfony2-Routing-2.0.15-3.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-07-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

Fedora Update System  changed:

   What|Removed |Added

 Status|NEW |MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #12 from Shawn Iwinski  ---
(In reply to comment #10)
> Sorry, but I have miss that one
> 
> /usr/share/pear/Symfony/Component/Routing/Loader/AnnotationClassLoader.php
> use Doctrine\Common\Annotations\Reader
> 
> So you probably need php-doctrine-DoctrineCommon (not yet packaged).
> 
> So please
> 1/ assure this is an optionnal dep
> 2/ consider submitting a review for this package.

Same response as https://bugzilla.redhat.com/show_bug.cgi?id=823075#c6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #11 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #10 from Remi Collet  ---
Sorry, but I have miss that one

/usr/share/pear/Symfony/Component/Routing/Loader/AnnotationClassLoader.php
use Doctrine\Common\Annotations\Reader

So you probably need php-doctrine-DoctrineCommon (not yet packaged).

So please
1/ assure this is an optionnal dep
2/ consider submitting a review for this package.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

Shawn Iwinski  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #9 from Shawn Iwinski  ---
New Package SCM Request
===
Package Name: php-symfony2-Routing
Short Description: Symfony2 Routing Component
Owners: siwinski
Branches: f16 f17 el6
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

Remi Collet  changed:

   What|Removed |Added

  Flags||fedora-review+

--- Comment #8 from Remi Collet  ---
No issue detected.

== APPROVED ==

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

Remi Collet  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|fed...@famillecollet.com

--- Comment #7 from Remi Collet  ---
Created attachment 593975
  --> https://bugzilla.redhat.com/attachment.cgi?id=593975&action=edit
php-symfony2-Routing-review.txt

Generated by fedora-review 0.1.3

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

Shawn Iwinski  changed:

   What|Removed |Added

 Whiteboard|NotReady|

--- Comment #6 from Shawn Iwinski  ---
Added optional requires

- Added php-pear(%{pear_channel}/Config) require
- Added php-pear(%{pear_channel}/Yaml) require
- Removed ownership for directories already owned by required packages

Update per comment #4

- Fix package.xml for *.xsd file issue

SPEC URL:
http://people.redhat.com/~siwinski/rpmbuild/SPECS/php-symfony2-Routing.spec

SRPM URL:
http://people.redhat.com/~siwinski/rpmbuild/SRPMS/php-symfony2-Routing-2.0.15-3.fc17.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

Shawn Iwinski  changed:

   What|Removed |Added

 Blocks||823075

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

Shawn Iwinski  changed:

   What|Removed |Added

 Depends On||823042, 817303

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #5 from Shawn Iwinski  ---
(In reply to comment #4)
> -- Code --
from Routing/Loader/XmlFileLoader.php

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #4 from Shawn Iwinski  ---
Issue: XSD file installed as doc, but referenced by code.

-- XSD file --
Symfony/Component/Routing/Loader/schema/routing/routing-1.0.xsd

-- Code --
protected function validate(\DOMDocument $dom)
{
$location = __DIR__.'/schema/routing/routing-1.0.xsd';

$current = libxml_use_internal_errors(true);
if (!$dom->schemaValidate($location)) {
throw new \InvalidArgumentException(implode("\n",
$this->getXmlErrors()));
}
libxml_use_internal_errors($current);
}



See original finding of this issue at
https://bugzilla.redhat.com/show_bug.cgi?id=823046#c8

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-06-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

Shawn Iwinski  changed:

   What|Removed |Added

 Whiteboard||NotReady

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-05-31 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #3 from Shawn Iwinski  ---
Updated to upstream version 2.0.15 & updates per bug #817303

- Removed "BuildRequires: php-pear >= 1:1.4.9-1.2"
- Updated %prep section
- Removed cleaning buildroot from %install section
- Removed documentation move from %install section (fixed upstream)
- Removed %clean section
- Updated %doc in %files section

SPEC URL:
http://people.redhat.com/siwinski/rpmbuild/SPECS/php-symfony2-Routing.spec

SRPM URL:
http://people.redhat.com/siwinski/rpmbuild/SRPMS/php-symfony2-Routing-2.0.15-1.fc16.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-05-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #2 from Shawn Iwinski  ---
Update per https://bugzilla.redhat.com/show_bug.cgi?id=823041#c5

- Moved documentation to correct location

SPEC URL:
http://people.redhat.com/siwinski/rpmbuild/SPECS/php-symfony2-Routing.spec

SRPM URL:
http://people.redhat.com/siwinski/rpmbuild/SRPMS/php-symfony2-Routing-2.0.14-3.fc16.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-05-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823060

--- Comment #1 from Shawn Iwinski  ---
Updates per comments in bug 823043

- Removed BuildRoot
- Changed php require to php-common
- Added the following requires based on phpci results:
  php-libxml, php-pcre, php-spl, php-tokenizer
- Removed %defattr from %files section

SPEC URL:
http://people.redhat.com/siwinski/rpmbuild/SPECS/php-symfony2-Routing.spec

SRPM URL:
http://people.redhat.com/siwinski/rpmbuild/SRPMS/php-symfony2-Routing-2.0.14-2.fc16.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823060] Review Request: php-symfony2-Routing - Symfony2 Routing Component

2012-05-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=823060

Shawn Iwinski  changed:

   What|Removed |Added

 Depends on||814994(php-channel-symfony2
   ||)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review