Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Execute as a script #276

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

dn2007hw
Copy link

@dn2007hw dn2007hw commented Apr 15, 2023

Script execution made simple with SML/NJ

Submitted By: Dayanandan Natarajan
Supervisor: Joe Wells
School of Mathematical and Computer Sciences
Heriot-Watt University

Objective

Primary objective of this change is to give the programmers and developers the capability of running a SML program as a script. Programmers and developers can write a SML program and run the program like a script over the command prompt.

Files modified

smlnj/base/cm/main/cm-boot.sml
smlnj/base/system/smlnj/internal/boot-env-fn.sml
smlnj/base/compiler/TopLevel/interact/interact.sig
smlnj/base/compiler/TopLevel/interact/interact.sml

Change details

  1. interact.sig & interact.sml

A new function (useScriptFile) is added to Backend.Interact structure, which takes the file name and its content as a stream and process the stream by passing it to EvalLoop.evalStream.

a) New function declaration is added to interact.sig,

val useStream : TextIO.instream -> unit
val useScriptFile : string * TextIO.instream -> unit (* Addded by DAYA *)
val evalStream : TextIO.instream * Environment.environment -> Environment.environment

b) New function definition is added to interact.sml,

    fun useScriptFile (fname, stream) = ( 

      (EvalLoop.evalStream (fname, stream))
        handle exn => ( 
          EvalLoop.uncaughtExnMessage exn
          )  
      )
  1. cm-boot.sml

The following changes were made in cm-boot.sml to recognise the new command line parameter passed from script.

a) In function args, line added to recognise the new command-line parameter ‘--script’, and a new function ‘nextargscript’ is called to initiate the process of the file.

        fun args ([], _) = ()
          | args ("-a" :: _, _) = nextarg autoload'
          | args ("-m" :: _, _) = nextarg make'
          | args (["-H"], _) = (help NONE; quit ())
          | args ("-H" :: _ :: _, mk) = (help NONE; nextarg mk)
          | args (["-S"], _) = (showcur NONE; quit ())
          | args ("-S" :: _ :: _, mk) = (showcur NONE; nextarg mk)
          | args (["-E"], _) = (show_envvars NONE; quit ())
          | args ("-E" :: _ :: _, mk) = (show_envvars NONE; nextarg mk)
          | args ("--script" :: _, _) = (nextargscript ())  (* line added by DAYA *)
          | args ("@CMbuild" :: rest, _) = mlbuild rest
          | args (["@CMredump", heapfile], _) = redump_heap heapfile
          | args (f :: rest, mk) =
          (carg (String.substring (f, 0, 2)
             handle General.Subscript => "",
             f, mk, List.null rest);
           nextarg mk)

        and nextarg mk =
        let val l = SMLofNJ.getArgs ()
        in SMLofNJ.shiftArgs (); args (l, mk)
        end

        (* nextargscript added by DAYA *)
        and nextargscript () =
        let val l = SMLofNJ.getArgs ()
        in SMLofNJ.shiftArgs (); processFileScript (hd l); quit ()
        end

b) In function init(), the new function (useScriptFile) is added as one of the parameter passed,

fun init (bootdir, de, er, useStream, useScriptFile, useFile, errorwrap, icm) = let

c) In function procCmdLine (), new function processFileScript is added to process the script file, function will check for whether the file passed on is a script file starting with ‘#!’ thru another new function checkSharpbang, consumes the first line thru another new function eatuntilneline and pass the remaining content of the file to function useScriptFile.

          (* DAYA change starts here *)
            fun eatuntilnewline (instream : TextIO.instream): bool = let
                val c = TextIO.input1 instream
                in
                    case TextIO.lookahead instream of
                        SOME #"\n" => true
                        | SOME c => eatuntilnewline instream
                        | NONE => false
                end

            fun checkSharpbang (instream : TextIO.instream): bool = let
                val c = TextIO.input1 instream
                in
                    case c of
                        SOME #"#" => (
                            case TextIO.lookahead instream of
                                SOME #"!" => eatuntilnewline instream
                                | SOME c => false
                                | NONE => false
                                )
                        | SOME c => false
                        | NONE => false
                end

            fun processFileScript (fname) = let
                val stream = TextIO.openIn fname
                val isscript = checkSharpbang stream
                in
                    if (isscript) = false  
                    then    ( Say.say [ "!* Script file doesn't start with #!. \n" ] ) 
                    else    ( useScriptFile (fname, stream) )
                end
            (* DAYA change ends here *)
  1. boot-env-fn.sml

In functor BootEnvF, cminit function declaration is amended to include the newly added function useScriptFile.

functor BootEnvF (datatype envrequest = AUTOLOAD | BARE
          val architecture: string
          val cminit : string * DynamicEnv.env * envrequest
                   * (TextIO.instream -> unit)(* useStream *)
                   * (string * TextIO.instream -> unit) (* useScriptFile *)
                   * (string -> unit) (* useFile *)
                   * ((string -> unit) -> (string -> unit))
                                      (* errorwrap *)
                   * ({ manageImport:
                      Ast.dec * EnvRef.envref -> unit,
                    managePrint:
                      Symbol.symbol * EnvRef.envref -> unit,
                    getPending : unit -> Symbol.symbol list }
                  -> unit)
                   -> (unit -> unit) option
          val cmbmake: string * bool -> unit) :> BOOTENV = struct

Writing a script

The script should start with ‘#!’ in the first line followed by the environment location, command-line parameters ‘-Ssml’ and ‘—script’, a new line and then followed by the SML code or program.

