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

Remove MUSL and enable threadless libgit2 support #411

Merged
merged 1 commit into from
Aug 1, 2022

Conversation

pjbgf
Copy link
Member

@pjbgf pjbgf commented Jul 29, 2022

Use of MUSL was a temporary solution to mitigate cross-platform
issues while building openssl and libssh2. Since Unmanaged transport has
been deprecated, openssl and libssh2 dependencies are no longer required
and by extension MUSL.

Enables libgit2 threadless support and provides a regression protection
for #339.

Relates to fluxcd/golang-with-libgit2#37
Fixes #339

@pjbgf pjbgf force-pushed the musl-free branch 3 times, most recently from 49e3562 to 0a2fbc7 Compare July 29, 2022 11:04
@pjbgf pjbgf added the area/git Git related issues and pull requests label Jul 29, 2022
@pjbgf pjbgf force-pushed the musl-free branch 5 times, most recently from d88b7d1 to 2e97297 Compare July 31, 2022 18:46
Use of MUSL was a temporary solution to mitigate cross-platform
issues while building openssl and libssh2. Since Unmanaged transport has
been deprecated, openssl and libssh2 dependencies are no longer required
and by extension MUSL.

Enables libgit2 threadless support and provides a regression assurance
for fluxcd#339.

Signed-off-by: Paulo Gomes <paulo.gomes@weave.works>
@pjbgf pjbgf changed the title Remove MUSL dependencies Remove MUSL and enable threadless libgit2 support Jul 31, 2022
@pjbgf pjbgf marked this pull request as ready for review July 31, 2022 19:16
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @pjbgf 🥇

@stefanprodan stefanprodan merged commit 34a10c0 into fluxcd:main Aug 1, 2022
@pjbgf pjbgf deleted the musl-free branch August 2, 2022 13:14
@pjbgf pjbgf added this to the GA milestone Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/git Git related issues and pull requests
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

image-automation-controller CrashLoopBack >= 0.20.2
2 participants