Skip to content

platform-support/netbsd.md: No longer mention 9.x, due to EoL. - #162268

Open
fraggerfox wants to merge 1 commit into
rust-lang:mainfrom
fraggerfox:update-netbsd-doc
Open

platform-support/netbsd.md: No longer mention 9.x, due to EoL.#162268
fraggerfox wants to merge 1 commit into
rust-lang:mainfrom
fraggerfox:update-netbsd-doc

Conversation

@fraggerfox

@fraggerfox fraggerfox commented Sep 4, 2026

Copy link
Copy Markdown

Change the pkgsrc-wip link to indicate a more current rust version.

Follow up to #160625

  • I did not use an LLM to create a change in this PR.
  • I used an LLM to create a change in this PR, and I have explained below how it was used.

cc: @he32 / @0323pin

@rustbot

rustbot commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Some changes occurred in src/doc/rustc/src/platform-support

cc @Noratrieb

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 4, 2026
@rustbot

rustbot commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the pull request, and welcome! The Rust Project has assigned @folkertdev (or someone else) to review your changes, you should hear from them (or someone else) within the next two weeks.

Please see the contribution instructions and our LLM policy for more information.

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 75 candidates
  • Random selection from 21 candidates

Comment on lines 30 to +31
with newer versions). `x86_64-unknown-netbsd` is built for NetBSD
10.x to access a newer gcc.
11.x to access a newer gcc.

@tgross35 tgross35 Sep 4, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there a PR in the works for this? Currently it looks like 10 is still used

SOURCE_URL=https://ci-mirrors.rust-lang.org/rustc/netbsd/2026-07-28-netbsd-10.1-src

View changes since the review

@0323pin 0323pin Sep 4, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@tgross35 why do you want to change this? Both 10 and 11 are supported and both have fixed gcc versions.

Edit: if this is important ... what do you say @he32 ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have no preference whether or not it's changed, just want our docs to be consistent

@fraggerfox fraggerfox Sep 4, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This I will need to verify with @0323pin

Looks like the maintainer has replied, so striking my comment out.

Comment on lines 26 to +28
All use the "native" `stdc++` library which goes along with the natively
supplied GNU C++ compiler for the given OS version. Most of the bootstraps
are built for NetBSD 9.x, although some exceptions exist (some are
are built for NetBSD 10.x, although some exceptions exist (some are

@tgross35 tgross35 Sep 4, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you make the supported version explicit with an "OS version" section, like we have here? https://doc.rust-lang.org/nightly/rustc/platform-support/apple-darwin.html#os-version

View changes since the review

@0323pin 0323pin Sep 4, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@tgross35 the whole 10.x series is binary compatible, i.e. it doesn't really matter if you build on 10.0 and are using 10.1 or the reverse.

Edit: thoughts @he32 ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm not sure what exactly that is in response to but to clarify, I'm just looking for an explicit "The minimum supported version is ..." sentence rather than inferring what "most of the bootstraps are built for" means.

And/or adding a (x.y+) note to the table on https://doc.rust-lang.org/nightly/rustc/platform-support.html like a lot of platforms have

@fraggerfox fraggerfox Sep 4, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Alright, let me go through the tooling and get the exact numbers out.

Looks like both the maintainers have replied, so striking my comment out.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure what exactly that is in response to but to clarify, I'm just looking for an explicit "The minimum supported version is ..." sentence rather than inferring what "most of the bootstraps are built for" means.

That's understandable... However, that is also a moving target, and depends on the rust version, well, really depends on the minimum supported LLVM version and/or the integrated LLVM version. So far (as of this writing, 1.98.1 is released), the new LLVM version (23.x?) which requires an "insanely fresh gcc to build correctly" and which pushes the minimum OS version target upwards has so far not materialized in a full rust release. It's due for the next one? So I hesitate to bump the minimum requirement for the base OS while we still build releases for 9.x (which results in binaries which can be run on both 10.x and 11.x).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The version we use for building can also be decoupled from the version we support. We should be able to bump build dependencies if something blocks it, but produce a rustc and other compiled binaries that work on older targets (like we do for many other platforms).

To be clear: I personally have no problem with whatever minimum version is picked and am not pushing for a bump, 8 or 9 or 10 or 11 makes no difference to me. We just need a clear answer for what we're trying to support so we can make decisions like what to do with rust-lang/libc#5243.

@he32 he32 Sep 5, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The version we use for building can also be decoupled from the version we support

Well, it's a bit more complicated than that.

In general, a binary built on 9.x can be run on 10.x and 11.x.
A binary built on 10.x on the other hand, all bets are off for being able to run it on 9.x.
Similarly, a binary built on 11.x there is no guarantee it will be possible to run on either of 10.x or 9.x.
There's some additional rules for what goes between 10.0 and subsequent 10.x versions such as 10.1, but that gets into the weeds and is probably not so relevant here.

So stating what version we build binaries for actually already specifies which versions the resultant binaries are usable on. That's also part of the reason that I don't want to prematurely bump the minimum supported OS version, and also why I stick to the oldest supported version in my target trees when doing cross compilation.

And, lastly, since rust requires a binary for the "previous" rust version to bootstrap, bumping the OS version we build for will cut off all lower-major-numbered OS releases from having any chance of getting rust installed, at least for a "native" build (I realized a cross-build could still be done, as long as the target's compiler is new enough, ref. the impending increase in requirement on that front for newer LLVM).

@folkertdev

Copy link
Copy Markdown
Contributor

r? tgross35

@rustbot rustbot assigned tgross35 and unassigned folkertdev Sep 5, 2026
@rustbot

rustbot commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

tgross35 is currently at their maximum review capacity.
They may take a while to respond.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants