EtchedPixels / FUZIX

FuzixOS: Because Small Is Beautiful
Other
2.19k stars 275 forks source link

netd applications not building on Z80 #1088

Closed electrified closed 2 months ago

electrified commented 2 months ago

Hey there.

I see the netd applications are no longer built for Z80.

After modifying Makefile.z80 so they are built (BUSTEDAPPS -> APPS) I got the following error.

netd.c:973 - out of index memory.

Any ideas on what to change/ how much work it would be to get netd working again?

Thanks.

EtchedPixels commented 2 months ago

That's coming from the compiler running out of internal space rather than a netd problem.

Which config are you building ?

electrified commented 2 months ago

I'm building rcbus-z180 - these are the only changes from master

diff --git a/Applications/netd/Makefile.z80 b/Applications/netd/Makefile.z80
index 4ee149fae..f44d43dd3 100644
--- a/Applications/netd/Makefile.z80
+++ b/Applications/netd/Makefile.z80
@@ -8,7 +8,7 @@ SRCS += htget.c tinyirc.c linein.c ifconfig.c

 OBJS = $(SRCS:.c=.o)

-BUSTEDAPPS = netd
+APPS = netd

 all: $(APPS)

diff --git a/Makefile b/Makefile
index ab6da4394..60e8d9d28 100644
--- a/Makefile
+++ b/Makefile
@@ -89,7 +89,7 @@
 # zxdiv:       ZX Spectrum 128K with DivIDE/DivMMC interface
 # zxuno:       ZX Uno FPGA system

-TARGET ?= zrc
+TARGET ?= rcbus-z180

 include version.mk

and the surrounding output is:

(cd netd; make -f Makefile.z80)
make[2]: Entering directory '/src/Applications/netd'
fcc -mz80 -X -Os -D__STDC__ -c -I/src/Library/include -I/src/Library/include/z80 -DNETD_LITTLE_ENDIAN   -c -o netd.o netd.c
<command-line>: warning: "__STDC__" redefined
<built-in>: note: this is the location of the previous definition
uip.h:1835 - struct declared twice
uip.h:1841 - struct declared twice
netd.c:973 - out of index memory
make[2]: *** [<builtin>: netd.o] Error 1
make[2]: Leaving directory '/src/Applications/netd'
make[1]: *** [Makefile:41: netd] Error 2
make[1]: Leaving directory '/src/Applications'
make: *** [Makefile:138: apps] Error 2
EtchedPixels commented 2 months ago

Should work now if you update kernel and compiler. Turns out much easier to fix than I thought.

electrified commented 2 months ago

All working now - many thanks.