Skip to content

Conversation

@sallam-dev
Copy link
Contributor

A minor go build change to trim the paths and remove some debug symbols without affecting panic(). This results in 7MB reduction in binary size

image

Copy link
Collaborator

@brandur brandur left a comment

Choose a reason for hiding this comment

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

Nice! Yeah. we use a similar technique at work and as far as I can tell there's no downsides whatsoever in practice (theoretically you might be trying to debug a binary, but I've never needed to do that).

Could you also sign our CLA? It just involves adding your name to a CSV:

https://github.com/riverqueue/rivercla

@sallam-dev sallam-dev force-pushed the optimize-docker-size branch from 8f5ad04 to dd86013 Compare August 9, 2025 17:45
Copy link
Collaborator

@brandur brandur left a comment

Choose a reason for hiding this comment

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

Thanks!

@brandur brandur merged commit f9e0280 into riverqueue:master Aug 9, 2025
10 of 13 checks passed
brandur added a commit that referenced this pull request Aug 9, 2025
brandur added a commit that referenced this pull request Aug 9, 2025
@sallam-dev sallam-dev deleted the optimize-docker-size branch August 9, 2025 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants