Update clap and fix regression.

Author: ptondereau

crates/cli/Cargo.toml

@@ -13,7 +13,7 @@ categories = ["api-bindings", "command-line-interface"]
 [dependencies]
 ext-php-rs = { version = ">=0.7.1", path = "../../" }
 
-clap = { version = "3.2", features = ["derive"] }
+clap = { version = ">=3.2.5", features = ["derive"] }
 anyhow = "1"
 dialoguer = "0.10"
 libloading = "0.7"

crates/cli/src/lib.rs

@@ -89,21 +89,21 @@ enum Args {
 struct Install {
     /// Changes the path that the extension is copied to. This will not
     /// activate the extension unless `ini_path` is also passed.
-    #[clap(long, value_parser)]
+    #[clap(long)]
     install_dir: Option<PathBuf>,
     /// Path to the `php.ini` file to update with the new extension.
-    #[clap(long, value_parser)]
+    #[clap(long)]
     ini_path: Option<PathBuf>,
     /// Installs the extension but doesn't enable the extension in the `php.ini`
     /// file.
-    #[clap(long, value_parser)]
+    #[clap(long)]
     disable: bool,
     /// Whether to install the release version of the extension.
-    #[clap(long, value_parser)]
+    #[clap(long)]
     release: bool,
     /// Path to the Cargo manifest of the extension. Defaults to the manifest in
     /// the directory the command is called.
-    #[clap(long, value_parser)]
+    #[clap(long)]
     manifest: Option<PathBuf>,
 }
 
@@ -112,14 +112,14 @@ struct Remove {
     /// Changes the path that the extension will be removed from. This will not
     /// remove the extension from a configuration file unless `ini_path` is also
     /// passed.
-    #[clap(long, value_parser)]
+    #[clap(long)]
     install_dir: Option<PathBuf>,
     /// Path to the `php.ini` file to remove the extension from.
-    #[clap(long, value_parser)]
+    #[clap(long)]
     ini_path: Option<PathBuf>,
     /// Path to the Cargo manifest of the extension. Defaults to the manifest in
     /// the directory the command is called.
-    #[clap(long, value_parser)]
+    #[clap(long)]
     manifest: Option<PathBuf>,
 }
 
@@ -128,22 +128,22 @@ struct Remove {
 struct Stubs {
     /// Path to extension to generate stubs for. Defaults for searching the
     /// directory the executable is located in.
-    #[clap(short, long, value_parser)]
+    #[clap(short, long)]
     ext: Option<PathBuf>,
     /// Path used to store generated stub file. Defaults to writing to
     /// `<ext-name>.stubs.php` in the current directory.
-    #[clap(short, long, value_parser)]
+    #[clap(short, long)]
     out: Option<PathBuf>,
     /// Print stubs to stdout rather than write to file. Cannot be used with
     /// `out`.
-    #[clap(long, conflicts_with = "out", value_parser)]
+    #[clap(long, conflicts_with = "out")]
     stdout: bool,
     /// Path to the Cargo manifest of the extension. Defaults to the manifest in
     /// the directory the command is called.
     ///
     /// This cannot be provided alongside the `ext` option, as that option
     /// provides a direct path to the extension shared library.
-    #[clap(long, conflicts_with = "ext", value_parser)]
+    #[clap(long, conflicts_with = "ext")]
     manifest: Option<PathBuf>,
 }