Bug 960851 (Horde_Dav) - Review Request: php-horde-Horde-Dav - Horde library for WebDAV, CalDAV, CardDAV
Summary: Review Request: php-horde-Horde-Dav - Horde library for WebDAV, CalDAV, CardDAV
Alias: Horde_Dav
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Shawn Iwinski
QA Contact: Fedora Extras Quality Assurance
: 960850 (view as bug list)
Depends On: Horde_Core
Blocks: Horde_Rpc
TreeView+ depends on / blocked
Reported: 2013-05-08 07:05 UTC by Remi Collet
Modified: 2014-02-11 19:14 UTC (History)
3 users (show)

Fixed In Version: php-horde-Horde-Dav-1.0.3-2.el6
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2014-01-27 15:57:10 UTC
Type: ---
shawn: fedora-review+
gwync: fedora-cvs+

Attachments (Terms of Use)
phpcompatinfo.log (51.86 KB, text/x-log)
2014-01-27 03:47 UTC, Shawn Iwinski
no flags Details
fedora-review.txt (6.43 KB, text/plain)
2014-01-27 03:48 UTC, Shawn Iwinski
no flags Details

Description Remi Collet 2013-05-08 07:05:15 UTC
Spec URL: https://raw.github.com/remicollet/remirepo/master/php/horde/php-horde-Horde-Dav/php-horde-Horde-Dav.spec
SRPM URL: http://rpms.famillecollet.com/SRPMS/php-horde-Horde-Dav-1.0.0-0.2.beta1.remi.src.rpm
This package contains all Horde-specific wrapper classes for the Sabre DAV

Fedora Account System Username: remi

P.S.1 : I will clean EL-5 stuff (Buildroot, clean, ...) after import
P.S.2 : SabreDAV 1.8 is not yet available in Fedora, just to prepare future

Comment 1 Remi Collet 2013-09-29 17:27:48 UTC
Not ready: blocked by SabreDav >= 1.8 requires (and owncloud need 1.7)

Comment 2 Remi Collet 2014-01-03 15:59:59 UTC
php-sabre-dav 1.8 and php-sabre-vobject 2.1.3 are now available in fedora repo (sabredav channel is no more maintained, but old versions, without namespace, are still used by owncloud).

To ensure the correct version is loaded, need to install only php-sabre-dav, or use php-horde-Horde-Autoloader 2.0.1-4 (in testing) which is patched to ensure correct load order.

Changes: https://github.com/remicollet/remirepo/commit/273c45aa482a2a4a9d2c20d3846553e09950b118

Spec: https://raw.github.com/remicollet/remirepo/273c45aa482a2a4a9d2c20d3846553e09950b118/php/horde/php-horde-Horde-Dav/php-horde-Horde-Dav.spec
Srpm: http://rpms.famillecollet.com/SRPMS/php-horde-Horde-Dav-1.0.3-2.remi.src.rpm

Comment 3 Remi Collet 2014-01-12 06:03:06 UTC
*** Bug 960850 has been marked as a duplicate of this bug. ***

Comment 4 Shawn Iwinski 2014-01-27 03:47:24 UTC
Created attachment 855893 [details]

phpcompatinfo version 2.26.0

Comment 5 Shawn Iwinski 2014-01-27 03:48:13 UTC
Created attachment 855894 [details]

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

Comment 6 Shawn Iwinski 2014-01-27 03:49:51 UTC
[!]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
     beginning of %install.
     Note: rm -rf %{buildroot} present but not required
[!]: Each %files section contains %defattr if rpm < 4.4
     Note: %defattr present but not needed
[!]: Buildroot is not present
     Note: Buildroot: present but not needed
[!]: Package has no %clean section with rm -rf %{buildroot} (or
     Note: %clean present but not required

Remove EPEL 5 bits after initial import.

[x]: Requires correct, justified where necessary.

     I am assuming all extensions found by phpcompatinfo are implicitly required
     by other required packages...  php-spl should be the same then correct?

Obviously wait for dependencies before importing.

No blockers

===== APPROVED =====

Comment 7 Remi Collet 2014-01-27 07:35:18 UTC
New Package SCM Request
Package Name: php-horde-Horde-Dav
Short Description: Horde library for WebDAV, CalDAV, CardDAV
Owners: remi
Branches: f20 el6 epel7

Comment 8 Gwyn Ciesla 2014-01-27 13:06:11 UTC
Git done (by process-git-requests).

Comment 9 Fedora Update System 2014-01-27 16:03:58 UTC
php-horde-Horde-Dav-1.0.3-2.fc20 has been submitted as an update for Fedora 20.

Comment 10 Fedora Update System 2014-01-27 16:04:18 UTC
php-horde-Horde-Dav-1.0.3-2.el6 has been submitted as an update for Fedora EPEL 6.

Comment 11 Fedora Update System 2014-02-05 03:33:35 UTC
php-horde-Horde-Dav-1.0.3-2.fc20 has been pushed to the Fedora 20 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Fedora Update System 2014-02-11 19:14:25 UTC
php-horde-Horde-Dav-1.0.3-2.el6 has been pushed to the Fedora EPEL 6 stable repository.  If problems still persist, please make note of it in this bug report.

Note You need to log in before you can comment on or make changes to this bug.