src/db/Cargo.toml

@@ -11,6 +11,11 @@ readme = "../../README.md"
 [lib]
 path = "lib.rs"
 
+[[bin]]
+name = "convert"
+path = "bin/convert.rs"
+required-features = ["cli"]
+
 [dependencies]
 err-derive = "0.3"
 hexdump = "0.1"

src/db/bin/convert.rs

@@ -1,30 +1,39 @@
 use std::path::PathBuf;
 
-use structopt::StructOpt;
-
 use garage_db::*;
 
+use clap::Parser;
+
 /// K2V command line interface
-#[derive(StructOpt, Debug)]
+#[derive(Parser, Debug)]
-pub struct ConvertDbOpt {
+#[clap(author, version, about, long_about = None)]
-	/// Input database path (not the same as metadata_dir, see
+struct Args {
-	/// https://garagehq.deuxfleurs.fr/documentation/reference-manual/configuration/#db-engine-since-v0-8-0)
+	/// Input DB path
-	#[structopt(short = "i")]
+	#[clap(short = 'i')]
 	input_path: PathBuf,
-	/// Input database engine (sled, lmdb or sqlite; limited by db engines
+	/// Input DB engine
-	/// enabled in this build)
+	#[clap(short = 'a')]
-	#[structopt(short = "a")]
 	input_engine: String,
 
-	/// Output database path
+	/// Output DB path
-	#[structopt(short = "o")]
+	#[clap(short = 'o')]
 	output_path: PathBuf,
-	/// Output database engine
+	/// Output DB engine
-	#[structopt(short = "b")]
+	#[clap(short = 'b')]
 	output_engine: String,
 }
 
-pub(crate) fn do_conversion(args: ConvertDbOpt) -> Result<()> {
+fn main() {
+	let args = Args::parse();
+	pretty_env_logger::init();
+
+	match do_conversion(args) {
+		Ok(()) => println!("Success!"),
+		Err(e) => eprintln!("Error: {}", e),
+	}
+}
+
+fn do_conversion(args: Args) -> Result<()> {
 	let input = open_db(args.input_path, args.input_engine)?;
 	let output = open_db(args.output_path, args.output_engine)?;
 	output.import(&input)?;
@@ -33,19 +42,16 @@ pub(crate) fn do_conversion(args: ConvertDbOpt) -> Result<()> {
 
 fn open_db(path: PathBuf, engine: String) -> Result<Db> {
 	match engine.as_str() {
-		#[cfg(feature = "sled")]
 		"sled" => {
 			let db = sled_adapter::sled::Config::default().path(&path).open()?;
 			Ok(sled_adapter::SledDb::init(db))
 		}
-		#[cfg(feature = "sqlite")]
 		"sqlite" | "sqlite3" | "rusqlite" => {
 			let db = sqlite_adapter::rusqlite::Connection::open(&path)?;
 			db.pragma_update(None, "journal_mode", &"WAL")?;
 			db.pragma_update(None, "synchronous", &"NORMAL")?;
 			Ok(sqlite_adapter::SqliteDb::init(db))
 		}
-		#[cfg(feature = "lmdb")]
 		"lmdb" | "heed" => {
 			std::fs::create_dir_all(&path).map_err(|e| {
 				Error(format!("Unable to create LMDB data directory: {}", e).into())
@@ -62,8 +68,6 @@ fn open_db(path: PathBuf, engine: String) -> Result<Db> {
 			let db = env_builder.open(&path)?;
 			Ok(lmdb_adapter::LmdbDb::init(db))
 		}
-		e => Err(Error(
+		e => Err(Error(format!("Invalid DB engine: {}", e).into())),
-			format!("Invalid or unsupported DB engine: {}", e).into(),
-		)),
 	}
 }

src/garage/cli/mod.rs

@@ -4,8 +4,6 @@ pub(crate) mod layout;
 pub(crate) mod structs;
 pub(crate) mod util;
 
-pub(crate) mod convert_db;
-
 pub(crate) use cmd::*;
 pub(crate) use init::*;
 pub(crate) use layout::*;

src/garage/cli/structs.rs

@@ -3,8 +3,6 @@ use structopt::StructOpt;
 
 use garage_util::version::garage_version;
 
-use crate::cli::convert_db;
-
 #[derive(StructOpt, Debug)]
 pub enum Command {
 	/// Run Garage server
@@ -56,10 +54,6 @@ pub enum Command {
 	/// Low-level debug operations on data blocks
 	#[structopt(name = "block", version = garage_version())]
 	Block(BlockOperation),
-
-	/// Convert metadata db between database engine formats
-	#[structopt(name = "convert-db", version = garage_version())]
-	ConvertDb(convert_db::ConvertDbOpt),
 }
 
 #[derive(StructOpt, Debug)]

src/garage/main.rs

@@ -176,9 +176,6 @@ async fn main() {
 		Command::OfflineRepair(repair_opt) => {
 			repair::offline::offline_repair(opt.config_file, opt.secrets, repair_opt).await
 		}
-		Command::ConvertDb(conv_opt) => {
-			cli::convert_db::do_conversion(conv_opt).map_err(From::from)
-		}
 		Command::Node(NodeOperation::NodeId(node_id_opt)) => {
 			node_id_command(opt.config_file, node_id_opt.quiet)
 		}