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 the npm version, 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)

Commits related to this issue

Most upvoted comments

Confirm. HEAD at f09d20, applied patch (otherwise, “argument list too long”), make test run 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 SyntaxError output.

UPDATE: if you do pull down the latest, you will need to reapply your patch in order to avoid argument length errors in bash.