Skip to content

Remove Emscripten from the docs #115977

Description

@brettcannon

Since Emscripten is no longer a tier 3 platform we should probably remove mentions of it from the docs.

Activity

  1. tavallaie commented on Mar 2, 2024

    @tavallaie
    Contributor

    how we can help this issue? I need to know more to solve it.

  2. brettcannon commented on Mar 4, 2024

    @brettcannon
    MemberAuthor

    Basically the docs say things like "Emscripten" or "not Emscripten" about platform support. Pretty much all mentions of "Emscripten" in the documentation should probably be removed.

  3. tavallaie commented on Mar 5, 2024

    @tavallaie
    Contributor

    Thansk, I will remove all of them, if there is any docs or related information about removing Emscripten like "why Emscripten is no longer a tier 3 platform "can mention in my PR please help me to find them.

  4. brettcannon commented on Mar 5, 2024

    @brettcannon
    MemberAuthor

    Thansk, I will remove all of them, if there is any docs or related information about removing Emscripten like "why Emscripten is no longer a tier 3 platform "can mention in my PR please help me to find them.

    You don't need an official link; the core team knows the "why".

  5. ned-deily commented on Mar 28, 2024

    @ned-deily
    Member

    It looks like there are still some Emscripten references in the docs beyond the ones removed by #117057.

  6. freakboy3742 commented on Mar 29, 2024

    @freakboy3742
    Contributor

    @ned-deily Which references in particular are you referring to?

    As noted in this comment, some of the Emscripten references were left in deliberately, reflecting the fact that the functionality exists, even if the platform isn't tier 3. This is analogous to the state of Android in previous releases - there are a handful of references to Android in the docs, even though Android isn't Tier 3 in those releases.

    I'm happy to do the cleanup on this if needed; I just need to be clear about what I'm cleaning up :-)

  7. nineteendo commented on Apr 18, 2024

    @nineteendo
    Contributor

    "Emscripten" is found in library/intro.rst, library/sys.rst & the release notes like mentioned in your comment. But also in using/configure.rst. If that's deliberate, then this issue should be closed, otherwise they should be removed from the latter.

  8. freakboy3742 commented on Apr 18, 2024

    @freakboy3742
    Contributor

    My read is that the mentions in using/configure.rst is in the same category as the ones in library - it's describing how to configure a build for Emscripten. That is still a valid configuration, just not a Tier 3 supported configuration.

  9. nineteendo commented on Apr 18, 2024

    @nineteendo
    Contributor

    OK, then I think you can close this.

  10. freakboy3742 commented on Apr 18, 2024

    @freakboy3742
    Contributor

    @ned-deily Did you have any specific concerns, or are you OK with closing this?

  11. ned-deily commented on Apr 19, 2024

    @ned-deily
    Member

    Sounds good, I think we can close this again.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    3.13only security fixesOS-wasidocsDocumentation in the Doc direasy

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions