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

Tracking issue for release notes of #138233: Windows: Don't link std (and run-make) against advapi32, except on win7 #138621

Open
rustbot opened this issue Mar 17, 2025 · 4 comments
Labels
O-windows Operating system: Windows relnotes Marks issues that should be documented in the release notes of the next release. relnotes-tracking-issue Marks issues tracking what text to put in release notes. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Milestone

Comments

@rustbot
Copy link
Collaborator

rustbot commented Mar 17, 2025

This issue tracks the release notes text for #138233.

cc @smmalis37, @ChrisDenton -- original issue/PR authors and assignees for drafting text

See the forge.rust-lang.org chapter about release notes for an overview of how the release team makes use of these tracking issues.

Release notes text

This section should be edited to specify the correct category(s) for the change, with succinct description(s) of what changed. Some things worth considering:

  • Does this need an additional compat notes section?
  • Was this a libs stabilization that should have additional headers to list new APIs under Stabilized APIs and Const Stabilized APIs?
# Compatibility Notes
- [Windows: The standard library no longer links advapi32, except on win7.](https://github.com/rust-lang/rust/pull/138233) Code such as C libraries that were relying on this assumption may need to explicitly link advapi32.

Tip

Use the previous releases for inspiration on how to write the release notes text and which categories to pick.

Release blog section

If this change is notable enough for inclusion in the blog post then this section should be edited to contain a draft for the blog post. Otherwise leave it empty.

Note

If a blog post section is required the release-blog-post label should be added (@rustbot label +release-blog-post) to this issue as otherwise it may be missed by the release team.

@rustbot rustbot added A-run-make Area: port run-make Makefiles to rmake.rs O-windows Operating system: Windows relnotes Marks issues that should be documented in the release notes of the next release. relnotes-tracking-issue Marks issues tracking what text to put in release notes. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Mar 17, 2025
@rustbot rustbot added this to the 1.87.0 milestone Mar 17, 2025
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Mar 17, 2025
@ChrisDenton
Copy link
Member

Does this require a relnote?

@smalis-msft
Copy link

I don't think so. The std library already wasn't depending on any functionality in advapi32, the PR just removed it from the list of available libs linked against.

@ehuss
Copy link
Contributor

ehuss commented Mar 17, 2025

I have marked this as relnotes because it caused a regression in git2. libgit2's C code was using some functions from advapi32, but the build script failed to include it explicitly (it was relying on the implicit inclusion from the standard library). I think that's the fault of git2's script, but it caused a regression nonetheless.

@smalis-msft
Copy link

Oh interesting. I wouldn't have expected that, but good to know, and I hope it was an easy fix. In that case then maybe this is worth mentioning yeah.

@jieyouxu jieyouxu added T-libs Relevant to the library team, which will review and decide on the PR/issue. and removed T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. A-run-make Area: port run-make Makefiles to rmake.rs labels Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-windows Operating system: Windows relnotes Marks issues that should be documented in the release notes of the next release. relnotes-tracking-issue Marks issues tracking what text to put in release notes. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

5 participants