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

Unresolved warnings on build #10

Open
pyrex41 opened this issue Jul 13, 2022 · 5 comments
Open

Unresolved warnings on build #10

pyrex41 opened this issue Jul 13, 2022 · 5 comments

Comments

@pyrex41
Copy link

pyrex41 commented Jul 13, 2022

Using Apple M1 Max.

npm run initialize runs / completes fine (once I initialized the ipfs daemon -- that wasn't mentioned in Readme but dug through the error messages)

I get the following error when I run npm run build-general:
Screen Shot 2022-07-12 at 5 23 43 PM

@pyrex41
Copy link
Author

pyrex41 commented Jul 13, 2022

should note that I am able to use snek just fine by just aliasing snek to node ~/snek/index.js $argv

@dmfxyz
Copy link
Contributor

dmfxyz commented Jul 13, 2022

Can you confirm that npm run build-general doesn't produce a usable snek binary in the same directory as the repo? I get similar warnings on mac but still get snek built.

these warnings should be looked at, just want to understand the issue.

@nmushegian
Copy link
Contributor

I think building a single binary like this is pretty non-standard, wondering if we can't just go with npm i -g type of thing. @stobiewan thoughts

@pyrex41 pyrex41 changed the title Not able to build binaries on Mac Unresolved warnings on build Jul 13, 2022
@pyrex41
Copy link
Author

pyrex41 commented Jul 13, 2022

@dmfxyz you are correct, the binary actually does build; apparently I missed / made assumptions based on the errors. Updated issue title accordingly.

@stobiewan
Copy link
Contributor

Originally added "build-linux" script just because it's helpful for testing and using myself. Something like npm install -g would be ideal, assume vyper+python devs are OK with using npm to get snek.

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

No branches or pull requests

4 participants