Skip to content

Commit

Permalink
[interpreter] Update Makefile to ignore output of cd
Browse files Browse the repository at this point in the history
The interpreter Makefile captures the output of cding into the unittest
directory and listing the unittest files it contains. Under some circumstances,
the `cd` command will print the new current directory, and that output was
incorrectly being interpreted as the path to a unittest, causing the build to
fail.

Fix this problem by redirecting the output of cd, if any, to /dev/null.
  • Loading branch information
tlively committed Sep 21, 2023
1 parent b174a7d commit 2210e03
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion interpreter/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ $(JSLIB):
# Unit tests

UNITTESTDIR = unittest
UNITTESTFILES = $(shell cd $(UNITTESTDIR); ls *.ml)
UNITTESTFILES = $(shell cd $(UNITTESTDIR) > /dev/null; ls *.ml)
UNITTESTS = $(UNITTESTFILES:%.ml=%)

.PHONY: unittest
Expand Down

0 comments on commit 2210e03

Please sign in to comment.