|
From: SourceForge.net <no...@so...> - 2009-10-20 19:27:58
|
Bugs item #2855430, was opened at 2009-09-09 21:36 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=2855430&group_id=191583 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Core Group: None >Status: Closed >Resolution: Fixed Priority: 6 Private: No Submitted By: Christian Boltz (christian_boltz) Assigned to: Nobody/Anonymous (nobody) Summary: setup.php: session support checked broken / too late Initial Comment: |Mike| just told me on IRC that he got an error message "Call to undefined function session_start() in .../common.php" when calling setup.php. He uses FreeBSD 7.2, which has a separate php5-session package. The problem: setup.php includes common.php at the beginning, which executes a session_start(). There's already a check for session support in setup.php, but it is done _after_ including common.php. A possible solution is to "define('POSTFIXADMIN', 1);" in setup.php which would avoid the session_start() call. The question is if this breaks something ;-) -> needs testing before commiting to SVN ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2009-10-20 21:27 Message: A small section at the beginning of common.php is the only place where the define(POSTFIXADMIN) in setup.php could break anything at all. Tested, does not break ;-) (except a "already defined" warning, also fixed in the commit) Also tested that setup.php now works as expected be using php.ini disable_function. Commited to SVN r733. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=2855430&group_id=191583 |
