GeezerVison

🤖 New build scheduled with the buildbot fleet by @tiran for commit 33d9e296124d7daeef89fca957bdab51e1e1d11b 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

🤖 New build scheduled with the buildbot fleet by @tiran for commit dd5937322e181781c03213621eddb0fb88527c48 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@erlend-aasland I have refactored the code. There is a bit of repetition but IMHO it's ok for a few lookups. We could add our own variant of AC_CHECK_LIB at a later point in time.

I have merged the PR. It works on all tested platforms. We can review and address the general issue with AC_CHECK_LIB in a future PR.

The change triggered a reference leak on Fedora: #94644

Hi, I think I found an issue.

On Solaris, even a simple code like:

char readline ();
int main (void) {
  return readline ();
}

needs to be linked with both -lreadline and -lncurses; that means that checking for readline in -lreadline fails. I thought that passing LIBREADLINE_LIBS would fix that, but the content of that variable is not used until the test passes, and LIBS="-lreadline $LIBS" is always used. (AFAICT, automatic pkg-config would also do nothing, although I didn't test that).

Is there something I am missing or something I can try?

The problem should be fixed by GH-94802.

Ah, it's indeed fixed. I am sorry, I thought I am looking into latest main but it was 12 days old (when this was pushed).

Thanks for help!