From bed25016b1e9944fbbfeaaf3d689965d10dc61aa Mon Sep 17 00:00:00 2001 From: Jesse Smith Date: Wed, 9 May 2018 17:55:35 -0300 Subject: [PATCH] Updated sulogin.c to pull in crypt.h as it is a new, undocumented dependency on Fedora 28. Confirmed we can still build on other/older platforms. --- doc/Changelog | 7 +++++++ src/Makefile | 2 +- src/sulogin.c | 1 + 3 files changed, 9 insertions(+), 1 deletion(-) diff --git a/doc/Changelog b/doc/Changelog index b1b7950..3dddec8 100644 --- a/doc/Changelog +++ b/doc/Changelog @@ -35,6 +35,13 @@ sysvinit (2.90) UNRELEASED; urgency=low Added .gitignore files to avoid git tracking object files. Removed old start-stop-daemon from contrib directory. (Patches provided by Guillem Jover.) + * Cleaned up most warnings generated by GCC 7 & 8. We still + get some from faulty "nonstring" reports, but silencing them on + GCC 8 results in more warnings on GCC 7 and Clang, so leaving them + for now. + * Fixed compile error on Fedora 28 where crypt won't build due to + undocumented dependency change. + sysvinit (2.89) world; urgency=low diff --git a/src/Makefile b/src/Makefile index 65ee5db..088fe22 100644 --- a/src/Makefile +++ b/src/Makefile @@ -10,7 +10,7 @@ CPPFLAGS = CFLAGS ?= -ansi -O2 -fomit-frame-pointer -fstack-protector -override CFLAGS += -W -Wall -Wunreachable-code -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -D_XOPEN_SOURCE -D_GNU_SOURCE +override CFLAGS += -W -Wall -Wunreachable-code -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -D_XOPEN_SOURCE -D_GNU_SOURCE override CFLAGS += $(shell getconf LFS_CFLAGS) STATIC = MANDB := s@^\('\\\\\"\)[^\*-]*-\*- coding: [^[:blank:]]\+ -\*-@\1@ diff --git a/src/sulogin.c b/src/sulogin.c index 483fce6..e46b11a 100644 --- a/src/sulogin.c +++ b/src/sulogin.c @@ -39,6 +39,7 @@ #include #include #include +#include /* added to make this compile on Fedora 28 */ #include #include #include