Example script named ‘sample’,
--------------beginning of the script-------------
#!/usr/bin/env -Ssml –script
;(--SML--)
val () = print "Hello World\n";
--------------end of the script---------------------

Running a script

The script ‘sample’ can be executed from Linux terminal or command prompt as a regular OS script as below provided it is given execution permission,

$ ./sample

SML/NJ Version used

Our development and testing is based on Standard ML of New Jersey (32-bit) v110.99.3 on an Intel based macOS 10.13.16.

Test Details

Test Script #1

#!/usr/bin/env -Ssml --script
;(* SML code starts here *)
val x = "Hello World x\n";
val () = print x;
val y = "Hello World y\n";
val () = print y;
val z = "Hello World z\n";
val () = print z;

Test Result #1

$ ./sample
Standard ML of New Jersey (32-bit) v110.99.3 [built: Mon Apr 10 18:03:19 2023]
val x = "Hello World x\n" : string
Hello World x
val y = "Hello World y\n" : string
Hello World y
val z = "Hello World z\n" : string
Hello World z
$

Test Script #2 (sample script with forced error)

#!/usr/bin/env -Ssml --script
;(--SML--)
val x = "Hello World x\n";
val () = print x;
val y = "Hello World y\n";
val () == print y;
val z = "Hello World z\n";
val () = print z;

Test Result #2
$ ./sample
Standard ML of New Jersey (32-bit) v110.99.3 [built: Sat Apr 15 18:38:28 2023]
val x = "Hello World x\n" : string
Hello World x
val y = "Hello World y\n" : string
./exml07:7.18-8.4 Error: syntax error: deleting SEMICOLON VAL

uncaught exception Compile [Compile: "syntax error"]
raised at: ../compiler/Parse/main/smlfile.sml:19.24-19.46
../compiler/TopLevel/interact/evalloop.sml:45.54
../compiler/TopLevel/interact/evalloop.sml:306.20-306.23

Changes made in cm-boot.sml to recognise the new command line parameter passed from script.

a)	In function args, line added to recognise the new command-line parameter ‘--script’, and a new function ‘nextargscript’ is called to initiate the process of the file.
b)	In function init(), the new function (useScriptFile) is added as one of the parameter passed.
c)	In function procCmdLine (), new function processFileScript is added to process the script file, function will check for whether the file passed on is a script file starting with ‘#!’ thru another new function checkSharpbang, consumes the first line thru another new function eatuntilneline and pass the remaining content of the file to function useScriptFile.
Changes made in cm-boot.sml to recognise the new command line parameter passed from script.

a)	In function args, line added to recognise the new command-line parameter ‘--script’, and a new function ‘nextargscript’ is called to initiate the process of the file.
b)	In function init(), the new function (useScriptFile) is added as one of the parameter passed.
c)	In function procCmdLine (), new function processFileScript is added to process the script file, function will check for whether the file passed on is a script file starting with ‘#!’ thru another new function checkSharpbang, consumes the first line thru another new function eatuntilneline and pass the remaining content of the file to function useScriptFile.
In functor BootEnvF, cminit function declaration is amended to include the newly added function useScriptFile.
A new function (useScriptFile) is added to Backend.Interact structure, which takes the file name and its content as a stream and process the stream by passing it to EvalLoop.evalStream. The compiler messages are muted and unmuted before the processing of the file.

a)	New function declaration is added to interact.sig,
b)	New function definition is added to interact.sml.
@dn2007hw dn2007hw changed the title Dn2007hw patch 1 execute as a script Execute as a script Apr 15, 2023
@dmacqueen
Copy link
Contributor

dmacqueen commented Apr 15, 2023

We need a fairly detailed description of what this does and how it works, and any new interfaces/signatures, and whether the top-level pervasive environment is changed. When an SML program is invoked as a "script", is it loaded and run in the REPL? Is the SML program that is run as a script an SML source file or is it a stand-alone program with an associated heap-image created by exportFn? In other words, is the goal to make the sml command behave like a Unix shell?

@dn2007hw
Copy link
Author

HI Dave, The pull request is still in draft, I will be adding all the details before I submit the request.

@dmacqueen
Copy link
Contributor

dmacqueen commented Apr 15, 2023 via email

@dmacqueen
Copy link
Contributor

dmacqueen commented Apr 15, 2023 via email

@dn2007hw
Copy link
Author

silenceCompiler function is not part of this patch, I have removed the reference. It's part of next patch, I will be adding the details in that.

@dn2007hw dn2007hw marked this pull request as ready for review April 16, 2023 23:44
@JohnReppy
Copy link
Contributor

We should think about the name of the command-line option. We currently do not use names like --script. I would suggest @SMLscript (using the @SML prefix is a way to avoid colliding with options that the script might want to handle).

Also, the code can be tightened up a bit and should be made more robust. For example, it might be more direct to process the first line of the script using TextIO.StreamIO.inputLine

(* if the first line of the input stream begins with "#!", then consume the line
 * and return `true`; otherwise return `false` and do not advance the input.
 *)
fun checkSharpbang inS = let
      val inS' = TextIO.getInstream inS
      in
        case TextIO.StreamIO.inputLine inS'
         of SOME(firstLn, inS'') =>
              if String.isPrefix "#!" firstLn
                then (TextIO.setInstream(inS, inS''); true)
                else false
          | _ => false
        (* end case *)
      end

JohnReppy added a commit that referenced this pull request Nov 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants