[WIP] Introduce an All target for outputting all supported module types by wasm-pack

src/bindgen.rs

@@ -20,6 +20,93 @@ pub fn wasm_bindgen_build(
     disable_dts: bool,
     target: Target,
     profile: BuildProfile,
+) -> Result<(), failure::Error> {
+    match target {
+        Target::All => {
+            // Bundler
+            // NOTE: We only generate our type definitions here if we want them
+            let mut out_name_for_bundler = Some(format!("{}_esm", data.crate_name()));
+            if let Some(value) = out_name {
+                out_name_for_bundler = Some(format!("{}_esm", value));
+            }
+            run_wasm_bindgen(
+                data,
+                bindgen,
+                out_dir,
+                &out_name_for_bundler,
+                disable_dts,
+                Target::Bundler,
+                profile,
+            )?;
+
+            // Web
+            let mut out_name_for_web = Some(format!("{}_web", data.crate_name()));
+            if let Some(value) = out_name {
+                out_name_for_web = Some(format!("{}_web", value));
+            }
+            run_wasm_bindgen(
+                data,
+                bindgen,
+                out_dir,
+                &out_name_for_web,
+                disable_dts,
+                Target::Web,
+                profile,
+            )?;
+
+            // Nodejs
+            let mut out_name_for_nodejs = Some(format!("{}_cjs", data.crate_name()));
+            if let Some(value) = out_name {
+                out_name_for_nodejs = Some(format!("{}_cjs", value));
+            }
+            run_wasm_bindgen(
+                data,
+                bindgen,
+                out_dir,
+                &out_name_for_nodejs,
+                true,
+                Target::Nodejs,
+                profile,
+            )?;
+
+            // NoModules
+            let mut out_name_for_nomodules = Some(format!("{}_browser", data.crate_name()));
+            if let Some(value) = out_name {
+                out_name_for_nomodules = Some(format!("{}_browser", value));
+            }
+            run_wasm_bindgen(
+                data,
+                bindgen,
+                out_dir,
+                &out_name_for_nomodules,
+                disable_dts,
+                Target::NoModules,
+                profile,
+            )?;
+        }
+        _ => {
+            run_wasm_bindgen(
+                data,
+                bindgen,
+                out_dir,
+                out_name,
+                disable_dts,
+                target,
+                profile,
+            )?;
+        }
+    }
+    Ok(())
+}
+
+fn run_wasm_bindgen(
+    data: &CrateData,
+    bindgen: &Download,
+    out_dir: &Path,
+    out_name: &Option<String>,
+    disable_dts: bool,
+    target: Target,
+    profile: BuildProfile,
 ) -> Result<(), failure::Error> {
     let release_or_debug = match profile {
         BuildProfile::Release | BuildProfile::Profiling => "release",
@@ -115,6 +202,7 @@ fn build_target_arg_legacy(target: Target, cli_path: &PathBuf) -> Result<String,
             }
         }
         Target::Bundler => "--browser",
+        _ => bail!(format!("The passed target, {}, is not supported", target)),
     };
     Ok(target_arg.to_string())
 }

src/command/build.rs

@@ -54,6 +54,8 @@ pub enum Target {
     /// in a browser but pollutes the global namespace and must be manually
     /// instantiated.
     NoModules,
+    /// Correspond to `--target all` where the output is all other targets
+    All,
 }
 
 impl Default for Target {
@@ -69,6 +71,7 @@ impl fmt::Display for Target {
             Target::Web => "web",
             Target::Nodejs => "nodejs",
             Target::NoModules => "no-modules",
+            Target::All => "all",
         };
         write!(f, "{}", s)
     }
@@ -82,6 +85,7 @@ impl FromStr for Target {
             "web" => Ok(Target::Web),
             "nodejs" => Ok(Target::Nodejs),
             "no-modules" => Ok(Target::NoModules),
+            "all" => Ok(Target::All),
             _ => bail!("Unknown target: {}", s),
         }
     }
@@ -120,7 +124,7 @@ pub struct BuildOptions {
     pub disable_dts: bool,
 
     #[structopt(long = "target", short = "t", default_value = "bundler")]
-    /// Sets the target environment. [possible values: bundler, nodejs, web, no-modules]
+    /// Sets the target environment. [possible values: bundler, nodejs, web, no-modules, all]
     pub target: Target,
 
     #[structopt(long = "debug")]