Recent activity

Re: [PATCH porporo] Add user configuration and MacOS specific flags to Makefile 8 months ago

From Marcin Cieslak to ~rabbits/public-inbox

On Sun, 19 May 2024, ~xsova wrote:

> From: Bryce Schultz <bryce@xsova.dev>
>
> -RELEASE_flags=-std=c89 -Os -DNDEBUG -g0 -s -Wall -Wno-unknown-pragmas
> +RELEASE_flags=-std=c89 -Os -DNDEBUG -g0 -Wall -Wno-unknown-pragmas
> DEBUG_flags=-std=c89 -DDEBUG -Wall -Wno-unknown-pragmas -Wpedantic -Wshadow -Wextra -Werror=implicit-int -Werror=incompatible-pointer-types -Werror=int-conversion -Wvla -g -Og -fsanitize=address -fsanitize=undefined

> bin/uxnasm: src/utils/uxnasm.c
> 	@ mkdir -p bin
> 	@ cc ${RELEASE_flags} src/utils/uxnasm.c -o ${ASM}
> -
> +	@ strip ${ASM}
> +

Re: [PATCH] Add mac native rom.s 1 year, 1 month ago

From Marcin Cieslak to ~rabbits/public-inbox

On Tue, 26 Dec 2023, Kyle Perik wrote:

> Do you think there's a way to avoid a different .s file?

In theory, there should be a way...

> I'm a C/ASM
> novice so forgive my ignorance

We are all learning here.... Is it possible to check
what assembler ("as" command maybe) is used on your system?

I try to reproduce the problem with a recent clang compiler
and everything works as expected (but I'm not on a Mac).

Re: [PATCH] Add mac native rom.s 1 year, 1 month ago

From Marcin Cieslak to ~rabbits/public-inbox

On Sat, 23 Dec 2023, Kyle Perik wrote:

>> +.section __TEXT,__text

What happens if you leave it out completely?
(gcc seems to use text segment here by default, without .section for me)

What is the error you are getting with .rodata here? or is it about the symbol underscore?

saper

Re: [PATCH] Declare snprintf to fix builds on macOS 1 year, 9 months ago

From Marcin Cieslak to ~rabbits/public-inbox

On Tue, 2 May 2023, Sigrid Solveig Haflínudóttir wrote:

> It'd also work, but only because snprintf is not a define. Imo, this is just wrong way to fix this.

Sorry, forget my babble. I've played too much with the macros that day. You are of course correct.

I just dislike the code sprinkled with some crazy system-specific #ifdefs

Re: [PATCH] Declare snprintf to fix builds on macOS 1 year, 9 months ago

From Marcin Cieslak to ~rabbits/public-inbox

On Mon, 1 May 2023, Hundred Rabbits wrote:

> Thanks Matus,
>
> Someone was hitting that same issue just yesterday, hopefully this
> will help them with their issue, merged :)

Would

#ifndef snprintf

also work?

Re: [PATCH uxn] A more elegant solution to the Windows bugs in filepaths and MinGW compilation 1 year, 10 months ago

From Marcin Cieslak to ~rabbits/public-inbox

On Sun, 19 Mar 2023, Deadly Headshot wrote:

> -		strcat(p, "/"); /* TODO: use a macro instead of '/' for the path delimiter */
> +		strcat(p, DIR_SEP_STR); /* TODO: use a macro instead of '/' for the path delimiter */

I think we can remove the TODO comments here?

Re: Tiny problem bootstrapping left 2 years ago

From Marcin Cieslak to ~rabbits/public-inbox

On Fri, 25 Nov 2022, Hundred Rabbits wrote:

> That's no the latest version of uxnasm, you should update and it'll be fine :)

Ouch, right:

> uxnasm src/left.tal l.rom
-- Unused label: draw-dec
-- Unused label: phex
Assembled l.rom in 9003 bytes(13.79% used), 424 labels, 0 macros.

Thanks!

Tiny problem bootstrapping left 2 years ago

From Marcin Cieslak to ~rabbits/public-inbox

Hello, I have built uxn from source (1b2049e238df96f32335edf1c6db35bd09f8b42d) on FreeBSD and then I have tried to compile left using uxnasm (commit 5fc8cc1d847a4b95dc6e09ea467bdc3c7729b973):

> uxnasm src/left.tal l.rom
Unknown token: =file-new
Unknown token: =file-new
Assembly: Failed to assemble rom.

I have downloaded https://rabbits.srht.site/drifblim/drifblim.rom
and it worked:

> uxncli ../play/drifblim.rom src/left.tal l.rom
Loaded ../play/drifblim.rom
Input(.tal): Output(.rom): 
-- Unused label: selection