This is the mail archive of the
cygwin
mailing list for the Cygwin project.
mintty needs a patch
- From: "Tony Kelman" <tony at kelman dot net>
- To: <cygwin at cygwin dot com>
- Cc: <mintty-discuss at groups dot google dot com>
- Date: Sat, 14 Feb 2015 23:10:07 -0800
- Subject: mintty needs a patch
- Authentication-results: sourceware.org; auth=none
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.
-Tony
--
Problem reports: http://cygwin.com/problems.html
FAQ: http://cygwin.com/faq/
Documentation: http://cygwin.com/docs.html
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple