[Bug 1162996] Review Request: php-phpseclib-crypt-base - Base class for phpseclib cipher implementations

2020-05-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162996

Mattia Verga  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |ERRATA
Last Closed||2020-05-30 13:45:17



--- Comment #9 from Mattia Verga  ---
This package was approved and imported in repositories, but this review ticket
was never closed.
I'm closing it now.


-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1162996] Review Request: php-phpseclib-crypt-base - Base class for phpseclib cipher implementations

2014-11-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162996



--- Comment #3 from Remi Collet fed...@famillecollet.com ---
Created attachment 956614
  -- https://bugzilla.redhat.com/attachment.cgi?id=956614action=edit
review.txt

Generated by fedora-review 0.5.2 (63c24cb) last change: 2014-07-14

-- 
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 1162996] Review Request: php-phpseclib-crypt-base - Base class for phpseclib cipher implementations

2014-11-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162996

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

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



--- Comment #2 from Remi Collet fed...@famillecollet.com ---
Created attachment 956613
  -- https://bugzilla.redhat.com/attachment.cgi?id=956613action=edit
phpci.log

 phpCompatInfo version 3.5.0 static analyze results

-- 
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 1162996] Review Request: php-phpseclib-crypt-base - Base class for phpseclib cipher implementations

2014-11-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162996

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

  Flags||fedora-review?



--- Comment #4 from Remi Collet fed...@famillecollet.com ---
MUST
[!]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/share/pear/Crypt
  = add %dir %{pear_phpdir}/Crypt
 (and of course you can drop ownership of this dir from packages which
require base)

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

-- 
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 1162996] Review Request: php-phpseclib-crypt-base - Base class for phpseclib cipher implementations

2014-11-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162996



--- Comment #5 from Adam Williamson (Red Hat) awill...@redhat.com ---
Just did a ninja bump which changes the directory ownership, thanks for the
catch (I'll adjust dependent packages later). Can't remember if we talked to
upstream about licenses already, I'll look into it. Thanks.

for the phpci, for the record, mcrypt dep is optional.

-- 
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 1162996] Review Request: php-phpseclib-crypt-base - Base class for phpseclib cipher implementations

2014-11-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162996

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #6 from Remi Collet fed...@famillecollet.com ---
$ colordiff -u php-phpseclib-crypt-base.spec.00 php-phpseclib-crypt-base.spec
--- php-phpseclib-crypt-base.spec.002014-11-12 08:37:39.0 +0100
+++ php-phpseclib-crypt-base.spec   2014-11-12 09:21:56.0 +0100
@@ -63,7 +63,7 @@

 %files
 %{pear_xmldir}/%{name}.xml
-%{pear_phpdir}/Crypt/Base.php
+%{pear_phpdir}/Crypt



=== 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 1162996] Review Request: php-phpseclib-crypt-base - Base class for phpseclib cipher implementations

2014-11-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162996

Adam Williamson (Fedora) adamw+fed...@happyassassin.net changed:

   What|Removed |Added

 CC||adamw+fedora@happyassassin.
   ||net
  Flags||fedora-cvs?



--- Comment #7 from Adam Williamson (Fedora) adamw+fed...@happyassassin.net 
---
New Package SCM Request
===
Package Name: php-phpseclib-crypt-base
Short Description: Base class for phpseclib cipher implementations
Owners: adamwill
Branches: f19 f20 f21 el6 epel7
InitialCC:

(still adamw, I split my bugzilla accounts a while back, sorry)

-- 
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 1162996] Review Request: php-phpseclib-crypt-base - Base class for phpseclib cipher implementations

2014-11-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162996

Jon Ciesla limburg...@gmail.com 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 1162996] Review Request: php-phpseclib-crypt-base - Base class for phpseclib cipher implementations

2014-11-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162996



--- Comment #8 from Jon Ciesla limburg...@gmail.com ---
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 1162996] Review Request: php-phpseclib-crypt-base - Base class for phpseclib cipher implementations

2014-11-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162996



--- Comment #1 from Adam Williamson (Red Hat) awill...@redhat.com ---
*** Bug 1162995 has been marked as a duplicate of this bug. ***

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