[compiler-v2 framework] fixed mpm opt parse error

starcoinorg · Nov 7, 2024 · eb53139 · eb53139
1 parent 173f978
commit eb53139
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/vm/move-package-manager/src/main.rs b/vm/move-package-manager/src/main.rs
@@ -3,9 +3,7 @@
 
 use anyhow::Result;
 use clap::Parser;
-// use move_cli::package::cli::handle_package_commands;
 use move_cli::Move;
-use move_core_types::errmap::ErrorMapping;
 use move_package_manager::compatibility_check_cmd::{
     handle_compatibility_check, CompatibilityCheckCommand,
 };
@@ -54,8 +52,8 @@ pub enum Commands {
 }
 
 fn main() -> Result<()> {
-    let _error_descriptions: ErrorMapping =
-        bcs_ext::from_bytes(stdlib::ERROR_DESCRIPTIONS).expect("Decode err map failed");
+    // let _error_descriptions: ErrorMapping =
+    //     bcs_ext::from_bytes(stdlib::ERROR_DESCRIPTIONS).expect("Decode err map failed");
     let args: CliOptions = CliOptions::parse();
 
     let move_args = &args.move_args;

diff --git a/vm/move-package-manager/src/release.rs b/vm/move-package-manager/src/release.rs
@@ -24,12 +24,12 @@ pub const DEFAULT_RELEASE_DIR: &str = "release";
 
 #[derive(Parser)]
 pub struct Release {
-    #[arg(name = "move-version", long = "move-version", default_value="6", value_parser = clap::builder::PossibleValuesParser::new(["5", "6"]))]
+    #[arg(name = "move-version", long = "move-version", default_value_t = 6)]
     /// specify the move lang version for the release.
     /// currently, only v6 are supported.
     language_version: u8,
 
-    #[arg(name="release-dir", long, value_parser = value_parser!(std::ffi::OsString), default_value=DEFAULT_RELEASE_DIR)]
+    #[arg(name="release-dir", long, value_parser = value_parser!(std::path::PathBuf), default_value=DEFAULT_RELEASE_DIR)]
     /// dir to store released blob
     release_dir: PathBuf,