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

Platform Support: Libs teams relationship with target maintainers #39

Open
4 tasks
yaahc opened this issue Feb 24, 2022 · 0 comments
Open
4 tasks

Platform Support: Libs teams relationship with target maintainers #39

yaahc opened this issue Feb 24, 2022 · 0 comments
Labels
meta Discussion about Libs itself T-libs Relevant to the libraries subteam, which will review and decide on the PR/issue.

Comments

@yaahc
Copy link
Member

yaahc commented Feb 24, 2022

Building upon the Target tier policy the libs team would like to more clearly define our relationship with target maintainers and ensure that we are providing support as necessary to them and the work they do within the rust project to maintain support for rust on their respective platforms.

This was brought up in the context of this PR modifying system library linkage on netbsd. In the process of reviewing that PR I realized that we don't have a designated maintainer for netbsd or for many other platforms which got grandfathered in to the new system.

After a discussion with the rest of the libs team we've resolved to work to improve this situation, and we've agreed on the following steps.

  • Open issues for existing platforms that do not have designated maintainers to identify and document existing maintainers, recruit new maintainers, or, if all of that fails, mark the platforms as as unmaintained
  • Create associated ping groups for "target experts" to notify platform maintainers and contributors of changes to their platform so they can participate in those discussions and provide reviews

This came up again in another issue where a new feature was introduced but existing maintainers were not notified of it or given a chance to implement it prior to stabilization. We will be discussing this issue further in the next libs team meeting but I will be recommending that we additionally institute the following changes to our process:

  • Setup general ping groups for each target tier
  • Add "Ping tier2 and tier3 maintainers" as a required step to new tracking issues and again during stabilization for issues introducing features that require platform specific implementations
@dtolnay dtolnay added T-libs Relevant to the libraries subteam, which will review and decide on the PR/issue. meta Discussion about Libs itself labels Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta Discussion about Libs itself T-libs Relevant to the libraries subteam, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

2 participants