Page MenuHomeFreeBSD
Differential D27509

xargs: yesexpr is no longer basic RE
ClosedPublic
Actions

Authored by yuripv on Dec 8 2020, 7:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 23, 1:42 PM
Unknown Object (File)
Dec 20 2023, 7:00 AM
Unknown Object (File)
Nov 21 2023, 10:40 PM
Unknown Object (File)
Nov 21 2023, 9:53 PM
Unknown Object (File)
Nov 13 2023, 1:23 PM
Unknown Object (File)
Nov 11 2023, 10:55 AM
Unknown Object (File)
Sep 2 2023, 2:07 AM
Unknown Object (File)
Sep 2 2023, 2:05 AM
View All 20 Files
Subscribers
imp

Details

Reviewers
bapt
pfg
kevans
Commits
rS368580: xargs: compile yesexpr as ERE
Summary

PR: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=238762

Comple RE using REG_EXTENDED as it's what yesexpr is nowadays.

Test Plan
$ export LANG=en_AU.UTF-8
$ locale -k yesexpr
yesexpr="^(([yY]([eE][sS])?)|([yY]))"
$ echo test | xargs -p echo
echo test?...y
test

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

yuripv created this revision.Dec 8 2020, 7:13 AM
Herald added a subscriber: imp. · View Herald TranscriptDec 8 2020, 7:13 AM
yuripv requested review of this revision.Dec 8 2020, 7:13 AM
kevans accepted this revision.Dec 8 2020, 12:37 PM
This revision is now accepted and ready to land.Dec 8 2020, 12:37 PM
Closed by commit rS368580: xargs: compile yesexpr as ERE (authored by yuripv). · Explain WhyDec 12 2020, 3:39 PM
This revision was automatically updated to reflect the committed changes.
yuripv added a commit: rS368580: xargs: compile yesexpr as ERE.

Revision Contents
Changeset List

PathSize
head/
usr.bin/
xargs/
2 lines

Diff 80638

View Options

head/usr.bin/xargs/xargs.c

Loading...