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

Extract feature-set crate #2172

Merged
merged 21 commits into from
Sep 10, 2024
Merged

Conversation

kevinheavey
Copy link

Problem

solana_sdk::feature_set blocks a lot of things from being pulled out of the sdk, including sdk::transaction

Summary of Changes

  • Move feature_set.rs to its own crate
  • Re-export it with a deprecation notice in solana-sdk
  • Update all usage of feature_set in this repo to use the new crate

Copy link

mergify bot commented Jul 17, 2024

If this PR represents a change to the public RPC API:

  1. Make sure it includes a complementary update to rpc-client/ (example)
  2. Open a follow-up PR to update the JavaScript client @solana/web3.js (example)

Thank you for keeping the RPC clients in sync with the server API @kevinheavey.

@kevinheavey kevinheavey force-pushed the extract-feature-set branch 4 times, most recently from 55d2f0b to 17fa98b Compare July 25, 2024 14:27
@kevinheavey kevinheavey force-pushed the extract-feature-set branch 4 times, most recently from 7c014d6 to 4ef520a Compare August 2, 2024 09:47
@kevinheavey kevinheavey force-pushed the extract-feature-set branch from e154f71 to e4ca26e Compare August 9, 2024 11:02
Copy link

mergify bot commented Aug 9, 2024

The Firedancer team maintains a line-for-line reimplementation of the
native programs, and until native programs are moved to BPF, those
implementations must exactly match their Agave counterparts.
If this PR represents a change to a native program implementation (not
tests), please include a reviewer from the Firedancer team. And please
keep refactors to a minimum.

@kevinheavey kevinheavey force-pushed the extract-feature-set branch 2 times, most recently from e02b542 to 075967f Compare September 8, 2024 14:12
Copy link

@joncinque joncinque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just nits, looks great!

local-cluster/Cargo.toml Outdated Show resolved Hide resolved
programs/system/Cargo.toml Show resolved Hide resolved
clock::Slot,
epoch_schedule::EpochSchedule,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Once EpochSchedule gets moved out of solana_program, this crate could depend only on pubkey, hash, clock, and that 🥳

programs/bpf_loader/src/syscalls/mod.rs Outdated Show resolved Hide resolved
Copy link

@joncinque joncinque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one also looks good, thanks! @yihau can you accept the crate ownership for solana-feature-set? Once checks pass, we can get this in

@yihau
Copy link
Member

yihau commented Sep 10, 2024

all green!

Copy link

@joncinque joncinque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good to go!

@joncinque joncinque merged commit 8116c10 into anza-xyz:master Sep 10, 2024
52 checks passed
@kevinheavey kevinheavey added the automerge automerge Merge this Pull Request automatically once CI passes label Sep 10, 2024
ray-kast pushed a commit to abklabs/agave that referenced this pull request Nov 27, 2024
* extract feature-set crate

* update feature_set dependents

* re-export with deprecation notice

* fix path after rebase

* fmt

* fmt after rebase

* fixes after rebase

* unused import

* fix import

* unused import

* post-rebase fixes

* fix path

* missing frozen-abi support

* update lock file

* use workspace lints

* fix paths

* fix import

* remove unused import

* remove unused deps

* consolidate imports

* add back dep (not unused after all)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge automerge Merge this Pull Request automatically once CI passes need:merge-assist
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants