zsh-workers
 help / color / mirror / code / Atom feed
* Asking for status of a submitted patch
@ 2020-04-23 16:05 Lars Wendler
  2020-04-25  9:54 ` Daniel Shahaf
  2021-04-11 21:02 ` Oliver Kiddle
  0 siblings, 2 replies; 4+ messages in thread
From: Lars Wendler @ 2020-04-23 16:05 UTC (permalink / raw)
  To: zsh-workers; +Cc: Tetja Rediske

[-- Attachment #1: Type: text/plain, Size: 476 bytes --]

Hi dear list,

on 2019 December 12th I had sent a patch to this list which fixes a
build issue in configure.ac when ncurses was built with separate
tinfo(w) lib.
The mail can be found here:
http://www.zsh.org/mla/workers/2019/msg01040.html

I wonder if there's anything wrong with the patch and if I need to
adjust it in order to get it accepted.

Kind regards
-- 
Lars Wendler
Gentoo package maintainer
GPG: 21CC CF02 4586 0A07 ED93  9F68 498F E765 960E 9B39

[-- Attachment #2: Digitale Signatur von OpenPGP --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* Re: Asking for status of a submitted patch
  2020-04-23 16:05 Asking for status of a submitted patch Lars Wendler
@ 2020-04-25  9:54 ` Daniel Shahaf
  2021-04-11 21:02 ` Oliver Kiddle
  1 sibling, 0 replies; 4+ messages in thread
From: Daniel Shahaf @ 2020-04-25  9:54 UTC (permalink / raw)
  To: Lars Wendler; +Cc: zsh-workers, Tetja Rediske

Lars Wendler wrote on Thu, 23 Apr 2020 18:05 +0200:
> Hi dear list,
> 
> on 2019 December 12th I had sent a patch to this list which fixes a
> build issue in configure.ac when ncurses was built with separate
> tinfo(w) lib.
> The mail can be found here:
> http://www.zsh.org/mla/workers/2019/msg01040.html
> 
> I wonder if there's anything wrong with the patch and if I need to
> adjust it in order to get it accepted.

I've followed up now.  Thanks for the reminder.

Daniel

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

* Re: Asking for status of a submitted patch
  2020-04-23 16:05 Asking for status of a submitted patch Lars Wendler
  2020-04-25  9:54 ` Daniel Shahaf
@ 2021-04-11 21:02 ` Oliver Kiddle
  2021-04-12 18:46   ` Lars Wendler
  1 sibling, 1 reply; 4+ messages in thread
From: Oliver Kiddle @ 2021-04-11 21:02 UTC (permalink / raw)
  To: Lars Wendler; +Cc: Zsh workers

On 23 Apr 2020, Lars Wendler wrote:
> on 2019 December 12th I had sent a patch to this list which fixes a
> build issue in configure.ac when ncurses was built with separate
> tinfo(w) lib.

To this day, your patch remains unmerged. At least in my case, I can't
work out what problem it solves, how that manifests, how I might
reproduce it. I can guess that you may be using gentoo Linux from your
e-mail but how you might have configured the ncurses build is not
obvious. Just knowing what configure options are used to build it may
help.

Without it, does zsh fail to build or is some functionality or feature
missing or broken?

Is it linking to the wrong library? Or failing to resolve a symbol -
which one(s)?

Oliver


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

* Re: Asking for status of a submitted patch
  2021-04-11 21:02 ` Oliver Kiddle
@ 2021-04-12 18:46   ` Lars Wendler
  0 siblings, 0 replies; 4+ messages in thread
From: Lars Wendler @ 2021-04-12 18:46 UTC (permalink / raw)
  To: Oliver Kiddle; +Cc: Zsh workers

[-- Attachment #1: Type: text/plain, Size: 1557 bytes --]

Hi Oliver,

On Sun, 11 Apr 2021 23:02:36 +0200 Oliver Kiddle wrote:

>On 23 Apr 2020, Lars Wendler wrote:
>> on 2019 December 12th I had sent a patch to this list which fixes a
>> build issue in configure.ac when ncurses was built with separate
>> tinfo(w) lib.
>
>To this day, your patch remains unmerged. At least in my case, I can't
>work out what problem it solves, how that manifests, how I might
>reproduce it. I can guess that you may be using gentoo Linux from your
>e-mail but how you might have configured the ncurses build is not
>obvious. Just knowing what configure options are used to build it may
>help.
>
>Without it, does zsh fail to build or is some functionality or feature
>missing or broken?
>
>Is it linking to the wrong library? Or failing to resolve a symbol -
>which one(s)?
>
>Oliver

my apologies for not coming back at you after I raised that inquiry.

To my surprise, that patch we needed for quite a while in order to get
zsh/curses module compiled correctly against ncurses with separate tinfo
lib, now seems to be no longer necessary. I can only guess that it
might have been a toolchain issue back then or perhaps an older ncurses
lib that had the symbols in the wrong lib or a broken ncurses pkgconfig
file...

But in the end the patch is no longer necessary to be applied to zsh
sources because the curses module now gets correctly linked without
this patch as well.

Kind regards
-- 
Lars Wendler
Gentoo package maintainer
GPG: 21CC CF02 4586 0A07 ED93  9F68 498F E765 960E 9B39

[-- Attachment #2: Digitale Signatur von OpenPGP --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

end of thread, other threads:[~2021-04-12 18:47 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-23 16:05 Asking for status of a submitted patch Lars Wendler
2020-04-25  9:54 ` Daniel Shahaf
2021-04-11 21:02 ` Oliver Kiddle
2021-04-12 18:46   ` Lars Wendler

Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).