demos/aclock: Fix batch build file #257

New Issue

ace-dent · 2025-05-28T16:09:08+02:00

ace-dent commented

2025-05-28 16:09:08 +02:00

Currently in build_all.bat we have:

cd demos\aclock\trunk\
nasmw -t -f bin -o ..\..\..\%BIN%\demos\aclock aclock.asm
cd ..\..\..

See here.

But the code was moved from trunk and now uses FASM, with the commit here, by @IgorA and committed by @mxlgv ...

build_all code should either be fixed or removed.
Double check other build files / other dependencies.

Currently in `build_all.bat` we have: ```batch cd demos\aclock\trunk\ nasmw -t -f bin -o ..\..\..\%BIN%\demos\aclock aclock.asm cd ..\..\.. ``` See [here](https://git.kolibrios.org/KolibriOS/kolibrios/src/commit/2dfb3ddff35719a43d95d6911bf88f03de3039d5/programs/build_all.bat#L207-L209). But the code was moved from `trunk` and now uses FASM, with the commit [here](https://git.kolibrios.org/KolibriOS/kolibrios/commit/98cccc438c55046601d095cccf3672561e3f8999), by @IgorA and committed by @mxlgv ... - [ ] `build_all` code should either be fixed or removed. - [ ] Double check other build files / other dependencies.

ace-dent added the

Kind

Build

Category/Applications

Priority

Low

labels 2025-05-28 16:09:08 +02:00

IgorA commented

2025-05-29 11:13:00 +02:00

@mxlgv says that now *.bat files are removed from the repository. Only *.tup are left. That's why I decided that there is no point in fixing the build_all.bat file.

@mxlgv says that now `*.bat` files are removed from the repository. Only `*.tup` are left. That's why I decided that there is no point in fixing the `build_all.bat` file.

Burer commented

2025-05-31 15:15:48 +02:00

@mxlgv says that now *.bat files are removed from the repository. Only *.tup are left. That's why I decided that there is no point in fixing the build_all.bat file.

We decided to remove *.bat files for separate programs, as they usually have literally one line like

fasm.exe program.asm

But build_all.bat provides nice function of automated build of big number of programs. So, can you fix it, please, as for now there is no any decision to delete it.

I can commit fix by myself, but not sure, how should new command look. Like this?

fasm demos\aclock\trunk\aclock.asm %BIN%\demos\aclock

> @mxlgv says that now `*.bat` files are removed from the repository. Only `*.tup` are left. That's why I decided that there is no point in fixing the `build_all.bat` file. We decided to remove `*.bat` files for separate programs, as they usually have literally one line like ```bat fasm.exe program.asm ``` But `build_all.bat` provides nice function of automated build of big number of programs. So, can you fix it, please, as for now there is no any decision to delete it. I can commit fix by myself, but not sure, how should new command look. Like this? ```bat fasm demos\aclock\trunk\aclock.asm %BIN%\demos\aclock ```

Sign in to join this conversation.

3 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: KolibriOS/kolibrios#257