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

avm: Remove avm from workspaces #1502

Merged

Conversation

tomlinton
Copy link
Contributor

Move avm out of the workspace config to avoid the error below on cargo build in the root. It looks like when Cargo has a --git argument it looks in the whole repo for a package named avm so I don't believe we need to update the install command for avm in the book.

warning: output filename collision.
The bin target `anchor` in package `avm v0.22.0 (/Users/tom/Code/anchor/avm)` has the same output filename as the bin target `anchor` in package `anchor-cli v0.22.0 (/Users/tom/Code/anchor/cli)`.
Colliding filename is: /Users/tom/Code/anchor/target/debug/anchor.dSYM
The targets should have unique names.
Consider changing their names to be unique or compiling them separately.
This may become a hard error in the future; see <https://github.com/rust-lang/cargo/issues/6313>.

@paul-schaaf
Copy link
Contributor

yup I can still download avm with the same command from your fork

@paul-schaaf paul-schaaf merged commit 4c9866f into coral-xyz:master Apr 10, 2022
@paul-schaaf
Copy link
Contributor

thank you!

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.

2 participants