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

Support Docker Workspace Root Dir #129

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rauljordan
Copy link
Contributor

Description

When running a project example that has a relative dependency to a workspace, such as the erc20 below, cargo stylus deploy will fail as the Docker container does not include the absolute workspace directory to look up the dependencies.

[package]
name = "erc20"
version = "0.1.0"
edition = "2021"

[dependencies]
alloy-primitives = "=0.8.13"
alloy-sol-types = "=0.8.13"
stylus-sdk = { path = "../../stylus-sdk" }
mini-alloc = { path = "../../mini-alloc" }

This PR attempts to fix the issue by allowing the user to specify the workspace directory to Docker. However, this is running into issues where we don't really support workspace directories in cargo stylus. As such, will mark as a draft until we have a longer-term solution here.

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 this pull request may close these issues.

1 participant