Skip to content

Conversation

@billythedummy
Copy link
Contributor

Problem

Current minor version constraint (cargo default) of solana-transaction-context dependency is too loose because v3.1 is incompatible with current mollusk:

error[E0432]: unresolved import `solana_transaction_context::TransactionAccount`
 --> /home/user/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/mollusk-svm-keys-0.7.1/src/accounts.rs:9:70
  |
9 |     solana_transaction_context::{IndexOfAccount, InstructionAccount, TransactionAccount},
  |                                                                      ^^^^^^^^^^^^^^^^^^
  |                                                                      |
  |                                                                      no `TransactionAccount` in the root
  |                                                                      help: a similar name exists in the module: `TransactionAccounts`

For more information about this error, try `rustc --explain E0432`.
error: could not compile `mollusk-svm-keys` (lib) due to 1 previous error
warning: build failed, waiting for other jobs to finish...

Solution(?)

Tighten version constraint to patch version.

This is probably just a stopgap measure and the root cause will probably be resolved in the next release with upgraded solana-* deps anyway, so no need to do anything with this PR here, making it mainly for informational purposes

@Aursen
Copy link

Aursen commented Nov 18, 2025

LGTM, but why not update the dependencies directly?

@buffalojoec
Copy link
Collaborator

LGTM, but why not update the dependencies directly?

Agreed! We can just go to 3.1 for Agave deps and fix up any broken callsites.

@buffalojoec buffalojoec merged commit 8aedb7b into anza-xyz:main Nov 18, 2025
5 checks passed
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