efcore icon indicating copy to clipboard operation
efcore copied to clipboard

Revisit tools architecture

Open bricelam opened this issue 6 years ago • 8 comments

Some ideas:

  • Can dotnet-ef use the MSBuild object model?
    • See https://github.com/NuGet/NuGet.Client/blob/fe240c4f457b30050e963dc19f916773ddf21d38/src/NuGet.Clients/NuGet.CommandLine/MsBuildUtility.cs
    • Makes #18060 and https://github.com/dotnet/efcore/issues/23853 easier
    • Consider shipping EntityFrameworkCore.targets in buildTransitive instead of creating dynamically.
  • Can ef.exe use AssemblyLoadContext instead of dotnet exec?
    • Could make it a single-file executable
      • Easier to use directly on assemblies (issue #16882)
      • Easier to use without the SDK (on the server/in a container)
    • Building block to enabling Xamarin/UWP projects (#7152)
  • Use System.CommandLine

bricelam avatar Nov 11 '19 17:11 bricelam

Note from triage: consider using System.CommandLine; see #6850 and #30487.

ajcvickers avatar Mar 27 '23 10:03 ajcvickers

@ajcvickers System.Commandline has never been released though

ErikEJ avatar Mar 27 '23 11:03 ErikEJ

It would only be a private dependency that ships as part of our tool. We wouldn't be the first to do this. 🙂

bricelam avatar Mar 30 '23 18:03 bricelam

Related to https://github.com/dotnet/efcore/issues/14531

AndriySvyryd avatar Feb 14 '24 00:02 AndriySvyryd

@AndriySvyryd any status update\is getting actively worked on? getting tracked internally?

ran into this wonderful bug today, luckily our EF projects are setup weird enough for --msbuildprojectextensionspath ../../../artifacts/obj/MyEfDataProject/ to work.... still annoying

KLuuKer avatar Jun 20 '24 10:06 KLuuKer

@KLuuKer This issue is in the Backlog milestone. This means that it is not planned for the next release. We will re-assess the backlog following this release and consider this item at that time. However, keep in mind that there are many other high priority features with which it will be competing for resources, see Release planning process. Make sure to vote (👍) for this issue if it is important to you.

AndriySvyryd avatar Jun 20 '24 18:06 AndriySvyryd