mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCH 1/2] readline: fix pkg-config errors with NCURSES_WIDE_CHAR
@ 2024-01-30 10:58 Michael Tretter
  2024-01-30 10:58 ` [ptxdist] [PATCH 2/2] pipewire: fix readline support Michael Tretter
  2024-02-08 16:03 ` [ptxdist] [APPLIED] readline: fix pkg-config errors with NCURSES_WIDE_CHAR Michael Olbrich
  0 siblings, 2 replies; 4+ messages in thread
From: Michael Tretter @ 2024-01-30 10:58 UTC (permalink / raw)
  To: ptxdist; +Cc: pza, mtr

readline.pc contains Requires.private=ncurses, even if readline is built
with ncursesw. When a user of readline uses pkg-config to ask for the
cflags, the lookup fails if ncursesw without compatibility with ncurses
is used.

Fix it by ensuring that ncurses compatibility is enabled if readline is
built with ncurses.

Signed-off-by: Michael Tretter <m.tretter@pengutronix.de>
---
 rules/readline.in | 1 +
 1 file changed, 1 insertion(+)

diff --git a/rules/readline.in b/rules/readline.in
index 036860409a56..7673ce7497eb 100644
--- a/rules/readline.in
+++ b/rules/readline.in
@@ -2,6 +2,7 @@
 menuconfig READLINE
 	tristate
 	select NCURSES if READLINE_NCURSES
+	select NCURSES_BACKWARD_COMPATIBLE_NON_WIDE_CHAR if READLINE_NCURSES && NCURSES_WIDE_CHAR
 	select TERMCAP if READLINE_TERMCAP
 	select GCCLIBS_GCC_S
 	prompt "readline                      "
-- 
2.39.2




^ permalink raw reply	[flat|nested] 4+ messages in thread

* [ptxdist] [PATCH 2/2] pipewire: fix readline support
  2024-01-30 10:58 [ptxdist] [PATCH 1/2] readline: fix pkg-config errors with NCURSES_WIDE_CHAR Michael Tretter
@ 2024-01-30 10:58 ` Michael Tretter
  2024-02-08 16:03   ` [ptxdist] [APPLIED] " Michael Olbrich
  2024-02-08 16:03 ` [ptxdist] [APPLIED] readline: fix pkg-config errors with NCURSES_WIDE_CHAR Michael Olbrich
  1 sibling, 1 reply; 4+ messages in thread
From: Michael Tretter @ 2024-01-30 10:58 UTC (permalink / raw)
  To: ptxdist; +Cc: pza, mtr

Due to the missing PTXCONF_ prefix, pw-ctl was always built without
readline support even though the rule looked like readline support was
enabled. Furthermore, the readline dependency was pulled in.

Add the PTXCONF_ prefix to actually enable readline support.

Signed-off-by: Michael Tretter <m.tretter@pengutronix.de>
---
 rules/pipewire.make | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/rules/pipewire.make b/rules/pipewire.make
index 34081c5ceeb5..8747ebe6cecc 100644
--- a/rules/pipewire.make
+++ b/rules/pipewire.make
@@ -90,7 +90,7 @@ PIPEWIRE_CONF_OPT	:= \
 	-Dpw-cat=enabled \
 	-Dpw-cat-ffmpeg=disabled \
 	-Draop=$(call ptx/endis,PTXCONF_PIPEWIRE_RAOP)d \
-	-Dreadline=$(call ptx/endis,PIPEWIRE_PW_CTL)d \
+	-Dreadline=$(call ptx/endis,PTXCONF_PIPEWIRE_PW_CTL)d \
 	-Drlimits-install=false \
 	-Drlimits-match=@pipewire \
 	-Drlimits-memlock=4194304 \
-- 
2.39.2




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [ptxdist] [APPLIED] readline: fix pkg-config errors with NCURSES_WIDE_CHAR
  2024-01-30 10:58 [ptxdist] [PATCH 1/2] readline: fix pkg-config errors with NCURSES_WIDE_CHAR Michael Tretter
  2024-01-30 10:58 ` [ptxdist] [PATCH 2/2] pipewire: fix readline support Michael Tretter
