Page MenuHomeFreeBSD
Differential D35876

sh: nullify ENV in tests
ClosedPublic
Actions

Authored by pstef on Jul 21 2022, 6:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 26, 6:27 AM
Unknown Object (File)
Mon, Nov 25, 6:01 PM
Unknown Object (File)
Sat, Nov 16, 6:47 AM
Unknown Object (File)
Sat, Nov 16, 6:45 AM
Unknown Object (File)
Sat, Nov 16, 3:51 AM
Unknown Object (File)
Sep 27 2024, 6:30 PM
Unknown Object (File)
Sep 24 2024, 10:11 AM
Unknown Object (File)
Sep 24 2024, 6:33 AM
View All 64 Files
Subscribers
imp

Details

Reviewers
jilles
Commits
rGa1423456415a: sh: nullify ENV in tests
Summary

This is to avoid loading .shrc which may contain commands that would result in output different than expected.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

pstef created this revision.Jul 21 2022, 6:45 PM
Herald added a reviewer: jilles. · View Herald TranscriptJul 21 2022, 6:45 PM
Herald added a subscriber: imp. · View Herald Transcript
pstef requested review of this revision.Jul 21 2022, 6:45 PM
jilles accepted this revision.Jul 22 2022, 8:47 AM

Thanks, I have also encountered this issue.

Perhaps this approach, adding ENV= before $SH -i uses in test scripts, is best since it supports running single test scripts without any wrapping. There are not that many test scripts that use interactive mode.

This revision is now accepted and ready to land.Jul 22 2022, 8:47 AM
Closed by commit rGa1423456415a: sh: nullify ENV in tests (authored by pstef). · Explain WhyAug 20 2022, 11:31 AM
This revision was automatically updated to reflect the committed changes.
pstef added a commit: rGa1423456415a: sh: nullify ENV in tests.

Revision Contents
Changeset List

PathSize
bin/
sh/
tests/
builtins/
2 lines
2 lines
execution/
2 lines

Diff 108399

View Options

bin/sh/tests/builtins/cd10.0

Loading...
View Options

bin/sh/tests/builtins/fc1.0

Loading...
View Options

bin/sh/tests/execution/int-cmd1.0

Loading...