Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Removes native loader entrypoints from the SDK#21185

Merged
Lichtso merged 1 commit intosolana-labs:masterfrom
Lichtso:remove/native_loader_entrypoint_from_sdk
Nov 5, 2021
Merged

Removes native loader entrypoints from the SDK#21185
Lichtso merged 1 commit intosolana-labs:masterfrom
Lichtso:remove/native_loader_entrypoint_from_sdk

Conversation

@Lichtso
Copy link
Copy Markdown
Contributor

@Lichtso Lichtso commented Nov 5, 2021

Problem

These two entrypoints are the last remaining connections that attach the SDK to the program runtime crate.
As the native loader is already prepared to be feature gated out, we can remove these from the public interface.
This PR is a preparation so that #21180 can fully separate the SDK and the program runtime crate.

Summary of Changes

The two entrypoints were moved from the SDK to the program-runtime crate.
Meaning they can still be used for loading existing but not for the deployment of new programs anymore.

Fixes #

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 5, 2021

Codecov Report

Merging #21185 (950404b) into master (a32413f) will increase coverage by 0.0%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #21185   +/-   ##
=======================================
  Coverage    81.5%    81.6%           
=======================================
  Files         499      499           
  Lines      140244   140244           
=======================================
+ Hits       114392   114448   +56     
+ Misses      25852    25796   -56     

@Lichtso Lichtso merged commit 6704fa0 into solana-labs:master Nov 5, 2021
@Lichtso Lichtso deleted the remove/native_loader_entrypoint_from_sdk branch November 5, 2021 11:49
dankelleher pushed a commit to identity-com/solana that referenced this pull request Nov 24, 2021
frits-metalogix added a commit to identity-com/solana that referenced this pull request Nov 24, 2021
@Lichtso Lichtso mentioned this pull request Jan 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant