GeckOS not compiling? No Preprosser in XA?

[halkun]

So I'm trying to compile GeckOS from scratch and for some reason XA (Which I also just compiled (v2.3.13)) doesn't recognize any of the preprosseor stuff in the code and spits back syntax errors. The documentation says it should be using a preprosseor but none of the Makefiles seem to reference it.

Am I missing a step?

[fachat]

The preprocessor is included in xa. It should work out of the box.

Which version of GeckOS are you trying to build, and how exactly?
There have been severe changes between v1 and v2 in that regard.

(Note currently on vacation with mobile only but trying to help from memory)

[halkun]

Gockos 2.1 Right from Git.

I compiled xa v2.3.13.

This is all under Cygwin (I don't have an extra machine to put Linux on so I grabbed the next best thing.)

xa compiled and created xa reloc65 ldo65 file65 printcbm uncpk (Well, technically, they all end in .exe)

when I run make in GeckOS, this is what happens.
----------------------------------------------------------

halku@DESKTOP-M50VPTV ~/GeckOS
$ make
(cd apps; make;)
make[1]: Entering directory '/home/halku/GeckOS/apps'
(for i in * ; do\
(if test -d $i; then cd $i; make; fi;); \
done)
make[2]: Entering directory '/home/halku/GeckOS/apps/file'
xa -I../../include -R -LLIB6502 mkdir.a65 -o mkdir
../lib6502/libjmp.a65
lib6502.i65:line 1: 1000:Syntax error
mkdir.a65:line 61: 1029:Label 'FC_MKDIR' not defined
mkdir.a65:line 62: 102c:Label 'fcmd' not defined
mkdir.a65:line 86: 1043:Label 'STDERR' not defined
mkdir.a65:line 91: 104d:Label 'fputc' not defined
Break after 5 errors

[fachat]

Hm interesting. I never tested with Cygwin.

It could be a) the symlink handling in that the file lib6502.i65 is not loaded but the contents of the symlink.
Or b) a lf/cr issue.

Did you check out the repo with options to convert line endings? I am not sure what git on cygwin does by default. Maybe try to checkout the original version if possible to see if tis is the case. I am unsure how xa handles CR only line endings or if there is a commandline switch.

Let me know if that works.
André

Edit: the included /include/lib6502.i65 file is actually a symbolic link to /lib6502/lib6502.i65

[halkun]

The links don't look correct
$ ls -lah lib6502.i65
lrwxrwxrwx 1 halku halku 21 Oct 22 02:21 lib6502.i65 -> ../lib6502/libjmp.a65

That is how it is in the repo as well. There is no /lib6502/lib6502.i65 -- just a /lib6502/lib6502.a65 that is it's own file.

==EDIT==
I decided to just set up Linux as a VM on my machine and I'm getting the same exact error as cygwin in windows - I think something is up with XA or that symlink is goofy.

I'm half tempted to port the thing to ca65 (Or at least an X16 version based on the C64 model) However, it is 6:00 in the morning and I've been goofing around with Linux all night so may wake up in a few hours with a concept of how terrible an idea that is.

[fachat]

I'm sorry I think I put the wrong destination into the post above. I'll have better access in a few days.

I looked up the docs, did you try to follow the instructions here also? https://github.com/fachat/GeckOS-V2/blo ... index.adoc
That's for building a runnable image.

The app build you tried should also work but without better access I just cannot try right now. So it would be good to see whether the runnable image works at least

[fachat]

I have been thinking about making it use a more standardnsyntx, but it heavily relies on xa features like preprocessor macros etc. Therefore it was low on my prio list