( )⚙ D8500 tests: use regular POSIX shell

This is an archive of the discontinued Mercurial Phabricator instance.

tests: use regular POSIX shell
ClosedPublic

Authored by joerg.sonnenberger on May 7 2020, 9:02 AM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Commits
rHGf727939f3513: tests: use regular POSIX shell
Summary

wait-on-file requires one POSIX extension (sleep with non-integral
argument), but it doesn't require any bash extensions, so just require a
normal POSIX shell. While here, use consistent formatting without
redundant ;

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Moving to /bin/sh seems fine. Did you run tests for this patch on debian/dash ? (pushing to heptapod should do it)

tests/testlib/wait-on-file
26

Unrelated gratuitous change ?

31

same here

This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.