Skip to content

Dockerfile.windows: remove deprecated 7Zip4Powershell #50936

Merged
thaJeztah merged 1 commit intomoby:masterfrom
vvoland:remove-7zip4ps
Sep 9, 2025
Merged

Dockerfile.windows: remove deprecated 7Zip4Powershell #50936
thaJeztah merged 1 commit intomoby:masterfrom
vvoland:remove-7zip4ps

Conversation

@vvoland
Copy link
Copy Markdown
Contributor

@vvoland vvoland commented Sep 9, 2025 •
edited
Loading

tar utility is included in Windows 10 (17063+) and Windows Server 2019+ so we can use it directly.

This fixes recent Windows CI failures: https://github.com/moby/moby/actions/runs/17578675113/job/49940905820?pr=50932

 WARNING: The specified PackageManagement provider 'NuGet' is not available.
Install-Package : No match was found for the specified search criteria and 
package name '7Zip4PowerShell'. Try Get-PackageSource to see all available 
registered package sources.
At line:1 char:3086
+ ... ownloading containerd; Install-Package -Force 7Zip4PowerShell; $locat ...
+                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Microsoft.Power....InstallPacka 
   ge:InstallPackage) [Install-Package], Exception
    + FullyQualifiedErrorId : NoMatchFoundForCriteria,Microsoft.PowerShell.Pac

@vvoland vvoland self-assigned this Sep 9, 2025
@vvoland vvoland changed the title Get rid of 7Zip4Powershell Dockerfile.windows: remove deprecated 7Zip4Powershell Sep 9, 2025
@vvoland vvoland added this to the 29.0.0 milestone Sep 9, 2025
@vvoland vvoland marked this pull request as ready for review September 9, 2025 13:09
thaJeztah
thaJeztah approved these changes Sep 9, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM always good to remove cruft

probably will have to be back ported to release-branches?

@vvoland
Dockerfile.windows: remove deprecated 7Zip4Powershell
8c8324b 
`tar` utility is included in Windows 10 (17063+) and Windows Server
2019+ so we can use it directly.

Signed-off-by: Paweł Gronowski <[email protected]>
@thaJeztah thaJeztah merged commit fcd536f into moby:master Sep 9, 2025
283 of 285 checks passed
This was referenced Sep 9, 2025
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@thaJeztah thaJeztah thaJeztah approved these changes

@austinvazquez austinvazquez austinvazquez approved these changes

@robmry robmry robmry approved these changes

Assignees

@vvoland vvoland

Labels

area/testing kind/bugfix PR's that fix bugs platform/windows process/cherry-picked

Projects

None yet

Milestone

29.0.0

Development

Successfully merging this pull request may close these issues.

5 participants

@vvoland @thaJeztah @austinvazquez @robmry @jho503