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

Fix looping bat files and add support for non-default file extensions #591

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

Conversation

abekornelis
Copy link
Collaborator

Fixes Issue #581 bat files for assembly c.a. do not accept non-default file extensions
Fixes Issue #582 bat files go into a loop when input file does not exist
Fixes Issue #590 bldjar.bat procedure fails to copy created jar file

@abekornelis abekornelis added bug Something isn't working component-build Related to build, distribution and packaging Lang:Script labels Oct 13, 2024
@abekornelis abekornelis requested a review from jyganci October 13, 2024 09:32
@abekornelis abekornelis self-assigned this Oct 13, 2024
bat/ASM.BAT Outdated Show resolved Hide resolved
Comment on lines +25 to +26
rem try with default extension of obj
set z_fext=.obj
Copy link
Contributor

@jyganci jyganci Dec 30, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines 25-26. obj should be OBJ.

Comment on lines +25 to +26
rem try with default extension of cbl
set z_fext=.cbl
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines 25-26. cbl should be CBL.

set /P z_file=Hit Enter for more help or suppply name of cbl file to compile:
:retry
set z_file=
set /P z_file=Hit Enter for more help or supply name of cbl file to compile:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cbl should be CBL.

Comment on lines +25 to +26
rem try with default extension of cbl
set z_fext=.cbl
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines 25-26. cbl should be CBL.

Comment on lines +25 to +26
rem try with default extension of cbl
set z_fext=.cbl
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines 25-26. cbl should be CBL.

set /P z_file=Hit Enter for more help or suppply name of cbl file to compile and link:
:retry
set z_file=
set /P z_file=Hit Enter for more help or supply name of cbl file to compile and link:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cbl should be CBL.

@@ -11,20 +11,23 @@ if /I "%1" == "tron" (set z_TraceMode=tron
set /A z_NestLevel=%z_NestLevel%+1
rem ----- Lvl(%z_NestLevel%) Start %0 %1 %2 %3 %4 %5 %6 %7 %8 %9

if /I %1. == . (set /P z_file=Hit Enter for help or suppply name of cbl file to compile:
if /I %1. == . (set /P z_file=Hit Enter for help or supply name of cbl file to compile:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cbl should be CBL.

@@ -11,20 +11,23 @@ if /I "%1" == "tron" (set z_TraceMode=tron
set /A z_NestLevel=%z_NestLevel%+1
rem ----- Lvl(%z_NestLevel%) Start %0 %1 %2 %3 %4 %5 %6 %7 %8 %9

if /I %1. == . (set /P z_file=Hit Enter for help or suppply name of cbl file to compile and link:
if /I %1. == . (set /P z_file=Hit Enter for help or supply name of cbl file to compile and link:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cbl should be CBL.

@@ -11,20 +11,23 @@ if /I "%1" == "tron" (set z_TraceMode=tron
set /A z_NestLevel=%z_NestLevel%+1
rem ----- Lvl(%z_NestLevel%) Start %0 %1 %2 %3 %4 %5 %6 %7 %8 %9

if /I %1. == . (set /P z_file=Hit Enter for help or suppply name of cbl file to compile, link, and execute:
if /I %1. == . (set /P z_file=Hit Enter for help or supply name of cbl file to compile, link, and execute:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cbl should be CBL.

set /P z_file=Hit Enter for more help or suppply name of cbl file to compile, link, and execute:
:retry
set z_file=
set /P z_file=Hit Enter for more help or supply name of cbl file to compile, link, and execute:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cbl should be CBL

@@ -11,28 +11,31 @@ if /I "%1" == "tron" (set z_TraceMode=tron
set /A z_NestLevel=%z_NestLevel%+1
rem ----- Lvl(%z_NestLevel%) Start %0 %1 %2 %3 %4 %5 %6 %7 %8 %9

if /I %1. == . (set /P z_file=Hit Enter for help or suppply name of obj file to link:
if /I %1. == . (set /P z_file=Hit Enter for help or supply name of obj file to link:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

obj should be OBJ

set /P z_file=Hit Enter for more help or suppply name of obj file to link:
:retry
set z_file=
set /P z_file=Hit Enter for more help or supply name of obj file to link:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

obj should be OBJ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working component-build Related to build, distribution and packaging Lang:Script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants