🤖 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 ofAC_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 withAC_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:

charreadline ();intmain (void) {returnreadline ();}

needs to be linked with both-lreadline and-lncurses; that means thatchecking for readline in -lreadline fails. I thought that passingLIBREADLINE_LIBS would fix that, but the content of that variable is not used until the test passes, andLIBS="-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 byGH-94802.

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

Thanks for help!