[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2015-01-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2015-01-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2015-01-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

Shawn Iwinski  changed:

   What|Removed |Added

 CC||shawn.iwin...@gmail.com
  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #18 from Shawn Iwinski  ---
Package Change Request
==
Package Name: php-xmlseclibs
New Branches: el6 epel7
Owners: fkooman siwinski
InitialCC: 

Co-maintainership approved through email with fkooman.  Does he need to add
approval here as well?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||php-xmlseclibs-1.3.1-1.fc19
 Resolution|--- |ERRATA
Last Closed||2013-06-29 14:44:52

--- Comment #17 from Fedora Update System  ---
php-xmlseclibs-1.3.1-1.fc19 has been pushed to the Fedora 19 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=9YrkiM9kZH&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #16 from Fedora Update System  ---
php-xmlseclibs-1.3.1-1.fc19 has been pushed to the Fedora 19 testing
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=7akCQAE5bS&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

--- Comment #15 from Fedora Update System  ---
php-xmlseclibs-1.3.1-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/php-xmlseclibs-1.3.1-1.fc18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=gfcXTUnOFG&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

--- Comment #14 from Fedora Update System  ---
php-xmlseclibs-1.3.1-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/php-xmlseclibs-1.3.1-1.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=OKmwxrZDZY&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=jreFrrlDgu&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |
  Flags||fedora-cvs+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=be8zQt96JE&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=7WAdgtvlEc&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

--- Comment #12 from Remi Collet  ---
@François, no thanks (If need I will use my provenpackager super-power).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=NTNUIe1ttC&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

--- Comment #11 from François Kooman  ---
@Remi: do you want to be co-owner?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=pdVKkc9f02&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

--- Comment #10 from François Kooman  ---
New Package SCM Request
===
Package Name: php-xmlseclibs
Short Description: PHP library for XML Security
Owners: fkooman
Branches: f18 f19
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=bOW0Edagiu&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

François Kooman  changed:

   What|Removed |Added

  Flags||fedora-cvs?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Z9IXV3LUPT&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

Remi Collet  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #9 from Remi Collet  ---
[x]: Requires correct, justified where necessary.
[x]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.

LICENSE file, provided by upstream now include.

So, no more blockers.

=== APPROVED ===

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=3D1AfxGMxj&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

--- Comment #8 from François Kooman  ---
Upstream released a new version, 1.3.1 which addresses all the issues as far as
I know and allows for the patch to be removed.

Spec URL: http://fkooman.fedorapeople.org/php-xmlseclibs/php-xmlseclibs.spec
SRPM URL:
http://fkooman.fedorapeople.org/php-xmlseclibs/php-xmlseclibs-1.3.1-1.fc18.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ISLNqFSCz2&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

--- Comment #7 from François Kooman  ---
Spec URL: http://fkooman.fedorapeople.org/php-xmlseclibs/php-xmlseclibs.spec
SRPM URL:
http://fkooman.fedorapeople.org/php-xmlseclibs/php-xmlseclibs-1.3.0-6.fc18.src.rpm

Updated the (Build)Requires according to phpci output. I hope the PEAR
requirements are sufficient (as it is just used for testing).

I also talked to upstream (mail) about the future of the project and they seem
to be getting back to updating the code soon.

I agree we should probably wait for a new release with the license stuff fixed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=GyT9IaMqL8&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

--- Comment #6 from Remi Collet  ---
[x]: Patches link to upstream bugs/comments/lists or are otherwise justified.
[x]: SourceX tarball generation or download is documented.
[x]: %check is present and all tests pass.

[!]: Requires correct, justified where necessary.

Still not correct.
From http://fedoraproject.org/wiki/Packaging:PHP#Extensions_Requires

"PHP extensions must have a Requires on ALL of the dependent extensions."

So should , according to phpci, should be, else please justify
php-dom
php-hash
php-libxml
php-mcrypt
php-openssl
(and remove php-xml)

[!]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.

The request is fine but not enough. From
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License_Text

* Include a copy of what they believe the license text is intended to be...
* Choose not to package that software for Fedora. 

So I propose to wait a little from upstream on your filed bug.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=UyhIBW5yvJ&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

--- Comment #5 from François Kooman  ---
I think I fixed all issues where possible.

Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5515838

Spec URL: http://fkooman.fedorapeople.org/php-xmlseclibs/php-xmlseclibs.spec
SRPM URL:
http://fkooman.fedorapeople.org/php-xmlseclibs/php-xmlseclibs-1.3.0-5.fc18.src.rpm

Thanks for your thorough review!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=irO2EL0GyZ&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

Remi Collet  changed:

   What|Removed |Added

 Attachment #761578|0   |1
is obsolete||
 Attachment #761578|review? |
  Flags||

--- Comment #4 from Remi Collet  ---
Created attachment 761579
  --> https://bugzilla.redhat.com/attachment.cgi?id=761579&action=edit
review.txt

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=10HFjsKzN3&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

Remi Collet  changed:

   What|Removed |Added

 Attachment #761578||review?
  Flags||

--- Comment #2 from Remi Collet  ---
Created attachment 761578
  --> https://bugzilla.redhat.com/attachment.cgi?id=761578&action=edit
review.txt

Generated by fedora-review 0.4.1 (b2e211f) last change: 2013-04-29
Buildroot used: fedora-rawhide-x86_64
Command line :/usr/bin/fedora-review -b 974492

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=xgH6w97uL4&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

Remi Collet  changed:

   What|Removed |Added

  Flags||fedora-review?

--- Comment #3 from Remi Collet  ---
Blockers

[!]: Requires correct, justified where necessary.
[!]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[!]: Patches link to upstream bugs/comments/lists or are otherwise justified.
[!]: SourceX tarball generation or download is documented.
[!]: %check is present and all tests pass.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=1snjVgAB7E&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

Remi Collet  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||fed...@famillecollet.com
   Assignee|nob...@fedoraproject.org|fed...@famillecollet.com

--- Comment #1 from Remi Collet  ---
Created attachment 761576
  --> https://bugzilla.redhat.com/attachment.cgi?id=761576&action=edit
phpci.log

phpcompatinfo version 2.17.0.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=j0RCs8QTRD&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974492] Review Request: php-xmlseclibs - PHP library for XML Security

2013-06-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974492

François Kooman  changed:

   What|Removed |Added

 Blocks||800867

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=nnsA6zCkRj&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review