aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatthew Wild <mwild1@gmail.com>2011-02-25 03:32:44 +0000
committerMatthew Wild <mwild1@gmail.com>2011-02-25 03:32:44 +0000
commit35a339b1d3defc1be8387324a601db4889096a22 (patch)
tree086fefd4ab54c5d19195a980f01100737e93c8ff
parent7c054e5160dc6e2a42ee8e66b9031f57c634421f (diff)
downloadprosody-35a339b1d3defc1be8387324a601db4889096a22.tar.gz
prosody-35a339b1d3defc1be8387324a601db4889096a22.zip
tools/migration/*.lua: Rename config to migrator.cfg.lua, add error handling for config and command-line parameters
-rw-r--r--tools/migration/main.lua77
-rw-r--r--tools/migration/migrator.cfg.lua (renamed from tools/migration/config.lua)0
2 files changed, 69 insertions, 8 deletions
diff --git a/tools/migration/main.lua b/tools/migration/main.lua
index 772349bf..e4891966 100644
--- a/tools/migration/main.lua
+++ b/tools/migration/main.lua
@@ -1,3 +1,5 @@
+local default_config = "./migrator.cfg.lua";
+
-- Command-line parsing
local options = {};
local handled_opts = 0;
@@ -15,23 +17,82 @@ end
table.remove(arg, handled_opts);
-- Load config file
-local function loadfilein(file, env) return loadin and loadin(env, io.open(file):read("*a")) or setfenv(loadfile(file), env); end
+local function loadfilein(file, env)
+ if loadin then
+ return loadin(env, io.open(file):read("*a"));
+ else
+ local chunk, err = loadfile(file);
+ if chunk then
+ setfenv(chunk, env);
+ end
+ return chunk, err;
+ end
+end
+
+local config_file = options.config or default_config;
+local from_store = arg[1] or "input";
+local to_store = arg[2] or "output";
+
config = {};
local config_env = setmetatable({}, { __index = function(t, k) return function(tbl) config[k] = tbl; end; end });
-loadfilein(options.config or "config.lua", config_env)();
+local config_chunk, err = loadfilein(config_file, config_env);
+if not config_chunk then
+ print("There was an error loading the config file, check the file exists");
+ print("and that the syntax is correct:");
+ print("", err);
+ os.exit(1);
+end
+
+config_chunk();
if not package.loaded["util.json"] then
package.path = "../../?.lua;"..package.path
package.cpath = "../../?.dll;"..package.cpath
end
-local from_store = arg[1] or "input";
-local to_store = arg[2] or "output";
+local have_err;
+if #arg > 0 and #arg ~= 2 then
+ have_err = true;
+ print("Error: Incorrect number of parameters supplied.");
+end
+if not config[from_store] then
+ have_err = true;
+ print("Error: Input store '"..from_store.."' not found in the config file.");
+end
+if not config[to_store] then
+ have_err = true;
+ print("Error: Output store '"..to_store.."' not found in the config file.");
+end
+if not config[from_store].type then
+ have_err = true;
+ print("Error: Input store type not specified in the config file");
+elseif not pcall(require, config[from_store].type) then
+ have_err = true;
+ print("Error: Unrecognised store type for '"..from_store.."': "..config[from_store].type);
+end
+if not config[to_store].type then
+ have_err = true;
+ print("Error: Output store type not specified in the config file");
+elseif not pcall(require, config[to_store].type) then
+ have_err = true;
+ print("Error: Unrecognised store type for '"..to_store.."': "..config[to_store].type);
+end
-assert(config[from_store], "no input specified")
-assert(config[to_store], "no output specified")
-local itype = assert(config[from_store].type, "no type specified for "..from_store);
-local otype = assert(config[to_store].type, "no type specified for "..to_store);
+if have_err then
+ print("");
+ print("Usage: "..arg[0].." FROM_STORE TO_STORE");
+ print("If no stores are specified, 'input' and 'output' are used.");
+ print("");
+ print("The available stores in your migrator config are:");
+ print("");
+ for store in pairs(config) do
+ print("", store);
+ end
+ os.exit(1);
+end
+
+local itype = config[from_store].type;
+local otype = config[to_store].type;
local reader = require(itype).reader(config[from_store]);
local writer = require(otype).writer(config[to_store]);
diff --git a/tools/migration/config.lua b/tools/migration/migrator.cfg.lua
index fa37f2a3..fa37f2a3 100644
--- a/tools/migration/config.lua
+++ b/tools/migration/migrator.cfg.lua