Skip to content

Tools: Rearrange package layout#11388

Merged
bricelam merged 1 commit intodotnet:release/2.1from
bricelam:shuffle
Mar 22, 2018
Merged

Tools: Rearrange package layout#11388
bricelam merged 1 commit intodotnet:release/2.1from
bricelam:shuffle

Conversation

@bricelam
Copy link
Copy Markdown
Contributor

We need to do this since dotnet-ef is bundled with the SDK.

@bricelam
Copy link
Copy Markdown
Contributor Author

@wli3 new layout of dotnet-ef package:

  • tools/
    • netcoreapp2.0/
      • any/
        • dotnet-ef.deps.json
        • dotnet-ef.dll
        • dotnet-ef.runtimeconfig.json
        • DotnetToolSettings.xml
        • tools/
          • net461/
            • any/
              • ef.exe
            • win-x86/
              • ef.exe
          • netcoreapp2.0/
            • any/
              • ef.dll

<None Include="..\ef\bin\$(Configuration)\net461\ef.exe">
<Pack>True</Pack>
<PackagePath>tools\net461</PackagePath>
<PackagePath>tools\$(TargetFramework)\any\tools\net461\any</PackagePath>
Copy link
Copy Markdown

@wli3 wli3 Mar 22, 2018

Choose a reason for hiding this comment

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

This should be good.

You can test it by install master build of CLI and unzip it under C:\Program Files\dotnet\sdk\2.1.300-preview2-008385\DotnetTools. Make sure the folder structure is similar to dotnet-watch which is also there.

Run dotnet ef, it should find it automatically.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Works great

@bricelam bricelam changed the base branch from dev to release/2.1 March 22, 2018 21:35
We need to do this since dotnet-ef is bundled with the SDK.
@bricelam bricelam merged commit 108d33e into dotnet:release/2.1 Mar 22, 2018
@bricelam bricelam deleted the shuffle branch March 22, 2018 21:44
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