jberezanski / ChocolateyPackages

Chocolatey packages maintained by me
MIT License
83 stars 51 forks source link

fix #115 Change dotnetfx 4.7.2 to use correct installer url #116

Closed tempora-mutantur closed 2 years ago

tempora-mutantur commented 2 years ago

The package was updated to just depend on the netfx-4.7.2 which is considered to be supported. dotnetfx is used by end-users but the installer URL was changed, so we update the package to work.

jberezanski commented 2 years ago

That's not correct. The dotnetfx package is designed to install .NET by itself (and be upgradeable via choco upgrade), not to reference other packages. The right fix would be to update the installer url and checksum to the proper ones (and mention it in release notes).

But... why do you still install 4.7.2 when 4.8 has been available for two years now (and contains a lot of fixes)?

tempora-mutantur commented 2 years ago

Because our older version of software works specifically on 4.7.2 and we still have compatibility testing of it so we need this version of dotnet

tempora-mutantur commented 2 years ago

@jberezanski I've changed the package the way you suggested, thanks

jberezanski commented 2 years ago

Published. Thanks for the contribution and apologies for the delay.