Skip to content

Remove rent exempt constraint#3870

Closed
jamie-osec wants to merge 5 commits intosolana-foundation:masterfrom
jamie-osec:remove-rent-exempt-constraint
Closed

Remove rent exempt constraint#3870
jamie-osec wants to merge 5 commits intosolana-foundation:masterfrom
jamie-osec:remove-rent-exempt-constraint

Conversation

@jamie-osec
Copy link
Copy Markdown
Collaborator

Closes #2347

Rebased version of #2708

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 21, 2025

@acheroncrypto is attempting to deploy a commit to the Solana Foundation Team on Vercel.

A member of the Team first needs to authorize it.

@jamie-osec jamie-osec force-pushed the remove-rent-exempt-constraint branch from 0a0408d to 9d5694a Compare August 21, 2025 11:14
@jacobcreech jacobcreech requested a review from Aursen August 21, 2025 21:54
@jacobcreech
Copy link
Copy Markdown
Collaborator

Could you rebase and rerun bench @swaroop-osec ?

@jamie-osec jamie-osec force-pushed the remove-rent-exempt-constraint branch 2 times, most recently from db3a9b5 to f52eada Compare August 22, 2025 10:35
@jamie-osec
Copy link
Copy Markdown
Collaborator Author

Pulled in @swaroop-osec's bench changes

@jamie-osec jamie-osec force-pushed the remove-rent-exempt-constraint branch from f52eada to 1651e4d Compare August 22, 2025 10:58
@BretasArthur1
Copy link
Copy Markdown
Contributor

BretasArthur1 commented Aug 30, 2025

Question, any existing code using #[account(rent_exempt = skip)] or #[account(rent_exempt = enforce)] will fail to compile? Since you removed those features... @jamie-osec

@jamie-osec
Copy link
Copy Markdown
Collaborator Author

As-is this will be a breaking change, yes. While this is for Anchor 1.0 iiuc (and therefore breakage is okay), we could mitigate this by parsing the attributes and only emitting a deprecation warning.

@BretasArthur1
Copy link
Copy Markdown
Contributor

Sg!

@0x4ka5h
Copy link
Copy Markdown
Collaborator

0x4ka5h commented Nov 18, 2025

@jamie-osec can you look at the failed tests once

@jamie-osec
Copy link
Copy Markdown
Collaborator Author

Spurious failure due to ratelimits. This will need rebasing for the benchmark changes though

@jamie-osec
Copy link
Copy Markdown
Collaborator Author

Given that this is breaking, I think we can just not port this when doing V2.

@jamie-osec jamie-osec closed this Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove rent_exempt constraint

6 participants