From 85a15c683ca048dbb80143f3f4cc667d7387cd33 Mon Sep 17 00:00:00 2001 From: Cody Boone Ferguson <53008573+xexyl@users.noreply.github.com> Date: Thu, 6 Jul 2023 10:56:24 -0700 Subject: [PATCH] Add -lm to 2005/jetro/Makefile Not all systems implicitly link in libm so this is necessary for those systems. --- 2005/jetro/Makefile | 2 +- thanks-for-fixes.md | 11 ++++++++--- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/2005/jetro/Makefile b/2005/jetro/Makefile index fad0824df..006131719 100644 --- a/2005/jetro/Makefile +++ b/2005/jetro/Makefile @@ -73,7 +73,7 @@ CFLAGS= ${CSTD} ${CWARN} ${ARCH} ${CDEFINE} ${CINCLUDE} ${OPT} # Libraries needed to build # -LIBS= `${SDL2_CONFIG} --libs` +LIBS= `${SDL2_CONFIG} --libs` -lm # C compiler to use # diff --git a/thanks-for-fixes.md b/thanks-for-fixes.md index dae18536b..0329c88d8 100644 --- a/thanks-for-fixes.md +++ b/thanks-for-fixes.md @@ -1155,12 +1155,17 @@ for the self-test feature. For much more details on the problem see [bugs.md](/bugs.md). +## [2005/jetro](2005/jetro/jetro.c) ([README.md](2005/jetro/README.md)) + +Cody added explicit linking of libm (`-lm`) for systems like linux that seem to +not do it implicitly (like macOS does). + ## [2005/mynx](2005/mynx/mynx.c) ([README.md](2005/mynx/README.md])) -Cody fixed this so that the -[configure](source/configure) script (which is not needed but part of the entry) -would work with compilers that have by default `-Werror` like clang in macOS. +Cody fixed this so that the [configure](source/configure) script (which is not +needed but part of the entry) would work with compilers that have by default +`-Werror` like clang in macOS. Cody also added the alt code which does scan for https. Futile, maybe, based on how https is set up, in which case just enjoy it for what it was. But there