Page MenuHomePhabricator

Add setup test for ctype "extension" because it's off by default in OpenSUSE
Closed, ResolvedPublic

Description

Getting this crash when downloading a file:

>>> UNRECOVERABLE FATAL ERROR <<<

Call to undefined function ctype_digit()

/srv/phabricator/phabricator/src/infrastructure/storage/lisk/LiskDAO.php:417


┻━┻ ︵ ¯\_(ツ)_/¯ ︵ ┻━┻

Event Timeline

hach-que assigned this task to epriestley.
hach-que raised the priority of this task from to Needs Triage.
hach-que updated the task description. (Show Details)
hach-que added a project: Phabricator.
hach-que added a subscriber: hach-que.

This is part of the PHP core, do you have a really weird version of PHP?

http://us1.php.net/ctype_digit

It's normally only disabled if PHP is built explicit with --disable-ctype:

http://us1.php.net/manual/en/ctype.installation.php

Well, this is interesting:

http://software.opensuse.org/package/php5-ctype?search_term=php5-ctype

I don't have that package installed on my Docker install, so I'm going to update it to install that package. What I don't understand is how this can be an optional extension if it's part of the PHP core (as per the documentation).

Most of the core is still built as "extensions" so it's technically possible (i.e., without ripping anything apart), it's just seems very odd to exclude ctype from a package since this doesn't imply any unusual build or install dependencies.

(We can add a setup check for the "extension" if your build was something quasi-mainstream, did you get it out of some package manager that other users might conceivably also use?)

This is the standard package manager on openSUSE, from the official openSUSE repositories.

epriestley renamed this task from LiskDAO crashes with 'undefined function ctype_digit' to Add setup test for ctype "extension" because it's off by default in OpenSUSE.Apr 29 2014, 5:16 PM
epriestley triaged this task as Normal priority.Apr 29 2014, 5:18 PM