xargs/tests: Include expected output file for -nMAX_INT
ClosedPublic
Actions

Authored by jlduran_gmail.com on Jul 13 2023, 11:01 PM.

Details

Reviewers

des

Group Reviewers

Klara

Commits

rGd234c8228ced: xargs: Install missing test output.

Summary

Fixes: eab91d008165e7bbf8ca7b87eabe4dc8bf3da191 (partially)

Test Plan

As stated in D41023, on my mac xargs -n2147483647 < usr.bin/xargs/tests/regress.in, works fine.
However both, on my testing FreeBSD VM, and on CirrusCI (https://cirrus-ci.com/task/4770881373732864), it fails -- maybe I need to update the VMs.

At any rate, this patch is needed in order to run the tests.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

jlduran_gmail.com created this revision.Jul 13 2023, 11:01 PM

Herald added a subscriber: imp. · View Herald TranscriptJul 13 2023, 11:01 PM

jlduran_gmail.com requested review of this revision.Jul 13 2023, 11:01 PM

jlduran_gmail.com edited the test plan for this revision. (Show Details)

des accepted this revision as: des.Jul 13 2023, 11:34 PM

This revision is now accepted and ready to land.Jul 13 2023, 11:34 PM

Closed by commit rGd234c8228ced: xargs: Install missing test output. (authored by jlduran_gmail.com, committed by des). · Explain WhyJul 13 2023, 11:41 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGd234c8228ced: xargs: Install missing test output..

des mentioned this in rG15991a682445: xargs: fix -R so that it accepts negative numbers again.Sep 7 2023, 2:52 PM

Revision Contents
Changeset List

Path

Size

usr.bin/

xargs/

tests/

Makefile

3 lines

Diff 124647

View Options

usr.bin/xargs/tests/Makefile

xargs/tests: Include expected output file for -nMAX_INTClosedPublicActions