Re: Patch to implement publicweb (oops)

my stupid diff program didn't say which file was which. The first one is for Common.php, the second for Config.php, and the third for Installer.php. I also missed the closing ), in the 2nd patch to Config.php Greg Greg Beaver wrote:

Hi all, I've got a few simple patches that make minimal changes to the PEAR installer, and support the new role=publicweb. Until it is decided as to how Merz's excellent role handling idea is implemented, can everyone who cares about it review the patches and +1/-1 it prior to my commit? Thanks, Greg ------------------------------------------------------------------------ 66c66 < $GLOBALS['_PEAR_Common_file_roles'] = array('php','ext','test','doc','data','src','script'); ---

$GLOBALS['_PEAR_Common_file_roles'] = array('php','ext','test','doc','data','src','script','publicweb'); ------------------------------------------------------------------------ 114a115,123

// Default for publicweb dir if (getenv('PHP_PEAR_PUBLICWEB_DIR')) { define('PEAR_CONFIG_DEFAULT_PUBLICWEB_DIR', getenv('PHP_PEAR_PUBLICWEB_DIR')); } else { // use the data directory if no publicweb option is defined define('PEAR_CONFIG_DEFAULT_PUBLICWEB_DIR',

          PEAR_CONFIG_DEFAULT_DATA_DIR);

}

281a291,296

       'publicweb_dir' => array(
           'type' => 'directory',
           'default' => PEAR_CONFIG_PUBLICWEB_DIR,
           'doc' => 'sub-directory of http doc root where public html-related files are installed',
           'prompt' => 'PEAR html-files directory',
           'group' => 'File Locations',

------------------------------------------------------------------------ 182a183

           case 'publicweb':