Bug 190453 - [PATCH] www/owncloud Added missing dependency for textproc/php5-xmlwriter
Summary: [PATCH] www/owncloud Added missing dependency for textproc/php5-xmlwriter
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Kevin Lo
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-06-01 23:17 UTC by Jonathan Price
Modified: 2014-06-04 06:17 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Price 2014-06-01 23:17:20 UTC
With a default installation of Owncloud, an error will be printed to the log file if a user attempts to generate a link from a mobile device using the mobile app.

The error is due to it missing the php5-xmlwriter package. After installing this package, the error goes away and the link copying works correctly. There may be other situations that need php5-xmlwriter but I have not yet determined any.

Additionally, I have replaced the deprecated USES_BZIP2= yes with USES= tar:bzip2.

How-To-Repeat:
Sign in to an Owncloud instance installed from ports, and attempt to generate a share link for a file in your account. See the error given on the mobile device, and the message printed in the error log on the server (accessible from the Owncloud web interface).

Fix:
Add a dependency on the textproc/php5-xmlwriter port. A patch has been attached with this change applied.

Patch attached with submission follows:

--- Makefile.orig       2014-06-01 14:07:55.967614155 +0000
+++ Makefile    2014-06-01 14:25:20.061622251 +0000
@@ -2,6 +2,7 @@

 PORTNAME=      owncloud
 PORTVERSION=   6.0.3
+PORTREVISION=  1
 CATEGORIES=    www
 MASTER_SITES=  http://download.owncloud.org/community/

@@ -10,10 +11,10 @@

 LICENSE=       AGPLv3

-USE_BZIP2=     yes
+USES=          tar:bzip2
 USE_PHP=       bz2 ctype curl dom fileinfo filter gd hash iconv json \
                mbstring pdo session simplexml xml xmlreader \
-               xsl wddx zip zlib
+               xmlwriter xsl wddx zip zlib
 WANT_PHP_WEB=  yes

 OWNCLOUD_USERNAME?=    www
Comment 1 Jonathan Price 2014-06-03 11:32:26 UTC
Updated title to include [PATCH] www/owncloud. Title was previously misleading as it sounded like a patch to textproc/php5-xmlwriter.
Comment 2 Kevin Lo freebsd_committer freebsd_triage 2014-06-04 06:17:25 UTC
Committed, thanks.
Comment 3 commit-hook freebsd_committer freebsd_triage 2014-06-04 06:17:57 UTC
A commit references this bug:

Author: kevlo
Date: Wed Jun  4 06:17:16 UTC 2014
New revision: 356432
URL: http://svnweb.freebsd.org/changeset/ports/356432

Log:
  Add missing dependency on php5-xmlwriter.

  PR:	ports/190453
  Submitted by:	Jonathan Price

Changes:
  head/www/owncloud/Makefile