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

Update patches #514

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update patches #514

wants to merge 1 commit into from

Conversation

smoelius
Copy link
Collaborator

@smoelius smoelius commented Feb 7, 2025

No description provided.

@smoelius
Copy link
Collaborator Author

smoelius commented Feb 7, 2025

With just this change:

diff --git a/programs/bpf_loader/Cargo.toml b/programs/bpf_loader/Cargo.toml
index 0503c51a8f..cfed8feeb2 100644
--- a/programs/bpf_loader/Cargo.toml
+++ b/programs/bpf_loader/Cargo.toml
@@ -52,6 +52,8 @@ solana-transaction-context = { workspace = true, features = ["bincode"] }
 solana-type-overrides = { workspace = true }
 thiserror = { workspace = true }
 
+test-fuzz = { path = "../../../../test-fuzz" }
+
 [dev-dependencies]
 assert_matches = { workspace = true }
 criterion = { workspace = true }

This command fails:

$ cargo check -p solana-account-info
error: failed to load source for dependency `solana-account-info`

Caused by:
  Unable to update /Users/samuel.moelius/gh/trailofbits/test-fuzz/third-party/agave/sdk/account-info

Caused by:
  failed to parse manifest at `/Users/samuel.moelius/gh/trailofbits/test-fuzz/third-party/agave/sdk/account-info/Cargo.toml`

Caused by:
  error inheriting `edition` from workspace root manifest's `workspace.package.edition`

Caused by:
  `workspace.package.edition` was not defined

@smoelius
Copy link
Collaborator Author

smoelius commented Feb 7, 2025

^ This is on commit f36a62dce8bbd9ee41c93626e86872241df31119.

@smoelius
Copy link
Collaborator Author

smoelius commented Feb 7, 2025

This appears to be the commit where things go awry: e0a2e662520dd8b3b06f528ff9562dc60b42087e

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