stdlib: `make test` says argument list is too long
Checklist
Please ensure the following tasks are completed before filing an issue.
- Read and understood the Code of Conduct.
- Searched for existing issues and pull requests.
- If this is a general question, searched the FAQ for an existing answer.
- If this is a feature request, the issue name begins with
RFC:.
Description
Description of the issue (or feature request).
While setting up the development environment, if I run make test I get the following output
$ make test
make: execvp: /bin/sh: Argument list too long
make: *** [/home/g/stdlib/tools/make/lib/test/javascript.mk:35: test-javascript-local] Error 127
Other
Any other information relevant to this issue (or feature request)? This may include screenshots, references, stack traces, sample output, and/or implementation notes.
$ cat .git/refs/heads/develop
e920ce317ed95adef324aeae09e903849cb25a70
Reproduction
If this issue is a bug report, what steps are required to reproduce the unexpected output? (If this is a feature request, remove this section.)
In order to reproduce this bug, do the following:
- Clone the repository, and cd to it
- make install
- make test
Expected Results
What are the expected results? (If this is a feature request, remove this section.)
The following results are expected:
Correctly run the tests
Actual Results
What are the actual results? (If this is a feature request, remove this section.)
The following are the actual results:
make test exits with status=2, error=127
Environments
If this issue is a bug report, what environments are affected; e.g.,
Node v0.4.x,Chrome,IE 11? If Node.js, include thenpmversion, operating system, and any other potentially relevant platform information. (If this is a feature request, remove this section.)
The following environments are affected:
- nodejs v7.10.0
About this issue
- Original URL
- State: closed
- Created 7 years ago
- Comments: 17 (10 by maintainers)
Confirm. HEAD at
f09d20, applied patch (otherwise, “argument list too long”),make testrun successfully.@giuscri So I investigated further. In short, the REPL tests were buggy due to variable built-in REPL behavior. This should be addressed in recent commits. If you pull down the latest, the tests should run as expected, with no scary
SyntaxErroroutput.UPDATE: if you do pull down the latest, you will need to reapply your patch in order to avoid argument length errors in bash.