Update bin/memcacheSync.php for php7 syntax

Bug #1562096 reported by Nish Aravamudan
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
simplesamlphp (Ubuntu)
Fix Released
Wishlist
Unassigned
Xenial
New
Undecided
Unassigned

Bug Description

The specified file should be updated to the current mechanism for
enabling PHP modules.

Tags: php7
Revision history for this message
Nish Aravamudan (nacc) wrote :
Nish Aravamudan (nacc)
Changed in simplesamlphp (Ubuntu):
importance: Undecided → Wishlist
Revision history for this message
Thijs Kinkhorst (kink) wrote :

Thanks! I've fixed this in simpleSAMLphp master in a slightly different way; it will likely be part of the next release.

Changed in simplesamlphp (Ubuntu):
status: New → Fix Committed
Jeremy Bícha (jbicha)
tags: added: php7
Revision history for this message
Nish Aravamudan (nacc) wrote :

@Thijs, I don't think the fix in Yakkety (1.14.4-1) has this particular issue fixed. You did fix the PHP dependencies, but as far as I can tell, bin/memcacheSync.php in the latest version still only does a php5 configuration check. Please reconsider this patch.

Changed in simplesamlphp (Ubuntu):
status: Fix Committed → New
Revision history for this message
Thijs Kinkhorst (kink) wrote :

It is part of 1.14.5-1.

Nish Aravamudan (nacc)
Changed in simplesamlphp (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.