Re: Cygwin Package ncurses 6.3-20221224 ABI > 9 config patches

[Sam James]

> On 1 Jan 2023, at 15:38, Thomas Dickey <dickey@his.com> wrote:
> 
> On Sun, Jan 01, 2023 at 08:19:36AM -0700, Brian Inglis wrote:
>> Happy New Year Thomas,
>> 
>> Just catching up on the latest package updates over the holiday season.
>> 
>> When building ncurses 6.3-20221224, got compile errors with
>> NCURSES_MOUSE_VERSION not set in curses.h, coming from configure.in,
>> autoconf generated as configure, run with option --with-abi-version=10,
>> which has an issue with the case $cf_cv_abi_version block setting only for
>> ABI 5-9.
>> 
>> I fixed my issue by setting the variable in a case branch handling 1[0-9],
>> initially to 4, but that is checked and reset to '', so set to 3, although
>> it should probably be 2 as the ABI version is originally 6.
> 
> ah... I overlooked that some packagers renumber the ABI's.
> 
> The basic problem there is that someone notices a functional change
> which doesn't affect the binary interface, and decides to increment
> the ABI.  (I don't recall anyone asking me before they did that).
> 
> I suppose a fix here would be to add options to set the mouse-version
> and perhaps a few other ABI-dependent features explicitly (and warn rather
> than error-out when a mismatch is seen).

I'm very surprised any packagers do that. Definite misbehaviour. :(

signature.asc
Description: Message signed with OpenPGP