Bug#821484: Bumping severity of PHP 7.0 transition bugs to serious

2016-05-07 Thread James Valleroy 
On 05/07/2016 12:48 AM, Sunil Mohan Adapa wrote:
> I have applied the patch and marked that it closes the bug. However,
> we need to address the issue of enabling php7 module for upgrading
> users. I propose that we add 'a2enmod php7.0' to post-install. What do
> you think? 

Yes, I think that would make sense.



signature.asc
Description: OpenPGP digital signature

Bug#821484: Bumping severity of PHP 7.0 transition bugs to serious

2016-05-06 Thread Sunil Mohan Adapa 
On 05/07/2016 02:05 AM, James Valleroy wrote:
> tags 821484 patch
> thanks
> 
> The attached patch will simply switch from PHP 5 to PHP 7.0.
> 

The patch looks correct but insufficient.  Here are my notes:

- ownCloud depends on php5 but is being removed.

- tt-rss seems to work okay with php7.  Faced some weird issues during
installation, probably unrelated.

- Shaarli is still dependent on php5 modules but seems to work with php7.

- Roundcube seems to have problem with php7 but upstream supports php7
in 1.2beta (unreleased and not yet available in Debian).

- Php7 module is not enabled in freedombox-setup for people upgrading
from an earlier version.

I have applied the patch and marked that it closes the bug.  However, we
need to address the issue of enabling php7 module for upgrading users.
I propose that we add 'a2enmod php7.0' to post-install.  What do you think?

-- 
Sunil



signature.asc
Description: OpenPGP digital signature

Bug#821484: Bumping severity of PHP 7.0 transition bugs to serious

2016-05-06 Thread James Valleroy 
tags 821484 patch
thanks

The attached patch will simply switch from PHP 5 to PHP 7.0.

--
James
From c31380d50e3f6fe55788d5302d324184273a6b6d Mon Sep 17 00:00:00 2001
From: James Valleroy 
Date: Fri, 6 May 2016 07:46:42 -0400
Subject: [PATCH] Switch to PHP 7.0.

---
 debian/control | 2 +-
 setup.d/90_apache2 | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/debian/control b/debian/control
index 8fd813c..bc1b86d 100644
--- a/debian/control
+++ b/debian/control
@@ -47,7 +47,7 @@ Depends: ${misc:Depends}
  , iputils-ping
  , iw
  , libapache2-mod-gnutls
- , libapache2-mod-php5
+ , libapache2-mod-php
  , libnss-gw-name
  , libnss-mdns
  , libnss-myhostname
diff --git a/setup.d/90_apache2 b/setup.d/90_apache2
index dd132f2..30a79de 100755
--- a/setup.d/90_apache2
+++ b/setup.d/90_apache2
@@ -25,7 +25,7 @@ a2enmod headers
 
 # enable some critical modules to avoid restart while installing
 # Plinth applications.
-a2enmod php5
+a2enmod php7.0
 a2enmod cgi
 a2enmod authnz_ldap
 
-- 
2.8.1



signature.asc
Description: OpenPGP digital signature

Bug#821484: Bumping severity of PHP 7.0 transition bugs to serious

2016-05-05 Thread Ondřej Surý 
Dear maintainer(s),

I am bumping the severity of this bug to serious, as we are going to
remove src:php5 from Debian and your package is blocking the first
step which is removal of php5 from testing.  Please either update your
package to support PHP 7.0 or remove the package from Debian unstable
alltogether.

Cheers,
Ondrej