On Fri, May 16, 2025 at 08:16:24AM -0500, Eric Blake wrote:
> Since you continue to patch things as fast as I report them (now
> testing with commit a4cc0f3c):
>
* At the command line, m4 treats -d as taking an optional argument
(POSIX discourages that practice, but m4 was doing it long before
POSIX, and since -d is a POSIX extension, it's better to keep status
quo). I don't know how easy or hard it is to make a short option take
an optional argument with python's argument parsing libraries; and I
also know you are still fine-tuning how debugmode() and --debug
behave:
$ echo 'define(a,1)a' | m4 -daet
m4trace: -1- define(a)
m4trace: -1- a -> 1
1
$ echo 'define(a,1)a' | m4p -daet
usage: m4p [-h] [-P] [-v] [-I DIRECTORY] [-D NAME[=VALUE]] [-U NAME]
[--debugfile [FILE]] [-d] [--debug FLAGS] [-t NAME]
[FILE ...]
m4p: error: unrecognized arguments: -aet
$ echo 'define(a,1)a' | m4p --debug aet
1
* Commit 45fdad3 broke reading large files:
$ seq 2000 > list
$ git checkout 45fdad3^
$ echo 'len(include(list))' | m4p
8893
$ git checkout 45fdad3^
$ echo 'len(include(list))' | m4p
8196
--
Eric Blake, Principal Software Engineer
Red Hat, Inc.
Virtualization: qemu.org | libguestfs.org
