[Bug 1087734] Review Request: php-horde-imp - A web based webmail system

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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|php-horde-imp-6.1.7-2.fc20  |php-horde-imp-6.1.7-2.el6



--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
php-horde-imp-6.1.7-2.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.
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 1087734] Review Request: php-horde-imp - A web based webmail system

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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version||php-horde-imp-6.1.7-2.fc20
 Resolution|NEXTRELEASE |ERRATA



--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
php-horde-imp-6.1.7-2.fc20 has been pushed to the Fedora 20 stable repository.

-- 
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 1087734] Review Request: php-horde-imp - A web based webmail system

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



--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
php-horde-imp-6.1.7-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/php-horde-imp-6.1.7-2.fc20

-- 
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 1087734] Review Request: php-horde-imp - A web based webmail system

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



--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
php-horde-imp-6.1.7-2.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-horde-imp-6.1.7-2.el6

-- 
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 1087734] Review Request: php-horde-imp - A web based webmail system

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



--- Comment #4 from Remi Collet fed...@famillecollet.com ---
Good catch !

https://github.com/remicollet/remirepo/commit/14f2b9bd6182eb35e75497aa22b3b8c7571905eb


Spec:
https://raw.githubusercontent.com/remicollet/remirepo/14f2b9bd6182eb35e75497aa22b3b8c7571905eb/php/horde/php-horde-imp/php-horde-imp.spec
Srpm: http://rpms.famillecollet.com/SRPMS/php-horde-imp-6.1.7-2.remi.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 1087734] Review Request: php-horde-imp - A web based webmail system

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

Shawn Iwinski shawn.iwin...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Shawn Iwinski shawn.iwin...@gmail.com ---
[x]: Package contains no bundled libraries without FPC exception.

 This was a license issue/update, not a bundled library issue/update.

[x]: Requires correct, justified where necessary.


HTTPD config updated.


No blockers.


= 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 1087734] Review Request: php-horde-imp - A web based webmail system

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

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

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #6 from Remi Collet fed...@famillecollet.com ---
Thanks!

New Package SCM Request
===
Package Name: php-horde-imp
Short Description: A web based webmail system
Owners: remi
Branches: f20 el6 epel7
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 1087734] Review Request: php-horde-imp - A web based webmail system

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

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 1087734] Review Request: php-horde-imp - A web based webmail system

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



--- Comment #7 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 1087734] Review Request: php-horde-imp - A web based webmail system

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

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

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2014-04-29 01:12:07



--- Comment #8 from Remi Collet fed...@famillecollet.com ---
Build in rawhide + epel7

Will push all the horde applications at the same time (when all will be
reviewed) in f20 / epel6

-- 
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 1087734] Review Request: php-horde-imp - A web based webmail system

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

Shawn Iwinski shawn.iwin...@gmail.com changed:

   What|Removed |Added

 CC||shawn.iwin...@gmail.com
   Assignee|nob...@fedoraproject.org|shawn.iwin...@gmail.com
  Flags||fedora-review?



-- 
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 1087734] Review Request: php-horde-imp - A web based webmail system

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



--- Comment #1 from Shawn Iwinski shawn.iwin...@gmail.com ---
Created attachment 890302
  -- https://bugzilla.redhat.com/attachment.cgi?id=890302action=edit
phpcompatinfo.log

phpcompatinfo version 2.26.0

-- 
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 1087734] Review Request: php-horde-imp - A web based webmail system

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



--- Comment #2 from Shawn Iwinski shawn.iwin...@gmail.com ---
Created attachment 890303
  -- https://bugzilla.redhat.com/attachment.cgi?id=890303action=edit
fedora-review.txt

Generated by fedora-review 0.5.1 (bb9bf27) last change: 2013-12-13
Command line :/usr/bin/fedora-review --mock-config fedora-rawhide-x86_64 -b
1087734
Buildroot used: fedora-rawhide-x86_64
Active plugins: Generic, PHP, Shell-api
Disabled plugins: Java, C/C++, Python, fonts, SugarActivity, Ocaml, Perl,
Haskell, R, Ruby
Disabled flags: EXARCH, EPEL5, BATCH, DISTTAG

-- 
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 1087734] Review Request: php-horde-imp - A web based webmail system

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



--- Comment #3 from Shawn Iwinski shawn.iwin...@gmail.com ---
[!]: Package contains no bundled libraries without FPC exception.

 /usr/share/horde/imp/js/murmurhash3.js = 
 https://github.com/kazuyukitanimura/murmurhash-js

 /**
  * JS Implementation of MurmurHash3
  *
  * Original version:
  * https://github.com/kazuyukitanimura/murmurhash-js
  *
  * Additions by Michael Slusarz slus...@horde.org

 It appears the original source has additions by horde.  The original
source is MIT and this file still has the MIT license text in it.  If this is
allowed, does that mean MIT should be listed in the license field as well? 
This is probably more of a license question than a bundled library
question/issue.





[!]: Requires correct, justified where necessary.

 Per package.xml, update
 Requires:   php-pear(%{pear_channel}/Horde_Mime) = 2.0.0
 to
 Requires:   php-pear(%{pear_channel}/Horde_Mime) = 2.1.0


 Per package.xml, update
 Requires:   php-pear(%{pear_channel}/Horde_Url) = 2.0.0
 to
 Requires:   php-pear(%{pear_channel}/Horde_Url) = 2.2.0


 Per package.xml, add
 Requires:   php-pear(%{pear_channel}/Horde_Text_Filter_Csstidy) 
3.0.0


 Update comment
 # Optional and implicitly required:
 # Horde_Vfs, Horde_History, Horde_Http, Horde_Stream_Filter
 to
 # Optional and implicitly required:
 # Horde_History, Horde_Http


 Per phpcompatinfo, I do not see php-ereg or php-posix dependencies.





The HTTP config has DirectoryMatch
%{pear_hordedir}/%{pear_name}/(config|lib|locale), but the templates
directory has the same .htaccess file contents that the listed directories have
so shouldn't it be added in the config 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