From f23ef1c96859f6b01a9268c8606a22ceb719cf3d Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Fri, 26 Apr 2013 05:26:58 -0400 Subject: [PATCH] Fix error handling in CLI mode, use STDERR and non-empty exit code (#1489043) --- bin/installto.sh | 39 +++++++++++++++++++++++++-------------- 1 files changed, 25 insertions(+), 14 deletions(-) diff --git a/bin/installto.sh b/bin/installto.sh index b1e2956..8e1ab1f 100755 --- a/bin/installto.sh +++ b/bin/installto.sh @@ -5,8 +5,11 @@ | bin/installto.sh | | | | This file is part of the Roundcube Webmail client | - | Copyright (C) 2011, The Roundcube Dev Team | - | Licensed under the GNU GPL | + | Copyright (C) 2012, The Roundcube Dev Team | + | | + | Licensed under the GNU General Public License version 3 or | + | any later version with exceptions for skins & plugins. | + | See the README file for a full license statement. | | | | PURPOSE: | | Update an existing Roundcube installation with files from | @@ -14,9 +17,6 @@ +-----------------------------------------------------------------------+ | Author: Thomas Bruederli <roundcube@gmail.com> | +-----------------------------------------------------------------------+ - - $Id$ - */ define('INSTALL_PATH', realpath(dirname(__FILE__) . '/..') . '/' ); @@ -26,17 +26,17 @@ $target_dir = unslashify($_SERVER['argv'][1]); if (empty($target_dir) || !is_dir(realpath($target_dir))) - die("Invalid target: not a directory\nUsage: installto.sh <TARGET>\n"); + rcube::raise_error("Invalid target: not a directory\nUsage: installto.sh <TARGET>", false, true); // read version from iniset.php $iniset = @file_get_contents($target_dir . '/program/include/iniset.php'); if (!preg_match('/define\(.RCMAIL_VERSION.,\s*.([0-9.]+[a-z-]*)/', $iniset, $m)) - die("No valid Roundcube installation found at $target_dir\n"); + rcube::raise_error("No valid Roundcube installation found at $target_dir", false, true); $oldversion = $m[1]; -if (version_compare($oldversion, RCMAIL_VERSION, '>=')) - die("Installation at target location is up-to-date!\n"); +if (version_compare(version_parse($oldversion), version_parse(RCMAIL_VERSION), '>=')) + rcube::raise_error("Installation at target location is up-to-date!", false, true); echo "Upgrading from $oldversion. Do you want to continue? (y/N)\n"; $input = trim(fgets(STDIN)); @@ -44,20 +44,31 @@ if (strtolower($input) == 'y') { $err = false; echo "Copying files to target location..."; - foreach (array('program','installer','bin','SQL','plugins','skins/default') as $dir) { - if (!system("rsync -avuC " . INSTALL_PATH . "$dir/* $target_dir/$dir/")) { + foreach (array('program','installer','bin','SQL','plugins','skins') as $dir) { + if (!system("rsync -avC " . INSTALL_PATH . "$dir/* $target_dir/$dir/")) { $err = true; break; } } - foreach (array('index.php','.htaccess','config/main.inc.php.dist','config/db.inc.php.dist','CHANGELOG','README','UPGRADING') as $file) { - if (!system("rsync -avu " . INSTALL_PATH . "$file $target_dir/$file")) { + foreach (array('index.php','.htaccess','config/main.inc.php.dist','config/db.inc.php.dist','CHANGELOG','README.md','UPGRADING','LICENSE') as $file) { + if (!system("rsync -av " . INSTALL_PATH . "$file $target_dir/$file")) { $err = true; break; } } echo "done.\n\n"; - + + if (is_dir("$target_dir/skins/default")) { + echo "Removing old default skin..."; + system("rm -rf $target_dir/skins/default $target_dir/plugins/jqueryui/themes/default"); + foreach (glob(INSTALL_PATH . "plugins/*/skins") as $plugin_skin_dir) { + $plugin_skin_dir = preg_replace('!^.*' . INSTALL_PATH . '!', '', $plugin_skin_dir); + if (is_dir("$target_dir/$plugin_skin_dir/classic")) + system("rm -rf $target_dir/$plugin_skin_dir/default"); + } + echo "done.\n\n"; + } + if (!$err) { echo "Running update script at target...\n"; system("cd $target_dir && bin/update.sh --version=$oldversion"); -- Gitblit v1.9.1