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

Explicitly disable use of cgo #101

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented Oct 6, 2020

As far as I can tell, the v0.2.5 release provided binaries
that are not statically linked against glibc and thus
would not be held to LGPL license requirements. AFAICT.

Update Makefile to disable use of cgo for binary generation
in an effort to avoid potential inclusion in the future.

This is likely not the last time I'll need to update
the Makefile build options for static binary generation,
but hopefully this will do for now.

refs #94
refs atc0005/mysql2sqlite#48

As far as I can tell, the v0.2.5 release provided binaries
that are *not* statically linked against glibc and thus
would not be held to LGPL license requirements. AFAICT.

Update Makefile to disable use of cgo for binary generation
in an effort to avoid potential inclusion in the future.

This is likely *not* the last time I'll need to update
the Makefile build options for static binary generation,
but hopefully this will do for now.

refs #94
refs atc0005/mysql2sqlite#48
@atc0005 atc0005 added bug Something isn't working builds labels Oct 6, 2020
@atc0005 atc0005 added this to the Next Release milestone Oct 6, 2020
@atc0005 atc0005 self-assigned this Oct 6, 2020
@atc0005 atc0005 added the license label Oct 6, 2020
@atc0005 atc0005 merged commit ede7c0e into master Oct 6, 2020
@atc0005 atc0005 deleted the i94-explicitly-disable-cgo-instead-of-allowing-potential-for-static-inclusion-of-glibc branch October 6, 2020 12:31
atc0005 added a commit to atc0005/check-cert that referenced this pull request Oct 7, 2020
The previous build command produces dynamic executables
instead of the intended (but not well noted in the Makefile)
static executables. This commit includes changes noted in
upstream `golang/go` issues which appear to work as intended
for others.

I also include doc comments and reference links describing
the use of the updated build options list and some problems
that I encountered during testing of `linkmode=external`.

- refs GH-92

- refs atc0005/check-mail#94
- refs atc0005/check-mail#95
- refs atc0005/check-mail#101

- refs golang/go 38789
- refs golang/go 26492
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working builds license
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant