-
Notifications
You must be signed in to change notification settings - Fork 132
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
SDK zips are missing NuGet.Tasks.Pack localization content #4953
Comments
Looks like NuGet.Client runs the MicroBuild localization targets to generate these files. The VMR currently doesn't run those (and can't outside of the official build), so we need to enable that in NuGet's VMR build (and only such in the official build). Figuring out how to do that now. |
We may need to set up parallel infra for localization for NuGet based on OneLocBuild or something to get xliff working with them so we don't need to install MicroBuild Localization in our pipelines. |
@jkoritzinsky Is it more expedient to use the microbuild tasks? |
I spent Friday iterating on a solution here. Here's our two options:
We can use the MicroBuild tasks as they are today. However, this also means that we need to onboard the Alternatively, we could try to switch NuGet.Build.Localization to flow the localization info as a package to NuGet.Client. If we're doing large infrastructure work in NuGet though, I'd prefer we go with the next option as we'd have to pull in people from NuGet as this would be a relatively large change.
We could onboard NuGet.Client to OneLocBuild with their own OneLocBuild job (not using the Arcade job as they still aren't onboarded onto Arcade) and .xlf files (or use MicroBuild localization with .xlf files if that tooling supports it). If we're able to get onto XLF files, then we can avoid onboarding custom localization tooling into the VMR itself as Arcade provides packages that can handle localization via XLF. |
It looks like MicroBuild's tasks have some issues with running on non-Windows platforms, so option 1 might not be as easy as I thought. |
The text was updated successfully, but these errors were encountered: