From 2d3c39207f76ff036d70e8c8aff914ad5a0b17c8 Mon Sep 17 00:00:00 2001 From: Cody Boone Ferguson <53008573+xexyl@users.noreply.github.com> Date: Thu, 6 Jul 2023 10:59:58 -0700 Subject: [PATCH] Add -lm to 2013/morgan1/Makefile Not all systems implicitly link in libm so this is necessary for those systems. --- 2013/morgan1/Makefile | 2 +- thanks-for-fixes.md | 5 +++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/2013/morgan1/Makefile b/2013/morgan1/Makefile index 3cd386f13..07364fe30 100644 --- a/2013/morgan1/Makefile +++ b/2013/morgan1/Makefile @@ -73,7 +73,7 @@ CFLAGS= ${CSTD} ${CWARN} ${ARCH} ${CDEFINE} ${CINCLUDE} ${OPT} # Libraries needed to build # -LIBS= -L ${X11_LIBDIR} -lX11 +LIBS= -L ${X11_LIBDIR} -lX11 -lm # C compiler to use # diff --git a/thanks-for-fixes.md b/thanks-for-fixes.md index 0329c88d8..7ce556a73 100644 --- a/thanks-for-fixes.md +++ b/thanks-for-fixes.md @@ -1294,6 +1294,11 @@ removed from SDL2 so the Makefile has to use `sdl-config` and not `sdl2-config`. This means that to be strictly technical, the removal of `localtime()` is not necessary but this change was left in place anyway. +## [2013/morgan1](2013/morgan1/morgan1.c) ([README.md](2013/morgan1/README.md)) + +Cody added explicit linking of libm (`-lm`) as not all systems do this +implicitly (linux doesn't seem to but macOS does). + ## [2014/maffiodo1](2014/maffiodo1/maffiodo1.c) ([README.md](2014/maffiodo1/README.md]))