Skip to content

Remove deprecated compilation options 2 #10391

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 23, 2025
Merged

Remove deprecated compilation options 2 #10391

merged 2 commits into from
Sep 23, 2025

Conversation

bjwtaylor
Copy link

@bjwtaylor bjwtaylor commented Sep 8, 2025 •
edited
Loading

Description

Restore config.py changes after dependency merge. resolves Mbed-TLS/TF-PSA-Crypto#375 depends Mbed-TLS/TF-PSA-Crypto#427

  1. Remove deprecated compilation options #10370
  2. Remove deprecated compilation options TF-PSA-Crypto#427
  3. Remove deprecated compilation options 2 #10391

PR checklist

@bjwtaylor bjwtaylor added the needs-preceding-pr Requires another PR to be merged first label Sep 8, 2025
This was referenced Sep 8, 2025
Merged
Merged
@bjwtaylor bjwtaylor force-pushed the remove-deprecated-compilation-options-2 branch from cd56705 to 88e19da Compare September 9, 2025 06:59
@bjwtaylor bjwtaylor added needs-review Every commit must be reviewed by at least two team members, needs-reviewer This PR needs someone to pick it up for review priority-medium Medium priority - this can be reviewed as time permits labels Sep 9, 2025
@bjwtaylor bjwtaylor marked this pull request as ready for review September 9, 2025 09:04
@valeriosetti valeriosetti self-requested a review September 9, 2025 12:21
@bjwtaylor bjwtaylor force-pushed the remove-deprecated-compilation-options-2 branch from 84ef81f to fec1c00 Compare September 22, 2025 08:13
@bjwtaylor bjwtaylor removed the needs-preceding-pr Requires another PR to be merged first label Sep 23, 2025
@gilles-peskine-arm gilles-peskine-arm removed the needs-reviewer This PR needs someone to pick it up for review label Sep 23, 2025
gilles-peskine-arm
gilles-peskine-arm approved these changes Sep 23, 2025
View reviewed changes
Copy link
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

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

LGTM

@github-project-automation github-project-automation bot moved this from In Development to Has Approval in Non-roadmap pull requests Sep 23, 2025
@gilles-peskine-arm gilles-peskine-arm added approved Design and code approved - may be waiting for CI or backports and removed needs-review Every commit must be reviewed by at least two team members, labels Sep 23, 2025
@gilles-peskine-arm gilles-peskine-arm added this pull request to the merge queue Sep 23, 2025
Merged via the queue into Mbed-TLS:development with commit caaf52d Sep 23, 2025
8 checks passed
@github-project-automation github-project-automation bot moved this from Has Approval to Done in Non-roadmap pull requests Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gilles-peskine-arm gilles-peskine-arm gilles-peskine-arm approved these changes

@valeriosetti valeriosetti valeriosetti approved these changes

Assignees
No one assigned
Labels
approved Design and code approved - may be waiting for CI or backports priority-medium Medium priority - this can be reviewed as time permits
Projects
Non-roadmap pull requests
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove deprecated compilation options before 1.0
3 participants
@bjwtaylor @gilles-peskine-arm @valeriosetti