microsoft / dotnet-framework-docker

The repo for the official docker images for .NET Framework on Windows Server Core.
https://hub.docker.com/_/microsoft-dotnet-framework
MIT License
698 stars 334 forks source link

Refactor patch variable usage in Dockerfile templates #989

Closed mthalman closed 2 years ago

mthalman commented 2 years ago

In an effort to simplify the Dockerfile templates and prepare for the addition of .NET Fx 4.8.1, these changes refactor the templates to avoid the need for if..else logic when referencing the KB patch info variables.

The naming of the variables was consistent and didn't directly correspond to available template variables, which caused the templates to be very specific to which variable they needed to reference.

These changes add more specificity to the variable names so that the templates can simply use the template PRODUCT_VERSION variable when needing to construct the variable name to reference.