oh I forgot about that - renaming the pkg-config file seems the most
sensible way, but I agree now isn't a great time - let's pick this up again
after unlock.
--
Sent from a phone, apologies for poor formatting.
On 28 September 2025 16:10:30 Jeremie Courreges-Anglas <[email protected]> wrote:
On Sun, Sep 28, 2025 at 02:42:39PM +0100, Stuart Henderson wrote:
On 2025/09/28 22:26, SASANO Takayoshi wrote:
> Hello,
>
> Current devel/readline has following problem, this is catastrophic
> with pkg_search_module() of CMake.
>
> openbsd-current-vm# pkg-config --libs readline
> Package termcap was not found in the pkg-config search path.
> Perhaps you should add the directory containing `termcap.pc'
> to the PKG_CONFIG_PATH environment variable
> Package 'termcap', required by 'readline', not found
> openbsd-current-vm#
>
> this is caused by "Requires.private: termcap" in
> /usr/local/lib/pkgconfig/readline.pc and the problem is not occur when
> remove this line.
btw I am actually currently testing an update to readline 8.3 with the
dependent ports.
> Here's the diff, ok?
not ok, see below
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/devel/readline/Makefile,v
> diff -u -p -r1.22 Makefile
> --- Makefile 16 Aug 2025 14:26:42 -0000 1.22
> +++ Makefile 28 Sep 2025 13:19:58 -0000
> @@ -3,7 +3,7 @@ COMMENT = library to edit command lines
> V = 8.2.13
> DISTNAME = readline-${V}
> PKGNAME = readline-${V}
> -REVISION = 1
> +REVISION = 2
>
> LIBV = 3.0
> SHARED_LIBS += ehistory ${LIBV}
> Index: patches/patch-readline_pc_in
> ===================================================================
> RCS file: patches/patch-readline_pc_in
> diff -N patches/patch-readline_pc_in
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ patches/patch-readline_pc_in 28 Sep 2025 13:19:58 -0000
> @@ -0,0 +1,9 @@
> +--- readline.pc.in.orig Tue Apr 9 03:22:47 2019
> ++++ readline.pc.in Sun Sep 28 22:15:11 2025
> +@@ -7,6 +7,5 @@ Name: Readline
> + Description: Gnu Readline library for command line editing
> + URL: http://tiswww.cwru.edu/php/chet/readline/rltop.html
> + Version: @LIBVERSION@
> +-Requires.private: @TERMCAP_PKG_CONFIG_LIB@
That's the wrong fix, TERMCAP_PKG_CONFIG_LIB is set incorrectly and
should actually end up as "ncurses".
That can be done with
CONFIGURE_ENV = bash_cv_termcap_lib=ncurses
> + Libs: -L${libdir} -lreadline
that line is wrong for this port too, it needs -lereadline not -lreadline.
matthieu@ tried to address these two issues, see
https://github.com/openbsd/ports/commits/master/devel/readline
Since it unbreaks detection of devel/readline it results in ports
picking it up, resulting in hidden deps or breakage during bulk
builds. Neutering these hidden deps in existing ports or moving the
.pc to ereadline.pc could be a solution, as discussed by matthieu@ and
ajacoutot@. No idea what's the best way forward, nor whether we
should actually do anything at this point in the release cycle.
--
jca
