Skip to content

Use autotools macros to find ncurses library#3

Open
ZedThree wants to merge 3 commits intodannywarren:masterfrom
ZedThree:autoconf
Open

Use autotools macros to find ncurses library#3
ZedThree wants to merge 3 commits intodannywarren:masterfrom
ZedThree:autoconf

Conversation

@ZedThree
Copy link
Copy Markdown
Contributor

  • Use a better macro to determine the C compiler. This will look for the CC environment variable first, so that it is possible to use e.g. clang.
  • Use macros to determine how to link ncurses. On some systems the correct flag is -lcurses and on others -lncurses. This should determine the correct form.

I've used autoconf 2.69 and aclocal 1.13.4 to update configure, and I've tested on OpenSUSE 13.2 and Scientific Linux 5.11. I'm afraid because configure is automatically generated, the changes are quite unreadable, but configure.in and Makefile.in are the important changes, and they are much easier to read.

ZedThree added 3 commits July 14, 2016 16:12
Will check CC environment variable if set
If pkg-config can't find ncurses, trying AC_CHECK_LIB with ncurses and
curses, before giving up
@dannywarren
Copy link
Copy Markdown
Owner

I'm going to hold off on this one until the next version, so that I can take some time to test this under a few different platforms. The other pulls related to the time parsing segfault and support for Nagios 4.x are important so I want to get those out now.

Thanks again for taking care of all this!

@ZedThree
Copy link
Copy Markdown
Contributor Author

No worries, glad to help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants