Skip to content

Conversation

@Tim-Zhang
Copy link
Member

Fixes: #413

@Tim-Zhang Tim-Zhang force-pushed the fix-loop-dev-musl branch from 0b9ffdf to 66edbe6 Compare July 17, 2025 11:56
@Tim-Zhang
Copy link
Member Author

@zzzzzzzzzy9 Can you help me out with the unit test mount_linux::tests::test_setup_loop_dev failed?

@Tim-Zhang
Copy link
Member Author

The unit test problem existed before the PR. This PR fixed the bug and exposes the existing problem.
@zzzzzzzzzy9 @mxpv Please help

@fidencio fidencio mentioned this pull request Jul 18, 2025
Copy link
Collaborator

@fidencio fidencio 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 @Tim-Zhang!

It does solve the build issue on musl. Just keep in mind that I'm not a maintainer here.

@mxpv mxpv merged commit d8f2855 into containerd:main Jul 18, 2025
10 of 15 checks passed
@Tim-Zhang Tim-Zhang deleted the fix-loop-dev-musl branch July 21, 2025 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-shim Containerd shim

Projects

None yet

Development

Successfully merging this pull request may close these issues.

containerd-shim: The variable ret does not defined in musl env

3 participants