Skip to content

ci: fix PowerShell .tar validation#1505

Merged
Richard Markiewicz (thenextman) merged 1 commit intomasterfrom
fix-tar-validation-in-package-yaml
Sep 12, 2025
Merged

ci: fix PowerShell .tar validation#1505
Richard Markiewicz (thenextman) merged 1 commit intomasterfrom
fix-tar-validation-in-package-yaml

Conversation

@thenextman
Copy link
Copy Markdown
Member

We saw that a .tar archive was corrupted with the warning "There are data after the archive [sic]". This seems like something transient, but validation didn't catch it.

tar -t {path} will not work as expected (at least on Windows); it's needed to pass f to specify the file path. Otherwise tar falls back on \\.\tape0)(!).

Regardless, when validating tar does not set a non-zero error code for this kind of issue.

Instead, use 7z -t which properly sets the error code and allows us to bail out.

@github-actions
Copy link
Copy Markdown

Let maintainers know that an action is required on their side

  • Add the label release-required Please cut a new release (Devolutions Gateway, Devolutions Agent, Jetsocat, PowerShell module) when you request a maintainer to cut a new release (Devolutions Gateway, Devolutions Agent, Jetsocat, PowerShell module)

  • Add the label release-blocker Follow-up is required before cutting a new release if a follow-up is required before cutting a new release

  • Add the label publish-required Please publish libraries (`Devolutions.Gateway.Utils`, OpenAPI clients, etc) when you request a maintainer to publish libraries (Devolutions.Gateway.Utils, OpenAPI clients, etc.)

  • Add the label publish-blocker Follow-up is required before publishing libraries if a follow-up is required before publishing libraries

@thenextman Richard Markiewicz (thenextman) merged commit fc76b9b into master Sep 12, 2025
41 checks passed
@thenextman Richard Markiewicz (thenextman) deleted the fix-tar-validation-in-package-yaml branch September 12, 2025 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants