remove --mod option

This was only used for the build subcommand, but all subcommands potentially
need to know where the go module is e.g. to locate `componentize-go.toml` files,
so we should either promote it to be a subcommand-independent option or drop it
and require that `componentize-go` be run in the directory containing the module
of interest.  I chose the latter in this case, but we could certainly revisit
that.

Author: dicej

src/cmd_build.rs

@@ -7,7 +7,6 @@ use std::{path::PathBuf, process::Command};
 /// If the module is not going to be adapted to the component model,
 /// set the `only_wasip1` arg to true.
 pub fn build_module(
-    go_module: Option<&PathBuf>,
     out: Option<&PathBuf>,
     go_path: Option<&PathBuf>,
     only_wasip1: bool,
@@ -33,22 +32,11 @@ pub fn build_module(
         .to_str()
         .ok_or_else(|| anyhow!("Output path is not valid unicode"))?;
 
-    let module_path = match &go_module {
-        Some(p) => {
-            if !p.is_dir() {
-                return Err(anyhow!("Module path '{}' is not a directory", p.display()));
-            }
-            p.to_str()
-                .ok_or_else(|| anyhow!("Module path is not valid unicode"))?
-        }
-        None => ".",
-    };
-
     // The -buildmode flag mutes the module's output, so it is ommitted
     let module_args = [
         "build",
         "-C",
-        module_path,
+        ".",
         "-ldflags=-checklinkname=0",
         "-o",
         out_path,
@@ -57,7 +45,7 @@ pub fn build_module(
     let component_args = [
         "build",
         "-C",
-        module_path,
+        ".",
         "-buildmode=c-shared",
         "-ldflags=-checklinkname=0",
         "-o",

src/command.rs

@@ -91,10 +91,6 @@ pub struct Build {
     #[arg(long, short = 'o')]
     pub output: Option<PathBuf>,
 
-    /// The directory containing the "go.mod" file (or current directory if `None`).
-    #[arg(long = "mod")]
-    pub mod_path: Option<PathBuf>,
-
     /// The path to the Go binary (or look for binary in PATH if `None`).
     #[arg(long)]
     pub go: Option<PathBuf>,
@@ -170,12 +166,7 @@ pub fn run<T: Into<OsString> + Clone, I: IntoIterator<Item = T>>(args: I) -> Res
 
 fn build(wit_opts: WitOpts, build: Build) -> Result<()> {
     // Build a wasm module using `go build`.
-    let module = build_module(
-        build.mod_path.as_ref(),
-        build.output.as_ref(),
-        build.go.as_ref(),
-        build.wasip1,
-    )?;
+    let module = build_module(build.output.as_ref(), build.go.as_ref(), build.wasip1)?;
 
     if !build.wasip1 {
         // Embed the WIT documents in the wasip1 component.