du: Set BLOCKSIZE before running tests
ClosedPublic
Actions

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

Details

Reviewers

Group Reviewers

Commits

rGc2b8a5769704: du: Set BLOCKSIZE before running tests
rG41ba2e3cb443: du: Set BLOCKSIZE before running tests
rG10ed7115165e: du: Set BLOCKSIZE before running tests
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 Skipped

Unit

Tests Skipped

Build Status

Buildable 72295
Build 69178: arc lint + arc unit

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.

des added a commit: rG10ed7115165e: du: Set BLOCKSIZE before running tests.Wed, Apr 22, 8:21 AM

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

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

Revision Contents
Changeset List

Path

Size

usr.bin/

du/

tests/

du_test.sh

4 lines

Diff 175775

View Options

usr.bin/du/tests/du_test.sh

du: Set BLOCKSIZE before running testsClosedPublicActions