CVS commit: pkgsrc/games/gnuchess

["Kevin Bloom" <ktnb%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   ktnb
Date:           Fri Oct 16 17:53:01 UTC 2026

Modified Files:
        pkgsrc/games/gnuchess: distinfo
Added Files:
        pkgsrc/games/gnuchess/patches: patch-src_adapter_main.cpp
            patch-src_engine_protocol.cpp patch-src_frontend_input.cc

Log Message:
gnuchess: fixed endless loop when building games/gnuchess-book


To generate a diff of this commit:
cvs rdiff -u -r1.21 -r1.22 pkgsrc/games/gnuchess/distinfo
cvs rdiff -u -r0 -r1.1 \
    pkgsrc/games/gnuchess/patches/patch-src_adapter_main.cpp \
    pkgsrc/games/gnuchess/patches/patch-src_engine_protocol.cpp \
    pkgsrc/games/gnuchess/patches/patch-src_frontend_input.cc

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/games/gnuchess/distinfo
diff -u pkgsrc/games/gnuchess/distinfo:1.21 pkgsrc/games/gnuchess/distinfo:1.22
--- pkgsrc/games/gnuchess/distinfo:1.21 Thu Oct 15 20:23:50 2026
+++ pkgsrc/games/gnuchess/distinfo      Fri Oct 16 17:53:00 2026
@@ -1,8 +1,11 @@
-$NetBSD: distinfo,v 1.21 2026/10/15 20:23:50 ktnb Exp $
+$NetBSD: distinfo,v 1.22 2026/10/16 17:53:00 ktnb Exp $
 
 BLAKE2s (gnuchess-6.3.0.tar.gz) = 6789eb377fa5f44a8c7f68fde1718f65863a40a392e85c66be3712cddedf6535
 SHA512 (gnuchess-6.3.0.tar.gz) = 093bdf37ab67d55ff449c8367ccf0e0c52012ee164a0cf11b42c44c4025df0996c9efdcf42136746633d2d020d7fb8f338c1b04dcf8044e48629f1da7642fb81
 Size (gnuchess-6.3.0.tar.gz) = 835620 bytes
+SHA1 (patch-src_adapter_main.cpp) = 5e4ad202ec4823feb15e2b5814a263cac7c6e391
 SHA1 (patch-src_adapter_pgn.cpp) = bbb71c1cd504b40d544fa0a0389b2c2d05fcb025
+SHA1 (patch-src_engine_protocol.cpp) = d50430cb78560104aad56fcf12e2b8f1aca2f6bb
 SHA1 (patch-src_frontend_cmd.cc) = 19f86e48a87bc9049cdb5eefc1ae20ac4e6329b6
+SHA1 (patch-src_frontend_input.cc) = bd212b47a1b636fbce4fab9ba657749e988ff65d
 SHA1 (patch-src_frontend_move.cc) = 53fc658e3ddf277623d730897fae9a2c0b048e68

Added files:

Index: pkgsrc/games/gnuchess/patches/patch-src_adapter_main.cpp
diff -u /dev/null pkgsrc/games/gnuchess/patches/patch-src_adapter_main.cpp:1.1
--- /dev/null   Fri Oct 16 17:53:01 2026
+++ pkgsrc/games/gnuchess/patches/patch-src_adapter_main.cpp    Fri Oct 16 17:53:01 2026
@@ -0,0 +1,18 @@
+$NetBSD: patch-src_adapter_main.cpp,v 1.1 2026/10/16 17:53:01 ktnb Exp $
+
+Don't die when file doesn't exist
+
+--- src/adapter/main.cpp.orig  2026-10-16 17:24:47.983164281 +0000
++++ src/adapter/main.cpp
+@@ -167,8 +167,9 @@ static void parse_option() {
+ 
+    file_name = option_get_string("OptionFile");
+ 
+-   file = fopen(file_name,"r");
+-   if (file == NULL) my_fatal( "Can't open file \"%s\": %s - using defaults\n",file_name,strerror(errno));
++   file = file_name ? fopen(file_name,"r") : NULL;;
++   if (file == NULL && file_name != NULL)
++     my_log( "Can't open file \"%s\": %s - using defaults\n",file_name,strerror(errno));
+ 
+    // PolyGlot options (assumed first)
+ 
Index: pkgsrc/games/gnuchess/patches/patch-src_engine_protocol.cpp
diff -u /dev/null pkgsrc/games/gnuchess/patches/patch-src_engine_protocol.cpp:1.1
--- /dev/null   Fri Oct 16 17:53:01 2026
+++ pkgsrc/games/gnuchess/patches/patch-src_engine_protocol.cpp Fri Oct 16 17:53:01 2026
@@ -0,0 +1,22 @@
+$NetBSD: patch-src_engine_protocol.cpp,v 1.1 2026/10/16 17:53:01 ktnb Exp $
+
+Actually quit the loop if the engine has quit
+
+--- src/engine/protocol.cpp.orig       2026-10-16 17:42:47.125065148 +0000
++++ src/engine/protocol.cpp
+@@ -109,7 +109,14 @@ void loop() {
+ 
+    // loop
+ 
+-   while (true) loop_step();
++   while (true) {
++     bool quit;
++     pthread_mutex_lock( &engine_quit_mutex );
++     quit = engine_quit;
++     pthread_mutex_unlock( &engine_quit_mutex );
++     if (quit) break;
++     loop_step();
++   }
+ }
+ 
+ // init()
Index: pkgsrc/games/gnuchess/patches/patch-src_frontend_input.cc
diff -u /dev/null pkgsrc/games/gnuchess/patches/patch-src_frontend_input.cc:1.1
--- /dev/null   Fri Oct 16 17:53:01 2026
+++ pkgsrc/games/gnuchess/patches/patch-src_frontend_input.cc   Fri Oct 16 17:53:01 2026
@@ -0,0 +1,17 @@
+$NetBSD: patch-src_frontend_input.cc,v 1.1 2026/10/16 17:53:01 ktnb Exp $
+
+On a EOF from stdin, just update the the user input rather than
+flagging as quit. This prevents issues with fgets(3) throwing an
+interrupted system call.
+
+--- src/frontend/input.cc.orig 2025-07-12 10:00:07.000000000 +0000
++++ src/frontend/input.cc
+@@ -152,7 +152,7 @@ void *input_func(void *arg __attribute__((unused)) )
+             (RealGameCnt+1)/2 + 1 );
+     }
+     if (!get_line(prompt)) {
+-      SET(flags, QUIT);
++      strcpy(userinputstr , "quit\n");
+     }
+     SendToFrontend( userinputstr );
+ #ifdef HAVE_LIBREADLINE