Skip to content

Portability fixes - #556

Closed
0-wiz-0 wants to merge 3 commits into
linux-test-project:masterfrom
0-wiz-0:master
Closed

0-wiz-0 wants to merge 3 commits into
linux-test-project:masterfrom
0-wiz-0:master

Conversation

@0-wiz-0

@0-wiz-0 0-wiz-0 commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

This addresses some issues found in #553

# test filename containing spaces
rm -rf ./mytest
mkdir -pv ./mytest
mkdir -p ./mytest

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread tests/common.tst
local class
for class in `find $dir -name '*.html' -print0 |
xargs -0 --no-run-if-empty grep -h -o -E 'class="[^"]+"' |
xargs -0 -r grep -h -o -E 'class="[^"]+"' |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@henry2cox confirmed via https://man.netbsd.org/xargs.1:

     -r      Compatibility with GNU xargs.  The GNU version of xargs runs the
             utility argument at least once, even if xargs input is empty, and
             it supports a -r option to inhibit this behavior.  The NetBSD
             version of xargs does not run the utility argument on empty
             input, but it supports the -r option for command-line compatibil-
             ity with GNU xargs; but the -r option does nothing in the NetBSD
             version of xargs.

@henry2cox

Copy link
Copy Markdown
Collaborator

subsumed by other PR

@henry2cox henry2cox closed this Oct 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants