From 35a339b1d3defc1be8387324a601db4889096a22 Mon Sep 17 00:00:00 2001
From: Matthew Wild <mwild1@gmail.com>
Date: Fri, 25 Feb 2011 03:32:44 +0000
Subject: tools/migration/*.lua: Rename config to migrator.cfg.lua, add error
 handling for config and command-line parameters

---
 tools/migration/config.lua       | 26 --------------
 tools/migration/main.lua         | 77 +++++++++++++++++++++++++++++++++++-----
 tools/migration/migrator.cfg.lua | 26 ++++++++++++++
 3 files changed, 95 insertions(+), 34 deletions(-)
 delete mode 100644 tools/migration/config.lua
 create mode 100644 tools/migration/migrator.cfg.lua

diff --git a/tools/migration/config.lua b/tools/migration/config.lua
deleted file mode 100644
index fa37f2a3..00000000
--- a/tools/migration/config.lua
+++ /dev/null
@@ -1,26 +0,0 @@
-local data_path = "../../data";
-
-input {
-	type = "prosody_files";
-	path = data_path;
-}
-
-output {
-	type = "prosody_sql";
-	driver = "SQLite3";
-	database = data_path.."/prosody.sqlite";
-}
-
---[[
-
-input {
-	type = "prosody_files";
-	path = data_path;
-}
-output {
-	type = "prosody_sql";
-	driver = "SQLite3";
-	database = data_path.."/prosody.sqlite";
-}
-
-]]
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/migrator.cfg.lua b/tools/migration/migrator.cfg.lua
new file mode 100644
index 00000000..fa37f2a3
--- /dev/null
+++ b/tools/migration/migrator.cfg.lua
@@ -0,0 +1,26 @@
+local data_path = "../../data";
+
+input {
+	type = "prosody_files";
+	path = data_path;
+}
+
+output {
+	type = "prosody_sql";
+	driver = "SQLite3";
+	database = data_path.."/prosody.sqlite";
+}
+
+--[[
+
+input {
+	type = "prosody_files";
+	path = data_path;
+}
+output {
+	type = "prosody_sql";
+	driver = "SQLite3";
+	database = data_path.."/prosody.sqlite";
+}
+
+]]
-- 
cgit v1.2.3