-
Notifications
You must be signed in to change notification settings - Fork 101
RFC: Merge Cortex teams into new Arm team #818
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
Conversation
1. Create new Arm team | ||
2. Add all Cortex-A, Cortex-R, and Cortex-M members to Arm team | ||
3. Add all Cortex-A, Cortex-R, and Cortex-M repositories to Arm team | ||
4. Delete Cortex-A, Cortex-R, and Cortex-M teams |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These teams also own crates on crates.io. Like https://crates.io/crates/cortex-m.
ye |
The overlap between M and R is quite big (both bare metal, both have no MMU, both use similar assembly) and the overlap between R and A is quite big (same vector tables and exception handling mechanism), so I'm all for removing this artificial distinction. There’s also a whole functioning cortex-r-rt crate I’m trying to donate but can’t because there’s no Cortex-R team members currently. You don't need to be a Cortex-R expert to review it - a Cortex-A person can review the vector tables and a Cortex-M person can review the inline assembly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it makes sense.
This comment was marked as outdated.
This comment was marked as outdated.
@adamgreig what does this need to pass? simple majority? |
I was told on Zulip that a 👍 on the PR doesn't count as 'approval'. @nchong-at-aws and @berkus - can you tick your boxes in the PR description if you approve? |
I can't - the description is not editable. |
Could you try now please? |
@nchong-at-aws and @berkus - please can you tick your boxes in the PR description if you approve? |
I did. |
…, r=Noratrieb Update target maintainers for thumb targets to reflect new REWG Arm team name Closes rust-lang#139027 The name of the team responsible for these targets has changed as the team was merged with other Arm-related teams (see rust-embedded/wg#818). The link gives an up-to-date list of github usernames that can be pinged, whereas the old email address is not very actively maintained or tracked.
Rollup merge of rust-lang#139038 - adamgreig:thumb-target-maintainers, r=Noratrieb Update target maintainers for thumb targets to reflect new REWG Arm team name Closes rust-lang#139027 The name of the team responsible for these targets has changed as the team was merged with other Arm-related teams (see rust-embedded/wg#818). The link gives an up-to-date list of github usernames that can be pinged, whereas the old email address is not very actively maintained or tracked.
As discussed in last week's meeting, I'm proposing we merge the existing Cortex teams into a new Arm team that takes over maintenance of all the previous Cortex repositories.
Please put in any comments or thoughts you have, especially if you're in one of those teams!
We can discuss further in the next weekly meeting if needed.
Relevant members voting: