From 0aa291aaff4aaa7be7a8dd857359453c59d27a21 Mon Sep 17 00:00:00 2001 From: Cody Boone Ferguson <53008573+xexyl@users.noreply.github.com> Date: Thu, 23 Mar 2023 10:45:12 -0700 Subject: [PATCH] Update bugs.md The segfault in 1996/westley I fixed some days ago so there's no need for it to be in bugs.md. Added the requirement for 32-bit arch and no optimiser in 2005/giljade but suggesting it might be possible to fix it for 64-bit systems. Probably this should be in a different section but I put it in the known bugs to fix (even though it's not necessarily a bug). Typo fix in 1990/jaw. To see 1984/mullender in action we don't need an emulator: one can use 2015/endoh3 instead. I noted this, linking to it, but also noted that otherwise one would need an emulator for the entry. 2004/gavin actually works. The problem is it doesn't work under macOS at least with the M1+ chip (might or might not work with Intel). I still kept the recent changes to the file but I noted it's not related to the problem; it's an architecture issue. --- bugs.md | 35 ++++++++++++++++++----------------- 1 file changed, 18 insertions(+), 17 deletions(-) diff --git a/bugs.md b/bugs.md index ce1788975..c49727fda 100644 --- a/bugs.md +++ b/bugs.md @@ -102,6 +102,15 @@ needless cast to `char *` from gets(). This cast remains in the code however. however. +### [2005/giljade](2005/giljade/giljade.c) ([README.md](2005/giljade/README.md)) + +[Cody Boone Ferguson](/winners.html#Cody_Boone_Ferguson) noted that this +requires both 32-bit CPUs (data size?) and also cannot have optimiser enabled. +It might be that it can be fixed, perhaps by changing the int sizes and bitwise +operations, but if not this will not work with some systems like modern Macs as +Apple has made it hard to compile 32-bit applications. + + ### [2014/vik](2014/vik/prog.c) ([README.md](2014/vik/README.md)) @@ -182,7 +191,7 @@ the judges and needn't be fixed. [Cody Boone Ferguson](/winners.html#Cody_Boone_Ferguson) fixed the scripts to work in modern systems. He notes however that the command in the try section, -the one to test the official C entry, appear to not work with macOS, giving: +the one to test the official C entry, appears to not work with macOS, giving: $ echo "Quartz glyph jocks vend, fix, BMW." | compress | ./btoa | ./jaw @@ -218,14 +227,6 @@ Should the entry use `perror()`? Perhaps not but we're not sure of its purpose so it can stay for now with this note. -### [1996/westley](1996/westley/westley.c) ([README.md](1996/westley/README.md)) - - -This entry will possibly segfault even when showing the proper output in the -`clock1`, `clock2` and `clock3` scripts. Though this doesn't need to be fixed as -long as it works right it wouldn't hurt to fix it either. - - ### [2019/endoh](2019/endoh/prog.c) ([README.md](2019/endoh/README.md)) @@ -295,13 +296,16 @@ XXX - .. more entries to be added later .. This entry is likely to fail and/or dump core on any computer other than a Vax-11 or pdp-11. In 1984 machine dependent code was allowed. -Perhaps what is needed is for someone to supply a Vax-11 or pdp-11 emulator for this entry. +To see this entry in action check out [Yusuke +Endoh](/winners.html#Yusuke_Endoh)'s [2015/endoh3](2015/endoh3) entry. Otherwise +a Vax-11 or pdp-11 emulator would be necessary. ### [2004/gavin](2004/gavin/gavin.c) ([README.md](2004/gavin//README.md)) -Segmentation fault occurs when running `gavin` to produce the kernel: +Segmentation fault will occur in some systems. For instance on macOS with the +arm64 chip: ```sh ./gavin > kernel @@ -318,13 +322,10 @@ ld: symbol(s) not found for architecture arm64 ``` -#### Recent for 2004/gavin mods: +#### Recent 2004/gavin mods: - -In case recent modification are part of the problem, here is a full disclosure -of what was changed: - -To compile on modern C compilers, the following patch was applied to `gavin.c`: +Although not related some recent changes were made to 2004/gavin to let it +compile under clang. The following patch was applied: ```patch diff --git a/2004/gavin/gavin.c b/2004/gavin/gavin.c