Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EDMX Metadata Generator options for .NET Core #828

Closed
robertmclaws opened this issue May 9, 2019 · 4 comments · Fixed by #892
Closed

EDMX Metadata Generator options for .NET Core #828

robertmclaws opened this issue May 9, 2019 · 4 comments · Fixed by #892

Comments

@robertmclaws
Copy link

Per the announcement post, EF 6.3 Preview 5 is currently shipping without EDMX support. Fortunately, I solved that problem about a year ago for SDK-style projects targeting the .NET Framework. I've open-sourced that tool here: https://github.com/CloudNimble/EF6.3MetadataGenerator.

Elements of this tool could be turned into a .NET Core Global Tool using dotnet-script and csx files (or possibly another option), but the code would have to be changed to manipulate the Project XML manually.

According to this tweet, the team is considering a couple different options. I thought this thread would be a good place to discuss those options, and consider if the tool I linked to would be useful in the final product.

@bricelam
Copy link
Contributor

My initial thought was just to port and package Microsoft.Data.Entity.targets and Microsoft.Data.Entity.Build.Tasks.dll as part of EntityFramework. It feels like a good candidate for NuGet's buildTransitive directory.

@bricelam
Copy link
Contributor

bricelam commented Jun 3, 2019

I've got this working on an internal branch using the approach I described. We're just waiting for approval to open source Microsoft.Data.Entity.Build.Tasks.dll...

bricelam added a commit to bricelam/ef6 that referenced this issue Jun 4, 2019
@ajcvickers ajcvickers modified the milestones: 6.3.0, 6.3.0-preview7 Jul 2, 2019
@GuyBenhaim
Copy link

Hello,
Was there a conclusion on this? in 6.3 Preview 7?
Thx

@ajcvickers
Copy link
Member

@GuyBenhaim The fix was included in preview 7.

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

Successfully merging a pull request may close this issue.

4 participants