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

fix: Update and fix dockerfile #54

Closed
wants to merge 3 commits into from

Conversation

austinabell
Copy link

Dockerfile was broken for a few reasons:

  • Copying source code fails in root directory (not sure how this ever worked before)
  • Outdated foundry install (also switched default to just using foundryup to be faster)
  • risc0 toolchain wasn't installed
    • I just used the same base image as the risc0 dockerfile, to ensure compatibility

doesn't matter to me if this is pulled in, just made these updates for my own use and figured I'd upstream

@nategraf nategraf closed this Feb 27, 2024
@nategraf nategraf reopened this Feb 27, 2024
@nategraf nategraf changed the base branch from main to relay February 27, 2024 00:48
@carterbrett
Copy link
Contributor

Decision in sync was to close this for now

@carterbrett carterbrett closed this Mar 5, 2024
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.

3 participants