On Feb 14 23:10, Tony Kelman wrote:
> I was trying to build mintty from its src package today. As a heads-up
> for whoever next tries to build the package, I get the following error
> on either 32 or 64 bit:
>
> gcc -c -MMD -MP -DTARGET=x86_64-pc-cygwin -DNDEBUG -std=gnu99 -include std.h
> -Wall -Wextra -Wundef -Werror -mtune=atom -fomit-frame-pointer -O2 wintext.c
> windres --preprocessor 'gcc -E -xc -DRC_INVOKED -MMD -MP
> -DTARGET=x86_64-pc-cygwin -DNDEBUG' res.rc res.o
> wintext.c: In function ‘win_text’:
> wintext.c:522:13: error: array subscript is above array bounds
> [-Werror=array-bounds]
> if (!fonts[nfont]) {
> ^
> cc1: all warnings being treated as errors
> Makefile:117: recipe for target 'wintext.o' failed
> make: *** [wintext.o] Error 1
>
>
> This can at least be made into a warning by taking off -Werror, but it looks
> like it might be a real off-by-one. Our neighbors over at MSYS2 have a patch
> https://github.com/Alexpux/MSYS2-packages/blob/master/mintty/01-array-bounds.patch
> (the packaging repo there has a BSD 3-clause license) to change FONT_MAXNO
> in wintext.c from 0x1F to 0x20.Sadly the mintty maintainer is AWOL for quite some time. Would you like to take over maintainership? Given the overall stability of the package, maintaining it shouldn't be much of a burden. Thanks, Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Maintainer cygwin AT cygwin DOT com Red Hat
pgpeM5lnf4AGw.pgp
Description: PGP signature
