Re: [PATCH 1/5] Don't include sys/ioctl.h and termios.h on Windows

Akim Demaille <[email protected]>
Newsgroups gmane.comp.parsers.bison.patches
Message-ID <[email protected]>
Hi Maarten,

I will install the patch below instead.  Could you please check this tarball asap?  I would like to release 3.7.1 tomorrow.

Cheers!

https://www.lrde.epita.fr/~akim/private/bison/bison-3.7.20-b4032.tar.gz
https://www.lrde.epita.fr/~akim/private/bison/bison-3.7.20-b4032.tar.lz
https://www.lrde.epita.fr/~akim/private/bison/bison-3.7.20-b4032.tar.xz


commit b40326ac48a5aeeb9cd05eefeadbc11b9292eaa2
Author: Akim Demaille <[email protected]>
Date:   Sat Aug 1 21:03:17 2020 +0200

    portability: we use termios.h and sys/ioctl.h
    
    Reported by Maarten De Braekeleer.
    https://lists.gnu.org/r/bison-patches/2020-07/msg00079.html
    
    * bootstrap.conf (gnulib_modules): Add termios and sys_ioctl.

diff --git a/NEWS b/NEWS
index 717ff8a9..f04a0b55 100644
--- a/NEWS
+++ b/NEWS
@@ -8,6 +8,8 @@ GNU Bison NEWS
 
   Portability issues with libtextstyle.
 
+  Portability issues of Bison itself with MSVC.
+
 ** Changes
 
   Improvements and fixes in the documentation.
diff --git a/TODO b/TODO
index 1b9fb510..b8b2befb 100644
--- a/TODO
+++ b/TODO
@@ -467,6 +467,10 @@ both deterministic and generalized parsing.
 
 It would be great for D to also support this.
 
+The basic ideas of GLR are explained here:
+
+https://www.codeproject.com/Articles/5259825/GLR-Parsing-in-Csharp-How-to-Use-The-Most-Powerful
+
 * Better error messages
 The users are not provided with enough tools to forge their error messages.
 See for instance "Is there an option to change the message produced by
diff --git a/bootstrap.conf b/bootstrap.conf
index 3f2fa842..577e30e5 100644
--- a/bootstrap.conf
+++ b/bootstrap.conf
@@ -45,6 +45,8 @@ gnulib_modules='
   relocatable-prog relocatable-script
   rename
   spawn-pipe stdbool stpcpy stpncpy strdup-posix strerror strverscmp
+  sys_ioctl
+  termios
   timevar
   unicodeio unistd unistd-safer unlink unlocked-io
   update-copyright unsetenv verify
diff --git a/lib/.gitignore b/lib/.gitignore
index 758edcc0..58aeac63 100644
--- a/lib/.gitignore
+++ b/lib/.gitignore
@@ -336,6 +336,7 @@
 /sys_types.in.h
 /sys_wait.in.h
 /sysexits.in.h
+/termios.in.h
 /textstyle.h
 /textstyle.in.h
 /time.h
diff --git a/m4/.gitignore b/m4/.gitignore
index 9bc07270..8d7a965a 100644
--- a/m4/.gitignore
+++ b/m4/.gitignore
@@ -183,6 +183,7 @@
 /strndup.m4
 /strnlen.m4
 /strverscmp.m4
+/sys_ioctl_h.m4
 /sys_resource_h.m4
 /sys_socket_h.m4
 /sys_stat_h.m4
@@ -190,6 +191,7 @@
 /sys_times_h.m4
 /sys_types_h.m4
 /sys_wait_h.m4
+/termios_h.m4
 /threadlib.m4
 /time_h.m4
 /timespec.m4
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.