@ 2024-02-08 16:03 ` Michael Olbrich
  1 sibling, 0 replies; 4+ messages in thread
From: Michael Olbrich @ 2024-02-08 16:03 UTC (permalink / raw)
  To: ptxdist; +Cc: Michael Tretter

Thanks, applied as 28a7d7667e32eb98ee2e7f25e88d48052a11d8f5.

Michael

[sent from post-receive hook]

On Thu, 08 Feb 2024 17:03:20 +0100, Michael Tretter <m.tretter@pengutronix.de> wrote:
> readline.pc contains Requires.private=ncurses, even if readline is built
> with ncursesw. When a user of readline uses pkg-config to ask for the
> cflags, the lookup fails if ncursesw without compatibility with ncurses
> is used.
> 
> Fix it by ensuring that ncurses compatibility is enabled if readline is
> built with ncurses.
> 
> Signed-off-by: Michael Tretter <m.tretter@pengutronix.de>
> Message-Id: <20240130105815.3091486-1-m.tretter@pengutronix.de>
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
> 
> diff --git a/rules/readline.in b/rules/readline.in
> index 036860409a56..7673ce7497eb 100644
> --- a/rules/readline.in
> +++ b/rules/readline.in
> @@ -2,6 +2,7 @@
>  menuconfig READLINE
>  	tristate
>  	select NCURSES if READLINE_NCURSES
> +	select NCURSES_BACKWARD_COMPATIBLE_NON_WIDE_CHAR if READLINE_NCURSES && NCURSES_WIDE_CHAR
>  	select TERMCAP if READLINE_TERMCAP
>  	select GCCLIBS_GCC_S
>  	prompt "readline                      "



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [ptxdist] [APPLIED] pipewire: fix readline support
  2024-01-30 10:58 ` [ptxdist] [PATCH 2/2] pipewire: fix readline support Michael Tretter
@ 2024-02-08 16:03   ` Michael Olbrich
  0 siblings, 0 replies; 4+ messages in thread
From: Michael Olbrich @ 2024-02-08 16:03 UTC (permalink / raw)
  To: ptxdist; +Cc: Michael Tretter

Thanks, applied as dd2d8a07716ca498ae9e0b69a0625c28d7be8655.

Michael

[sent from post-receive hook]

On Thu, 08 Feb 2024 17:03:21 +0100, Michael Tretter <m.tretter@pengutronix.de> wrote:
> Due to the missing PTXCONF_ prefix, pw-ctl was always built without
> readline support even though the rule looked like readline support was
> enabled. Furthermore, the readline dependency was pulled in.
> 
> Add the PTXCONF_ prefix to actually enable readline support.
> 
> Signed-off-by: Michael Tretter <m.tretter@pengutronix.de>
> Message-Id: <20240130105815.3091486-2-m.tretter@pengutronix.de>
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
> 
> diff --git a/rules/pipewire.make b/rules/pipewire.make
> index 34081c5ceeb5..8747ebe6cecc 100644
> --- a/rules/pipewire.make
> +++ b/rules/pipewire.make
> @@ -90,7 +90,7 @@ PIPEWIRE_CONF_OPT	:= \
>  	-Dpw-cat=enabled \
>  	-Dpw-cat-ffmpeg=disabled \
>  	-Draop=$(call ptx/endis,PTXCONF_PIPEWIRE_RAOP)d \
> -	-Dreadline=$(call ptx/endis,PIPEWIRE_PW_CTL)d \
> +	-Dreadline=$(call ptx/endis,PTXCONF_PIPEWIRE_PW_CTL)d \
>  	-Drlimits-install=false \
>  	-Drlimits-match=@pipewire \
>  	-Drlimits-memlock=4194304 \



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-02-08 16:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-30 10:58 [ptxdist] [PATCH 1/2] readline: fix pkg-config errors with NCURSES_WIDE_CHAR Michael Tretter
2024-01-30 10:58 ` [ptxdist] [PATCH 2/2] pipewire: fix readline support Michael Tretter
2024-02-08 16:03   ` [ptxdist] [APPLIED] " Michael Olbrich
2024-02-08 16:03 ` [ptxdist] [APPLIED] readline: fix pkg-config errors with NCURSES_WIDE_CHAR Michael Olbrich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox