smlnj / legacy

This project is the old version of Standard ML of New Jersey that continues to support older systems (e.g., 32-bit machines).
BSD 3-Clause "New" or "Revised" License
25 stars 10 forks source link

Starting SML/NJ on Windows produces unsightly REM lines #294

Closed kcrary closed 5 months ago

kcrary commented 5 months ago

Version

110.99.4 (Latest)

Operating System

OS Version

Windows 10

Processor

System Component

Core system

Severity

Cosmetic

Description

When starting SML/NJ on Windows, you see a bunch of comment lines:

The solution is to move the @echo OFF line in bin/sml.bat to the top.

Transcript

Microsoft Windows [Version 10.0.22621.2861]
(c) Microsoft Corporation. All rights reserved.

c:\crary>sml c:/crary/lib/sml/misc.sml

c:\crary>REM sml.bat 

c:\crary>REM

c:\crary>REM Copyright 2020 The Fellowship of SML/NJ (http://www.smlnj.org) 

c:\crary>REM All rights reserved. 

c:\crary>REM

c:\crary>REM The standard driver for SML/NJ under the new runtime system 

c:\crary>REM
Standard ML of New Jersey (32-bit) v110.99.3 [built: Thu Jul 28 00:45:02 2022]

Expected Behavior

Steps to Reproduce

Additional Information

Email address

JohnReppy commented 5 months ago

Made the suggested edit to the sml.bat file. Fixed for 110.99.5.