-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build all test projects in temporary folder #31
Conversation
@megri Could you please check whether the tests pass on macOS? |
I'm still getting errors. Full log here
|
I fixed the failing |
New log:
|
I believe Node.js is not installed on your system. After I removed Node.js here, the log output looks identical to yours. This actually shows a usability problem in Bloop as it only returns a non-zero exit code without writing the cause of failure to stderr: /tmp/seed1566034107928575769/example-paradise-platform $ bloop run example-js
Generated JavaScript file '/tmp/seed1566034107928575769/example-paradise-platform/build/example.js'
/tmp/seed1566034107928575769/example-paradise-platform $ echo $?
64 |
- Use `createTempDirectory` instead of `/tmp` for better portability - PathUtil: Use `normalisePath` as `getCanonicalPath` resolves symbolic links The first change makes the test suite compatible with macOS. Closes #30.
I do have node.js installed.
|
I tried running
|
This looks like a problem with Bloop indeed. You could pass in |
createTempDirectory
instead of/tmp
for better portabilitynormalisePath
asgetCanonicalPath
resolvessymbolic links
The first change makes the test suite compatible with macOS.
Closes #30.