Skip to content

Update toolchain availability reference #42

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

Merged
merged 9 commits into from
Jun 2, 2025

Conversation

dviererbe
Copy link
Collaborator

Updates the toolchain availability reference

  • separates the toolchain availability sections into separate articles by toolchain
  • adds LLVM/Clang version info for questing
  • updates LLVM default version for plucky
  • adds .NET 10 preview info
  • adds .NET backports/previews PPA reference
  • minor styling changes

dviererbe added 4 commits May 23, 2025 12:51
The shared links document is unused, because it requires
reStructuredText to use. We use myST which does not have a
similar feature.

The document was just part of the Canonical Sphinx docs starter pack.

Signed-off-by: Dominik Viererbe <[email protected]>
Seperate the reference of available versions for the toolchains into
seperate documents by toolchain.

This commit also applies minor updates to the availability matracies.

Signed-off-by: Dominik Viererbe <[email protected]>
Add a smalll reference outlining the .NET backports and previews PPA.

Signed-off-by: Dominik Viererbe <[email protected]>
Signed-off-by: Dominik Viererbe <[email protected]>
@dviererbe dviererbe requested a review from rkratky May 23, 2025 10:17
@rkratky
Copy link
Collaborator

rkratky commented May 23, 2025

Thanks, @dviererbe. Maybe we could use Matt's suggestion to have the toolchain availability under tabs like the IDE reference? I'm guessing a separate page for every toolchain might not be necessary.

@dviererbe
Copy link
Collaborator Author

@rkratky I started out with Tabs, but ref linking is a bit weird with tabs.

Copy link
Collaborator

@rkratky rkratky left a comment

Choose a reason for hiding this comment

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

Thank you, @dviererbe (and apologies for the delay). Added some comments.

@rkratky
Copy link
Collaborator

rkratky commented Jun 2, 2025

@rkratky I started out with Tabs, but ref linking is a bit weird with tabs.

It should work with defining a sync key: https://sphinx-design.readthedocs.io/en/latest/tabs.html?category=key2#synchronised-tabs

::::{tab-set}
:sync-group: category

:::{tab-item} Label1
:sync: key1

Content 1
:::

:::{tab-item} Label2
:sync: key2

Content 2
:::

::::
  • ?category=key1
  • ?category=key2

@dviererbe
Copy link
Collaborator Author

dviererbe commented Jun 2, 2025

@rkratky I started out with Tabs, but ref linking is a bit weird with tabs.

It should work with defining a sync key: https://sphinx-design.readthedocs.io/en/latest/tabs.html?category=key2#synchronised-tabs
[...]

issue: Yes, it is possible to manually create a link like that to link to a heading within a tab item , but this does not work properly with dynamic links using the {ref}`link` syntax.

Let's take this small example:

::::{tab-set}
:sync-group: category

:::{tab-item} .NET
:sync: dotnet

(dotnet-sub-heading)=
## dotnet sub heading

.NET content

:::

:::{tab-item} LLVM
:sync: llvm

(llvm-sub-heading)=
## llvm sub heading

LLVM content

:::

::::

If I now want to link to the llvm sub heading (on the 2nd tab) using this syntax {ref}`llvm-sub-heading` I will just see the .NET content¹ when clicking on the link, because the dynamic link does not add ?category=llvm.

¹ actually it will show you the tab that you last visited, which is the first tab (.NET in this example). This behaviour in combination with dynamic linking is even more confusing.

@rkratky
Copy link
Collaborator

rkratky commented Jun 2, 2025

this does not work with dynamic links using the {ref}link syntax does not work properly

Ah, right. You obviously thought this through much more than I have ;) OK, point taken -- it'll be better to have indiv. pages/files.

dviererbe and others added 5 commits June 2, 2025 17:09
Remove the row with the links to the toolchain availability pages from
the index page of the docs.

This is based on the feedback of our technical author Robert Krátký:

> I think extending the table this way is not necessary -- let's keep
> the landing page uncluttered. While the reference info is useful, it's
> not the main content of the docs set. I'd like the table to link
> (only) to the two basic topics:
> - how to get the toolchains
> - how to start using them

Co-authored-by: Robert Krátký <[email protected]>
Signed-off-by: Dominik Viererbe <[email protected]>
Move the reference for valid components and versions to install to the
.NET availability reference and link to it from the how-to/dotnet-setup
article.

Signed-off-by: Dominik Viererbe <[email protected]>
Replace the <span> elements to use a custom css class by defining a
spinx role and using this instead.

This makes the source text much more readable and maintainable.

Signed-off-by: Dominik Viererbe <[email protected]>
Instead of using a custom default-package role to highlight the
llvm-default version in orange, we replace it by using a sperate row
that shows the default version.

This was suggested by Robert Krátký.

Co-authored-by: Robert Krátký <[email protected]>
Signed-off-by: Dominik Viererbe <[email protected]>
A previous commit accidentially removed a `|` from index.md at the
end of a table column.

Signed-off-by: Dominik Viererbe <[email protected]>
@rkratky rkratky merged commit 24ea2c2 into main Jun 2, 2025
3 checks passed
@rkratky rkratky deleted the feat/update-toolchain-availability-refernce branch June 2, 2025 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants