On Fri, May 16, 2025 at 09:05:14AM -0500, Eric Blake wrote: > 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^
Typo, omit the ^ in that line. > $ echo 'len(include(list))' | m4p > 8196 > -- Eric Blake, Principal Software Engineer Red Hat, Inc. Virtualization: qemu.org | libguestfs.org
