Success

User tests: Successful: Unsuccessful:

avatar elkuku
elkuku
8 Jul 2015

Hello again ?

I recently had to setup my environment on a new computer and I ran into some installation problems, so this PR tries to fix this for new installs (exising install should not be affected so this should be full B/C ?)

  1. php-curl is not installed

    In this case HttpFactory::getAvailableDriver() returns false. We miss a check in our GitHubLoginHelper here so the next call will produce a fatal error.

    I also added the CURL extension as a dependency to our composer.json file so the application wouldn't even install if CURL is not available.
  2. The log path is not available

    This might have changed in recent Apache versions, not sure. It is required to use a full path in the config file for the log path setting as a relative path would not work (at least on my system).

Please review.

avatar elkuku elkuku - open - 8 Jul 2015
avatar mbabker mbabker - reference | e17ae6b - 8 Jul 15
avatar mbabker mbabker - merge - 8 Jul 2015
avatar mbabker mbabker - close - 8 Jul 2015
avatar mbabker mbabker - change - 8 Jul 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-07-08 16:34:03
Closed_By mbabker
avatar mbabker mbabker - close - 8 Jul 2015
avatar mbabker mbabker - head_ref_deleted - 8 Jul 2015

Add a Comment

Login with GitHub to post a comment