From 37639d68d740a887a5aec58c79441fb6b10546f0 Mon Sep 17 00:00:00 2001
From: Casey Rodarmor <casey@rodarmor.com>
Date: Mon, 8 Apr 2019 14:28:17 -0700
Subject: [PATCH] Allow shebang lines so justfiles can be used as scripts
 (#393)

---
 README.adoc              | 21 +++++++++++++++++++++
 justfile                 |  4 ++++
 src/compilation_error.rs |  4 ----
 src/lexer.rs             | 16 ++--------------
 tests/integration.rs     | 22 ----------------------
 5 files changed, 27 insertions(+), 40 deletions(-)
 mode change 100644 => 100755 justfile

diff --git a/README.adoc b/README.adoc
index ba48c28..fa2b7b4 100644
--- a/README.adoc
+++ b/README.adoc
@@ -653,6 +653,27 @@ $ just foo/build
 $ just foo/
 ```
 
+=== Just Scripts
+
+By adding a shebang line to the top of a justfile and making it executable, `just` can be used as an interpreter for scripts:
+
+```sh
+$ cat > script <<EOF
+#!/usr/bin/env just --justfile
+
+foo:
+  echo foo
+EOF
+$ chmod +x script
+$ ./script foo
+echo foo
+foo
+```
+
+When a script with a shebang is executed, the system supplies the path to the script as an argument to the command in the shebang. So, with a shebang of `#!/usr/bin/env just --justfile`, the command will be `/usr/bin/env just --justfile PATH_TO_SCRIPT`.
+
+With the above shebang, `just` will change its working directory to the location of the script. If you'd rather leave the working directory unchanged, use `#!/usr/bin/env just --working-directory . --justfile`.
+
 == Frequently Asked Questions
 
 === What are the idiosyncrasies of make that just avoids?
diff --git a/justfile b/justfile
old mode 100644
new mode 100755
index fe9b3c1..8d5d909
--- a/justfile
+++ b/justfile
@@ -1,3 +1,7 @@
+#!/usr/bin/env just --justfile
+# ^ A shebang isn't required, but allows a justfile to be executed
+#   like a script, with `./justfile test`, for example.
+
 bt='0'
 
 export RUST_BACKTRACE=bt
diff --git a/src/compilation_error.rs b/src/compilation_error.rs
index 712ee34..01383bf 100644
--- a/src/compilation_error.rs
+++ b/src/compilation_error.rs
@@ -62,7 +62,6 @@ pub enum CompilationErrorKind<'a> {
   MixedLeadingWhitespace {
     whitespace: &'a str,
   },
-  OuterShebang,
   ParameterFollowsVariadicParameter {
     parameter: &'a str,
   },
@@ -229,9 +228,6 @@ impl<'a> Display for CompilationError<'a> {
           show_whitespace(found)
         )?;
       }
-      OuterShebang => {
-        writeln!(f, "`#!` is reserved syntax outside of recipes")?;
-      }
       UnknownDependency { recipe, unknown } => {
         writeln!(
           f,
diff --git a/src/lexer.rs b/src/lexer.rs
index 018cb74..abc2ea0 100644
--- a/src/lexer.rs
+++ b/src/lexer.rs
@@ -135,7 +135,7 @@ impl<'a> Lexer<'a> {
       static ref BACKTICK: Regex = token(r"`[^`\n\r]*`");
       static ref COLON: Regex = token(r":");
       static ref COMMA: Regex = token(r",");
-      static ref COMMENT: Regex = token(r"#([^!\n\r][^\n\r]*)?\r?$");
+      static ref COMMENT: Regex = token(r"#([^\n\r][^\n\r]*)?\r?$");
       static ref EOF: Regex = token(r"\z");
       static ref EOL: Regex = token(r"\n|\r\n");
       static ref EQUALS: Regex = token(r"=");
@@ -322,8 +322,6 @@ impl<'a> Lexer<'a> {
           return Err(self.error(UnterminatedString));
         }
         (prefix, &self.rest[start..content_end + 1], StringToken)
-      } else if self.rest.starts_with("#!") {
-        return Err(self.error(OuterShebang));
       } else {
         return Err(self.error(UnknownStartOfToken));
       };
@@ -340,7 +338,7 @@ impl<'a> Lexer<'a> {
           _ => {
             return Err(last.error(Internal {
               message: format!("zero length token: {:?}", last),
-            }))
+            }));
           }
         }
       }
@@ -647,16 +645,6 @@ c: b
     kind:   InconsistentLeadingWhitespace{expected: "\t\t", found: "\t  "},
   }
 
-  error_test! {
-    name: tokenize_outer_shebang,
-    input: "#!/usr/bin/env bash",
-    index:  0,
-    line:   0,
-    column: 0,
-    width:  None,
-    kind:   OuterShebang,
-  }
-
   error_test! {
     name: tokenize_unknown,
     input: "~",
diff --git a/tests/integration.rs b/tests/integration.rs
index 400b40e..044c848 100644
--- a/tests/integration.rs
+++ b/tests/integration.rs
@@ -578,28 +578,6 @@ wut:
   status:   EXIT_SUCCESS,
 }
 
-integration_test! {
-  name:     outer_shebang,
-  justfile: r#"#!/lol/wut
-export FOO = "a"
-baz = "c"
-export BAR = "b"
-export ABC = FOO + BAR + baz
-
-wut:
-  #!/bin/sh
-  echo $FOO $BAR $ABC
-"#,
-  args:     (),
-  stdout:   "",
-  stderr:   "error: `#!` is reserved syntax outside of recipes
-  |
-1 | #!/lol/wut
-  | ^
-",
-  status:   EXIT_FAILURE,
-}
-
 integration_test! {
   name:     export_shebang,
   justfile: r#"