Differential D56476

du: Set BLOCKSIZE before running tests
ClosedPublic
Actions

Authored by des on Fri, Apr 17, 6:52 PM.

Details

Reviewers

kevans

Group Reviewers

Klara

Commits

rGd618ba314d14: du: Set BLOCKSIZE before running tests

Summary

Several testcases assume BLOCKSIZE=K, so set it at the top of the
script. This fixes an issue where the tests would sometimes fail
when run under sudo.

MFC after: 1 week
Sponsored by: Klara, Inc.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.Fri, Apr 17, 6:52 PM

Herald added a subscriber: imp. · View Herald TranscriptFri, Apr 17, 6:52 PM

des requested review of this revision.Fri, Apr 17, 6:52 PM

Harbormaster completed remote builds in B72295: Diff 175775.Fri, Apr 17, 6:52 PM

Nice catch

This revision is now accepted and ready to land.Sat, Apr 18, 11:04 AM

Closed by commit rGd618ba314d14: du: Set BLOCKSIZE before running tests (authored by des). · Explain WhySat, Apr 18, 12:48 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGd618ba314d14: du: Set BLOCKSIZE before running tests.

Revision Contents
Changeset List

Path

Size

usr.bin/

du/

tests/

du_test.sh

4 lines

Diff 175835

View Options

usr.bin/du/tests/du_test.sh

du: Set BLOCKSIZE before running testsClosedPublicActions