Skip to content

WebAssembly should support handling runtimeconfig.json #38433

Closed

Description

@akoeplinger

Activity

  1. added this to the 5.0.0 milestone on Jun 26, 2020
  2. marek-safar commented on Jun 30, 2020

    @marek-safar
    Contributor

    Could you clarify why? We'd need to copy the files to the temp file system and add all the baggage for handling the json logic which is not necessary for self-contained setup.

    I think a better approach is to add the configProperties section which is probably the only relevant part for self-contained configs to csproj PropertyGroup and handle it during build.

  3. modified the milestones: 5.0.0, Future on Jun 30, 2020
  4. removed
    untriagedNew issue has not been triaged by the area owner
    on Jun 30, 2020
  5. akoeplinger commented on Jun 30, 2020

    @akoeplinger
    MemberAuthor

    Yeah I was imprecise when creating the title, I don't think we need to necessarily read the file as part of the WASM process but we need to handle it somehow rather than just ignoring it.

  6. changed the title [-]WebAssembly should support reading runtimeconfig.json[/-] [+]WebAssembly should support handling runtimeconfig.json[/+] on Jun 30, 2020
  7. radical commented on May 24, 2021

    @radical
    Member

    I think a better approach is to add the configProperties section which is probably the only relevant part for self-contained configs to csproj PropertyGroup and handle it during build.

    Do you mean - read the configProperties section of the runtimeConfig.json, and add it to the build, so that we don't have to support reading runtimeConfig.json itself in the wasm app?

  8. marek-safar commented on May 25, 2021

    @marek-safar
    Contributor

    We should just leverage the runtimeConfig.json task we have for mobile AOT for WASM AOT

  9. modified the milestones: Future, 6.0.0 on Jun 22, 2021
  10. lambdageek commented on Jun 22, 2021

    @lambdageek
    Member

    Yea, this is exactly what the runtimeConfig.json task is meant to do. The WasmApp target needs to be updated to use it, and then driver.c needs to be updated to call the runtime API that consumes the binary blob of the parsed json file.

  11. self-assigned this
    on Jul 9, 2021
  12. SamMonoRT commented on Jul 9, 2021

    @SamMonoRT
    Member

    cc @lewing @radical - Fan will give this a shot.

  13. eerhardt commented on Jul 28, 2021

    @eerhardt
    Member

    Is this going to be completed for .NET 6? Using AppContext switches is becoming more and more important in order to support feature switches. For example, the plan in #55102 is relying on the fact that an AppContext switch will be set in applications that will be trimmed during F5/dotnet run.

  14. SamMonoRT commented on Jul 28, 2021

    @SamMonoRT
    Member

    Is this going to be completed for .NET 6? Using AppContext switches is becoming more and more important in order to support feature switches. For example, the plan in #55102 is relying on the fact that an AppContext switch will be set in applications that will be trimmed during F5/dotnet run.

    Yes, a PR should be up soon

  15. ghost added
    in-prThere is an active PR which will close this issue when it is merged
    on Jul 28, 2021
  16. ghost removed
    in-prThere is an active PR which will close this issue when it is merged
    on Aug 6, 2021
  17. ghost locked as resolved and limited conversation to collaborators on Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions