downstream rescript changes

Author: jfrolich

src/build.rs

@@ -72,7 +72,7 @@ pub fn get_compiler_args(
         rescript_version
     } else {
         let bsc_path = match bsc_path {
-            Some(bsc_path) => bsc_path,
+            Some(bsc_path) => helpers::get_abs_path(&bsc_path),
             None => helpers::get_bsc(&package_root, workspace_root.to_owned()),
         };
         helpers::get_rescript_version(&bsc_path)

src/build/clean.rs

@@ -351,7 +351,7 @@ pub fn clean(path: &str, show_progress: bool, bsc_path: Option<String>) -> Resul
     )?;
     let root_config_name = packages::read_package_name(&project_root)?;
     let bsc_path = match bsc_path {
-        Some(bsc_path) => bsc_path,
+        Some(bsc_path) => helpers::get_abs_path(&bsc_path),
         None => helpers::get_bsc(&project_root, workspace_root.to_owned()),
     };
 

src/main.rs

@@ -149,6 +149,7 @@ fn main() -> Result<()> {
                     args.after_build,
                     args.create_sourcedirs,
                     args.dev,
+                    args.bsc_path,
                 );
 
                 Ok(())

src/watcher.rs

@@ -54,9 +54,18 @@ async fn async_watch(
     after_build: Option<String>,
     create_sourcedirs: bool,
     build_dev_deps: bool,
+    bsc_path: Option<String>,
 ) -> notify::Result<()> {
-    let mut build_state = build::initialize_build(None, filter, show_progress, path, None, build_dev_deps)
-        .expect("Can't initialize build");
+    let mut build_state = build::initialize_build(
+        None,
+        filter,
+        show_progress,
+        path,
+        None,
+        build_dev_deps,
+        bsc_path.clone(),
+    )
+    .expect("Can't initialize build");
     let mut needs_compile_type = CompileType::Incremental;
     // create a mutex to capture if ctrl-c was pressed
     let ctrlc_pressed = Arc::new(Mutex::new(false));
@@ -91,6 +100,20 @@ async fn async_watch(
         }
 
         for event in events {
+            // if there is a file named rewatch.lock in the events path, we can quit the watcher
+            if let Some(_) = event.paths.iter().find(|path| path.ends_with("rewatch.lock")) {
+                match event.kind {
+                    EventKind::Remove(_) => {
+                        if show_progress {
+                            println!("\nExiting... (lockfile removed)");
+                        }
+                        clean::cleanup_after_build(&build_state);
+                        return Ok(());
+                    }
+                    _ => (),
+                }
+            }
+
             let paths = event
                 .paths
                 .iter()
@@ -214,9 +237,8 @@ async fn async_watch(
             }
             CompileType::Full => {
                 let timing_total = Instant::now();
-                build_state =
-                    build::initialize_build(None, filter, show_progress, path, None, build_dev_deps)
-                        .expect("Can't initialize build");
+                build_state = build::initialize_build(None, filter, show_progress, path, None)
+                    .expect("Can't initialize build");
                 let _ = build::incremental_build(
                     &mut build_state,
                     None,
@@ -260,6 +282,7 @@ pub fn start(
     after_build: Option<String>,
     create_sourcedirs: bool,
     build_dev_deps: bool,
+    bsc_path: Option<String>,
 ) {
     futures::executor::block_on(async {
         let queue = Arc::new(FifoQueue::<Result<Event, Error>>::new());
@@ -280,6 +303,7 @@ pub fn start(
             after_build,
             create_sourcedirs,
             build_dev_deps,
+            bsc_path,
         )
         .await
         {

testrepo/package.json

@@ -11,6 +11,9 @@
       "packages/with-dev-deps"
     ]
   },
+  "dependencies": {
+    "rescript": "11.1.4"
+  },
   "scripts": {
     "build": "../target/release/rewatch build .",
     "build:rescript": "rescript build -with-deps",

testrepo/packages/dep01/package.json

@@ -12,7 +12,6 @@
   "author": "",
   "license": "MIT",
   "dependencies": {
-    "@testrepo/dep02": "*",
-    "rescript": "*"
+    "@testrepo/dep02": "*"
   }
 }

testrepo/packages/dep02/package.json

@@ -10,8 +10,5 @@
     "rescript"
   ],
   "author": "",
-  "license": "MIT",
-  "dependencies": {
-    "rescript": "*"
-  }
+  "license": "MIT"
 }

testrepo/packages/main/package.json

@@ -12,7 +12,6 @@
   "author": "",
   "license": "MIT",
   "dependencies": {
-    "@testrepo/dep01": "*",
-    "rescript": "*"
+    "@testrepo/dep01": "*"
   }
 }

testrepo/packages/new-namespace/package.json

@@ -5,8 +5,5 @@
     "rescript"
   ],
   "author": "",
-  "license": "MIT",
-  "dependencies": {
-    "rescript": "*"
-  }
+  "license": "MIT"
 }

testrepo/yarn.lock

@@ -7,7 +7,7 @@
   resolved "https://registry.yarnpkg.com/@rescript/core/-/core-1.6.1.tgz#159670c94d64a2b8236f46be2bf09a007b1ece08"
   integrity sha512-vyb5k90ck+65Fgui+5vCja/mUfzKaK3kOPT4Z6aAJdHLH1eljEi1zKhXroCiCtpNLSWp8k4ulh1bdB5WS0hvqA==
 
-rescript@*:
+rescript@*, [email protected]:
   version "11.1.4"
   resolved "https://registry.yarnpkg.com/rescript/-/rescript-11.1.4.tgz#9a42ebc4fc5363707e39cef5b3188160b63bee42"
   integrity sha512-0bGU0bocihjSC6MsE3TMjHjY0EUpchyrREquLS8VsZ3ohSMD+VHUEwimEfB3kpBI1vYkw3UFZ3WD8R28guz/Vw==