Skip to content
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

yarn synthetic-chain build relies on tsx which may not be installed #110

Closed
ivanlei opened this issue Feb 18, 2024 · 2 comments · Fixed by #111
Closed

yarn synthetic-chain build relies on tsx which may not be installed #110

ivanlei opened this issue Feb 18, 2024 · 2 comments · Fixed by #111
Assignees

Comments

@ivanlei
Copy link

ivanlei commented Feb 18, 2024

Description

When attempting to yarn synthetic-chain build in agoric-sdk/a3p-integration I ran into an issue where tsx was not installed:

$ yarn synthetic-chain build
$ /Users/ivanleichtling/source/agoric/agoric-sdk/a3p-integration/node_modules/.bin/synthetic-chain build
env: tsx: No such file or directory
error Command failed with exit code 127.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 127.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 127.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

The issue seems to stem from the removal of tsx and a dependency in package.json in #102 while not fully removing the production dependency on tsx

Impact

This breaks local builds of agoric-sdk when tsx is not globally installed.

Work Around

$ npm install --global 'tsx@^3.14.0' 
@ivanlei
Copy link
Author

ivanlei commented Feb 18, 2024

Note CI does not catch this which suggests CI globally installs tsx.

@mhofman
Copy link
Member

mhofman commented Feb 18, 2024

I'm not sure how I didn't catch this. tsx is not installed on my machine, but synthetic-chain does start correctly.

CI does still install tsx globally I believe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants