Mon Apr 1 10:26:42 2019 UTC ()
nano: fixing a possible non-functioning ^S keystroke


(wiedi)
diff -r1.78 -r1.79 pkgsrc/editors/nano/Makefile
diff -r1.50 -r1.51 pkgsrc/editors/nano/distinfo
diff -r0 -r1.3 pkgsrc/editors/nano/patches/patch-src_files.c

cvs diff -r1.78 -r1.79 pkgsrc/editors/nano/Makefile (expand / switch to unified diff)

--- pkgsrc/editors/nano/Makefile 2019/03/24 19:42:34 1.78
+++ pkgsrc/editors/nano/Makefile 2019/04/01 10:26:42 1.79
@@ -1,16 +1,17 @@ @@ -1,16 +1,17 @@
1# $NetBSD: Makefile,v 1.78 2019/03/24 19:42:34 wiedi Exp $ 1# $NetBSD: Makefile,v 1.79 2019/04/01 10:26:42 wiedi Exp $
2 2
3DISTNAME= nano-4.0 3DISTNAME= nano-4.0
 4PKGREVISION= 1
4CATEGORIES= editors 5CATEGORIES= editors
5MASTER_SITES= https://www.nano-editor.org/dist/v${PKGVERSION_NOREV:R}/ 6MASTER_SITES= https://www.nano-editor.org/dist/v${PKGVERSION_NOREV:R}/
6EXTRACT_SUFX= .tar.xz 7EXTRACT_SUFX= .tar.xz
7 8
8MAINTAINER= wiedi@frubar.net 9MAINTAINER= wiedi@frubar.net
9HOMEPAGE= https://www.nano-editor.org/ 10HOMEPAGE= https://www.nano-editor.org/
10COMMENT= Small and friendly text editor (a free replacement for Pico) 11COMMENT= Small and friendly text editor (a free replacement for Pico)
11LICENSE= gnu-gpl-v3 12LICENSE= gnu-gpl-v3
12 13
13USE_PKGLOCALEDIR= yes 14USE_PKGLOCALEDIR= yes
14USE_TOOLS+= msgfmt gmake 15USE_TOOLS+= msgfmt gmake
15GNU_CONFIGURE= yes 16GNU_CONFIGURE= yes
16CONFIGURE_ARGS+= --enable-color 17CONFIGURE_ARGS+= --enable-color

cvs diff -r1.50 -r1.51 pkgsrc/editors/nano/distinfo (expand / switch to unified diff)

--- pkgsrc/editors/nano/distinfo 2019/03/24 19:42:34 1.50
+++ pkgsrc/editors/nano/distinfo 2019/04/01 10:26:42 1.51
@@ -1,7 +1,8 @@ @@ -1,7 +1,8 @@
1$NetBSD: distinfo,v 1.50 2019/03/24 19:42:34 wiedi Exp $ 1$NetBSD: distinfo,v 1.51 2019/04/01 10:26:42 wiedi Exp $
2 2
3SHA1 (nano-4.0.tar.xz) = 586f369d622a883acf06d1143a7b3c8754acc011 3SHA1 (nano-4.0.tar.xz) = 586f369d622a883acf06d1143a7b3c8754acc011
4RMD160 (nano-4.0.tar.xz) = b6bb83e30ec4484a6fa8f3d5366387918a480e1f 4RMD160 (nano-4.0.tar.xz) = b6bb83e30ec4484a6fa8f3d5366387918a480e1f
5SHA512 (nano-4.0.tar.xz) = 2cef28df05f890bc27ef8faa9eb5350f8de2cc6cc69b4dc8e11c19248911630a7088ddbee57c8a8aa56df6efaab13124132d0cda55e5ae1f151265723d7d0ede 5SHA512 (nano-4.0.tar.xz) = 2cef28df05f890bc27ef8faa9eb5350f8de2cc6cc69b4dc8e11c19248911630a7088ddbee57c8a8aa56df6efaab13124132d0cda55e5ae1f151265723d7d0ede
6Size (nano-4.0.tar.xz) = 1533992 bytes 6Size (nano-4.0.tar.xz) = 1533992 bytes
7SHA1 (patch-configure) = 4a618bf99332fcfce2413be14400b919d042cc30 7SHA1 (patch-configure) = 4a618bf99332fcfce2413be14400b919d042cc30
 8SHA1 (patch-src_files.c) = 59c486e32c18f883139c918fe846946a67a826a6

File Added: pkgsrc/editors/nano/patches/Attic/patch-src_files.c
$NetBSD: patch-src_files.c,v 1.3 2019/04/01 10:26:42 wiedi Exp $

The lack of initialization caused a nasty bug on some targets (such as
ARMv7) which would make it so that ^S would just say "Cancelled".

While x86 (both 64 and 32 bits) seems to initialize 'response' to zero or
a positive number, ARM does not, and there is usually a negative value in
its place, which triggers the 'if (response < 0)' check and, as a result,
the code says "Cancelled".

This fixes https://savannah.gnu.org/bugs/?56023.
Reported-by: Devin Hussey <husseydevin@gmail.com>

Bug existed since version 4.0, commit 0f9d60a3.

--- src/files.c.orig	2019-03-23 08:26:13.000000000 +0000
+++ src/files.c
@@ -2101,7 +2101,7 @@ int do_writeout(bool exiting, bool withp
 
 	while (TRUE) {
 		const char *msg;
-		int response, choice;
+		int response = 0, choice = 0;
 		functionptrtype func;
 #ifndef NANO_TINY
 		const char *formatstr, *backupstr;