[Bug 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2013-12-31 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Adam Williamson  changed:

   What|Removed |Added

 Depends On||1047596




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1047596
[Bug 1047596] Review Request: php-channel-phpseclib - Adds the phpseclib
channel to PEAR
-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2013-12-31 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Adam Williamson  changed:

   What|Removed |Added

 Blocks||1047611




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1047611
[Bug 1047611] Review Request: php-phpseclib-Net-SSH2 - Pure-PHP
implementation of SSHv2
-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2013-12-31 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Remi Collet  changed:

   What|Removed |Added

 CC||fed...@famillecollet.com



--- Comment #1 from Remi Collet  ---
Quick note about the renaming: the prefered way to alter pear installation is
to change do this in %prep and fixing the package.xml, so the file will be
installed as wanted.

Any change after "pear install" breaks the pear metadata : "pear list-files"
will be inconsistent with "rpm -ql".

-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2014-01-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



--- Comment #2 from Adam Williamson  ---
Updated:

https://www.happyassassin.net/reviews/php-phpseclib-crypt-blowfish/php-phpseclib-crypt-blowfish.spec
https://www.happyassassin.net/reviews/php-phpseclib-crypt-blowfish/php-phpseclib-crypt-blowfish-0.3.5-2.fc21.src.rpm

please check if I got the renaming right! Thanks.

-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2014-01-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



--- Comment #3 from Remi Collet  ---
For now build fails.

ERROR: bad md5sum for file
/dev/shm/extras/BUILDROOT/php-phpseclib-crypt-blowfish-0.3.5-2.fc19.remi.x86_64/usr/share/pear/Crypt/Blowfish-phpseclib.php


sed -i -e 's,Blowfish.php,Blowfish-phpseclib.php,g' package.xml => OK.
sed -i -e 's,Blowfish.php,Blowfish-phpseclib.php,g' ...Blowfish.php => KO

If you change the content of a file, you need to clear the md5sum in the
package.xml else the pear install fails.

-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2014-01-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



--- Comment #4 from Remi Collet  ---
Also need to be fixed:
php-phpseclib-crypt-blowfish.src: W: invalid-url URL:
http://phpseclib.sourceforge.net/package/Crypt_Blowfish HTTP Error 404: Not
Found

-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2014-01-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Bug 1047601 depends on bug 1047596, which changed state.

Bug 1047596 Summary: Review Request: php-channel-phpseclib - Adds the phpseclib 
channel to PEAR
https://bugzilla.redhat.com/show_bug.cgi?id=1047596

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |RAWHIDE



-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2014-01-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



--- Comment #6 from Adam Williamson  ---
grr.

https://www.happyassassin.net/reviews/php-phpseclib-crypt-blowfish/php-phpseclib-crypt-blowfish-0.3.5-3.fc21.src.rpm

-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2014-01-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



--- Comment #5 from Adam Williamson  ---
Updated:

https://www.happyassassin.net/reviews/php-phpseclib-crypt-blowfish/php-phpseclib-crypt-blowfish.spec
https://www.happyassassin.net/reviews/php-phpseclib-crypt-blowfish/php-phpseclib-
crypt-blowfish-0.3.5-3.fc21.src.rpm

-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2014-01-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Remi Collet  changed:

   What|Removed |Added

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



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

-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2014-01-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



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

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

-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2014-01-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Remi Collet  changed:

   What|Removed |Added

  Flags||fedora-review?



--- Comment #9 from Remi Collet  ---
[!]: Requires correct, justified where necessary.
  From package.xml, option so package choice
php-mcrypt
  From code, used in Crypt_Blowfish::setPawword()
php-pear(phpseclib.sourceforge.net/Crypt_Hash)

[-]: 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 text is included in the single source header

[x]: Package does not own files or directories owned by other packages.
  Ok, but if you add require on Crypt_Hash, you have to drop the ownership of
  /usr/share/pear/Crypt

-- 
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 1047601] Review Request: php-phpseclib-Crypt-Blowfish - Pure-PHP implementation of Blowfish

2014-01-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



--- Comment #10 from Adam Williamson  ---
Updated:

https://www.happyassassin.net/reviews/php-phpseclib-crypt-blowfish/php-phpseclib-crypt-blowfish.spec
https://www.happyassassin.net/reviews/php-phpseclib-crypt-blowfish/php-phpseclib-
crypt-blowfish-0.3.5-4.fc21.src.rpm

-- 
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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-01-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Adam Williamson  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |php-phpseclib-Crypt-Blowfis |php-phpseclib-crypt-blowfis
   |h - Pure-PHP implementation |h - Pure-PHP implementation
   |of Blowfish |of Blowfish



-- 
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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-01-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Remi Collet  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #11 from Remi Collet  ---
[x]: Requires correct, justified where necessary.
[x]: Package does not own files or directories owned by other packages.

== APPROVED ==

-- 
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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-01-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Adam Williamson  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #12 from Adam Williamson  ---
New Package SCM Request
===
Package Name: php-phpseclib-crypt-blowfish
Short Description: Pure-PHP implementation of Blowfish
Owners: adamwill
Branches: f20 el6
InitialCC:

-- 
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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-01-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Jason Tibbitts  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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-01-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



--- Comment #13 from Jason Tibbitts  ---
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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-01-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Adam Williamson  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2014-01-14 16:47:46



--- Comment #14 from Adam Williamson  ---
Building for Rawhide:
http://koji.fedoraproject.org/koji/taskinfo?taskID=6405986

-- 
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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-02-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Adam Williamson  changed:

   What|Removed |Added

  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #15 from Adam Williamson  ---
Package Change Request
==
Package Name: php-phpseclib-net-ssh2
New Branches: f19 epel7

-- 
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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-02-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



--- Comment #16 from Adam Williamson  ---
correction:

Package Change Request
==
Package Name: php-phpseclib-crypt-blowfish
New Branches: f19 epel7

-- 
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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-02-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



--- Comment #17 from Jon Ciesla  ---
No owners specified.

-- 
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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-02-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Jon Ciesla  changed:

   What|Removed |Added

  Flags|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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

Adam Williamson  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #18 from Adam Williamson  ---
Package Change Request
==
Package Name: php-phpseclib-crypt-blowfish
New Branches: f19 epel7
Owners: adamwill

-- 
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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601

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 1047601] Review Request: php-phpseclib-crypt-blowfish - Pure-PHP implementation of Blowfish

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047601



--- 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