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

Missing github or pypi dir on install will trigger Vyper error No such file or directory #209

Open
s3bc40 opened this issue Feb 21, 2025 · 2 comments
Labels

Comments

@s3bc40
Copy link
Contributor

s3bc40 commented Feb 21, 2025

It happens when we install a project with only pypi dependencies (or only GitHub). It seems that telling Vyper compiler to check the specific lib dirs triggers the errors:

"vyper.command": "vyper -p ./lib/github -p ./lib/pypi"

Image

FileNotFoundError: [Errno 2] 
No such file or directory: '/home/s3bc40/mox-cu/mox-signature-cu/lib/github'
FileNotFoundError: [Errno 2] No such file or directory: '/home/s3bc40/mox-cu/mox-signature-cu/lib/github'

Fixed if we manually add github dir inside the lib dir. But maybe we can do something about it?

Moccasin CLI v0.3.6

@PatrickAlphaC
Copy link
Member

Ah good shout. Yeah, we can just have the tool auto-add the folder.

@s3bc40
Copy link
Contributor Author

s3bc40 commented Feb 22, 2025

Yes, it could be added by default. Maybe after this addition we need to check the tests. Since some of them assert if the specific lib dir exist.

s3bc40 added a commit to s3bc40/moccasin that referenced this issue Feb 25, 2025
- add tests for install command
- add tests for impacted commands
    - run
    - test
    - deploy
    - compile
- using helper utility to handle moccasin toml scenarios
- add auto-install lib/github and lib/pypi folders when install (fix issue Cyfrin#209)
- fix issues with tests using COMPLEX_PROJECT path and not temporary one
- update test constants

WARNING: just test-z and live test KO since anvil-zksync is not installed (reported Cyfrin#216)
PatrickAlphaC added a commit that referenced this issue Mar 5, 2025
* feat: auto-install dependencies

* feat: implement tests for install command

- add tests for install command
- add tests for impacted commands
    - run
    - test
    - deploy
    - compile
- using helper utility to handle moccasin toml scenarios
- add auto-install lib/github and lib/pypi folders when install (fix issue #209)
- fix issues with tests using COMPLEX_PROJECT path and not temporary one
- update test constants

WARNING: just test-z and live test KO since anvil-zksync is not installed (reported #216)

* fix: refactor auto-install tests

* fix: lint

---------

Co-authored-by: s3bc40 <s3bc40@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants