oetiker / mrtg

MRTG - Multi Router Traffic Grapher
http://www.mrtg.org
243 stars 60 forks source link

Include <strings.h> header file #101

Closed youpong closed 1 year ago

youpong commented 1 year ago

Description

Include the header file containing the strncasecmp() declaration.

Related Issue

99

oetiker commented 1 year ago

please extend configure.ac to test this ...

youpong commented 1 year ago

The configure script already checks for the presence of the header file strings.h.

config.h.in:15

/* Define to 1 if you have the <strings.h> header file. */
#undef HAVE_STRINGS_H
oetiker commented 1 year ago

so you can add an ifdef around you include

----- On Oct 27, 2022, at 1:48 PM, youpong @.***> wrote:

The configure script already checks for the presence of the header file strings.h .

config.h.in:15 / Define to 1 if you have the header file. /

undef HAVE_STRINGS_H

— Reply to this email directly, [ https://github.com/oetiker/mrtg/pull/101#issuecomment-1293408039 | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/AADIZXYKRIS2WMV567SJKP3WFJTZNANCNFSM6AAAAAARPJ47VI | unsubscribe ] . You are receiving this because your review was requested. Message ID: <oetiker/mrtg/pull/101/c1293408039 @ github . com>

-- Tobi Oetiker, OETIKER+PARTNER AG, Aarweg 15 CH-4600 Olten, Switzerland www.oetiker.ch @.*** +41 62 775 9902

youpong commented 1 year ago

Added the ifdef.

oetiker commented 1 year ago

thanks!