diff options
author | Jarkko Kniivilä <jkniiv@gmail.com> | 2015-09-07 23:43:02 +0300 |
---|---|---|
committer | Mike Burns <mike@mike-burns.com> | 2015-11-01 16:54:34 +0100 |
commit | 5f592878eed79dd28f9f93418f9456185fd21ac9 (patch) | |
tree | 1e79fe49cbac3485bb0d25df3e9aca1f814d1e6e /CONTRIBUTING.md | |
parent | 39a32b9295c10176eeca3f21534caa0209cb1172 (diff) |
Do not depend on readlink(1) or -q
Solaris 10 lacks readlink(1). Additionally, its different grep(1) and
diff(1) do not take a `-q` flag.
Use a Perl one-liner instead of readlink(1) which is missing on Solaris 10.
Also because /usr/bin/grep and diff(1) don't understand the `-q` option, make
them go quiet by redirecting stdout to `/dev/null` instead.
The Perl dependency only exists in the test suite, so it does not incur
a runtime penalty for end users.
This is to work around the fact that readlink(1) is missing on Solaris.
The tradeoff is: on Solaris, installing third-party software (readlink)
is a pain, whereas on e.g. FreeBSD installing third-party software
(Perl) is simple and common.
Diffstat (limited to 'CONTRIBUTING.md')
-rw-r--r-- | CONTRIBUTING.md | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 59d240d..31eb49b 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -57,6 +57,9 @@ The test suite uses [cram][]. It is an integration suite, meaning the programs are exercised from the outside and assertions are made only on their output or effects. +The test suite requires Perl with the `Cwd` module. It expects to find Perl as +`perl` in `$PATH`. + All tests can be run like so: $ make check |