Allow overriding target directory

scarb/src/bin/scarb/args.rs

@@ -36,6 +36,15 @@ pub struct ScarbArgs {
     #[arg(long, env = "SCARB_OFFLINE", global = true)]
     pub offline: bool,
 
+    /// Directory for all generated artifacts.
+    #[arg(
+        long,
+        env = "SCARB_TARGET_DIR",
+        value_name = "DIRECTORY",
+        global = true
+    )]
+    pub target_dir: Option<Utf8PathBuf>,
+
     /// Subcommand and its arguments.
     #[command(subcommand)]
     pub command: Command,

scarb/src/bin/scarb/main.rs

@@ -41,7 +41,7 @@ fn cli_main(args: ScarbArgs) -> Result<()> {
     let ui = Ui::new(args.ui_verbosity(), args.output_format());
 
     let manifest_path = ops::find_manifest_path(args.manifest_path.as_deref())?;
-    let mut config = Config::init(manifest_path, dirs, ui)?;
+    let mut config = Config::init(manifest_path, dirs, ui, args.target_dir)?;
     config.set_offline(args.offline);
     commands::run(args.command, &mut config)
 }

scarb/src/core/config.rs

@@ -32,7 +32,12 @@ pub struct Config {
 }
 
 impl Config {
-    pub fn init(manifest_path: Utf8PathBuf, dirs: AppDirs, ui: Ui) -> Result<Self> {
+    pub fn init(
+        manifest_path: Utf8PathBuf,
+        dirs: AppDirs,
+        ui: Ui,
+        target_dir_override: Option<Utf8PathBuf>,
+    ) -> Result<Self> {
         let creation_time = Instant::now();
 
         if tracing::enabled!(tracing::Level::TRACE) {
@@ -41,12 +46,12 @@ impl Config {
             }
         }
 
-        let target_dir = RootFilesystem::new_output_dir(
+        let target_dir = RootFilesystem::new_output_dir(target_dir_override.unwrap_or_else(|| {
             manifest_path
                 .parent()
                 .expect("parent of manifest path must always exist")
-                .join(DEFAULT_TARGET_DIR_NAME),
-        );
+                .join(DEFAULT_TARGET_DIR_NAME)
+        }));
 
         let dirs = Arc::new(dirs);
 

scarb/src/ops/subcommands.rs

@@ -26,6 +26,7 @@ pub fn execute_external_subcommand(cmd: &str, args: &[&OsStr], config: &Config)
     cmd.env("PATH", config.dirs().path_env());
     cmd.env("SCARB_CACHE", config.dirs().cache_dir.path_unchecked());
     cmd.env("SCARB_CONFIG", config.dirs().config_dir.path_unchecked());
+    cmd.env("SCARB_TARGET_DIR", config.target_dir().path_unchecked());
     cmd.env("SCARB_MANIFEST_DIR", config.root());
     cmd.env("SCARB_UI_VERBOSITY", config.ui().verbosity().to_string());
     cmd.env("SCARB_LOG", config.scarb_log());

scarb/tests/e2e/build.rs

@@ -516,3 +516,24 @@ fn compile_with_custom_lib_target() {
     t.child("target/release/hello.casm")
         .assert(predicates::path::exists().not());
 }
+
+#[test]
+fn override_target_dir() {
+    let target_dir = assert_fs::TempDir::new().unwrap();
+
+    let t = assert_fs::TempDir::new().unwrap();
+    ProjectBuilder::start().name("hello").build(&t);
+
+    Scarb::quick_snapbox()
+        .arg("--target-dir")
+        .arg(target_dir.path())
+        .arg("build")
+        .current_dir(&t)
+        .assert()
+        .success();
+
+    t.child("target").assert(predicates::path::exists().not());
+    target_dir
+        .child("release/hello.sierra")
+        .assert(predicates::path::exists());
+}

scarb/tests/e2e/subcommand.rs

@@ -54,6 +54,7 @@ fn env_variables_are_passed() {
                 SCARB
                 SCARB_CACHE
                 SCARB_CONFIG
+                SCARB_TARGET_DIR
                 SCARB_MANIFEST_DIR
                 SCARB_UI_VERBOSITY
             )

scarb/tests/e2e/support/command.rs

@@ -60,6 +60,7 @@ impl Scarb {
                 path_dirs: Vec::new(),
             },
             Ui::new(Verbosity::Verbose, OutputFormat::Text),
+            None,
         )
         .unwrap()
     }