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

Custom tags for wasm release builds #58

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

stefanb
Copy link
Contributor

@stefanb stefanb commented Dec 12, 2023

Feel free to adjust tags as needed. Just note that cross platform Go builds are not possible to do for features requiring C bindings (CGO). There are workarounds, but likely not needed: https://goreleaser.com/cookbooks/cgo-and-crosscompiling/

@@ -34,7 +34,14 @@ builds:
# goarch: arm

flags:
- -tags=b_tiny,b_norepl
- -tags=b_tiny
Copy link
Contributor Author

@stefanb stefanb Dec 12, 2023

Choose a reason for hiding this comment

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

eg. with the ultimate bloat working set of tags: 😄

Suggested change
- -tags=b_tiny
- -tags=b_sqlite,b_http,b_sql,b_postgres,b_bson,b_crypto,b_smtpd,b_mail,b_bcrypt,b_telegram,b_html,b_contrib,b_openai,b_email,b_mail,b_mysql,b_nats,b_psql,b_psutil,b_sxml,b_echo

Those tags are from buld CI workflow so they work on ubuntu/amd64 runners, but cross compilation would depend on their use of CGO.

@refaktor refaktor merged commit d1ca2cf into refaktor:main Dec 12, 2023
2 checks passed
@stefanb stefanb deleted the wasm-release-tags branch December 12, 2023 14:35
@refaktor
Copy link
Owner

Thank you again for PR and explanation in the previous one! I will test it in the evening and try to create a release.

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