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

Arcade --> Korebuild "gap" #135

Closed
markwilkie opened this issue Apr 25, 2018 · 12 comments
Closed

Arcade --> Korebuild "gap" #135

markwilkie opened this issue Apr 25, 2018 · 12 comments

Comments

@markwilkie
Copy link
Member

The initial pass at the gap is written down by Nate here: https://gist.github.com/natemcmaster/0ef23512c011b7ec09f25ad13a1d65e2

This will likely be translated to specific issues once we start.

@mmitche
Copy link
Member

mmitche commented Nov 7, 2019

@dougbu We're off Korebuild for all the aspnet repos (3.x onward), right?

@dougbu
Copy link
Member

dougbu commented Nov 8, 2019

I believe we have at least a couple of things in the Internal.AspNetCore.BuildTasks.nupkg package that we still need. Then again, that's currently listed as a workaround for #204, not this issue.

See also dotnet/msbuild#4018 (Nate's ask about contribution to MSBuild). Should we instead contribute the GenerateFileFromTemplate task to Arcade?

@ilyas1974
Copy link
Contributor

Triage: Based on the conversation within this thread, it appears that we no longer use Korebuild. Does anyone know what the status of this work is?

@dougbu
Copy link
Member

dougbu commented Mar 26, 2021

We still use KoreBuild aka Internal.AspNetCore.Sdk in at least dotnet/aspnetcore, dotnet/efcore and dotnet/extensions for 2.1 builds. That's fine.


However, we also use GenerateFileFromTemplate in later branches due to restrictions in WriteLinesToFile and ease of use. Would make sense to remove that dependency (the entire Internal.AspNetCore.BuildTasks package) if possible.

Since dotnet/msbuild#4018 has no traction, I ask again

Should we instead contribute the GenerateFileFromTemplate task to Arcade?

The source for that is small and straightforward. See https://github.com/aspnet/BuildTools/tree/main/src/Internal.AspNetCore.BuildTasks

Ideal would be to add this in the Arcade servicing branches too. Failing that, we should probably add this task to dotnet/aspnetcore's RepoTasks hack (most of which isn't worth contributing).

See also #204 (which @markwilkie and @ViktorHofer seemed to like) and dotnet/msbuild#4018.

@markwilkie
Copy link
Member Author

Thanks @dougbu - sounds like we're essentially sorted here.

@dougbu
Copy link
Member

dougbu commented Mar 31, 2021

Agree we don't need to discuss the broad "gap" further. But, do we have something tracking GenerateFileFromTemplate or an upcoming discussion around it❔

@markwilkie
Copy link
Member Author

But, do we have something tracking GenerateFileFromTemplate or an upcoming discussion around it❔

Not that I'm aware of, but to be honest, I don't think I'm even aware what GenerateFileFromTemplate is... :(

@dougbu
Copy link
Member

dougbu commented Mar 31, 2021

@markwilkie GenerateFileFromTemplate is a very easy-to-use way to generate files from templates without the downsides of WriteLinesToFile (which is inappropriate for large files) or T4 (which is often overkill). GenerateFileFromTemplate is also the remaining item I wanted to consider in the context of gaps between KoreBuild and Arcade.

There's a bit more information in the links from #135 (comment) and #135 (comment)

@ViktorHofer
Copy link
Member

We use a similar mechanism in dotnet/runtime to generate our test execution scripts: https://github.com/dotnet/runtime/blob/main/src/tasks/installer.tasks/GenerateRunScript.cs.

It would be great if either msbuild or Arcade would offer such a task.

@markwilkie
Copy link
Member Author

I see - makes sense. Could one of you create another issue to track the generate file stuff?

@ViktorHofer
Copy link
Member

I'll let @dougbu do that as it sounds like they still have a Korebuild dependency in their build system because of that. For us in dotnet/runtime it's mostly clean-up.

@dougbu
Copy link
Member

dougbu commented Apr 8, 2021

I just added to existing issue #204

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

No branches or pull requests

5 participants