Skip to content

Remove redundant .resx files when updated dotnet ships. #3905

Closed

Description

@brettfo

PR #3904 required that some .resx files be added to the src/buildfromsource/ directory. When a new dotnet sdk is shipped that has changes #3577 and #3607, then the .resx files (and their accompanying .fs files) should be removed.

Activity

  1. self-assigned this
    on Nov 7, 2017
  2. dsyme commented on Dec 1, 2017

    @dsyme
    Contributor

    +100

  3. cartermp commented on Feb 14, 2018

    @cartermp
    Contributor

    @brettfo Does this still need to be open?

  4. brettfo commented on Feb 15, 2018

    @brettfo
    MemberAuthor

    There are still some we need to keep around for the proto build and buildfromsource, so it should stay open a bit longer.

  5. TD5 commented on Oct 10, 2018

    @TD5
    Contributor

    Does resolving this issue remove the need to manually copy any files at all across to src\buildfromsource\FSharp.Compiler.Private\ (as per DEVGUIDE.md)?

  6. brettfo commented on Oct 10, 2018

    @brettfo
    MemberAuthor

    Yes, once this is resolved that manual copy step will go away. It should be handled by my changes in #5842.

  7. cartermp commented on Jan 30, 2019

    @cartermp
    Contributor

    @brettfo Now that we have an updated SDK we build against, can these files be removed?

  8. added this to the Unknown milestone on Jan 30, 2019
  9. cartermp commented on May 1, 2019

    @cartermp
    Contributor

    Closing this as we no longer have this directory in our source build.

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

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions