Skip to content
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

Disable UPX Compression #2212

Closed
wants to merge 5 commits into from
Closed

Disable UPX Compression #2212

wants to merge 5 commits into from

Conversation

benr
Copy link
Member

@benr benr commented Oct 14, 2023

No description provided.

Signed-off-by: Ben Rockwood <[email protected]>
@benr benr marked this pull request as draft October 14, 2023 01:42
benr added 4 commits October 13, 2023 18:48
@chris-rock
Copy link
Member

@jaym is that PR still required?

@jaym
Copy link
Contributor

jaym commented Oct 24, 2023

if its causing problems with antivirus, i don't see a problem with disabling it.

@chris-rock
Copy link
Member

we did the upx removal in #2214

@benr benr self-assigned this Oct 24, 2023
@benr
Copy link
Member Author

benr commented Oct 24, 2023

Closing this as duplicate of #2214, this PR also included a skip publish capability for testing but that may cause additional problems in the future so I'll drop it for now.

@benr benr closed this Oct 24, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Oct 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants