Skip to content

shim/loop-dev: fix variable undefined error for musl#414

Merged
mxpv merged 1 commit intocontainerd:mainfrom
Tim-Zhang:fix-loop-dev-musl
Jul 18, 2025
Merged

shim/loop-dev: fix variable undefined error for musl#414
mxpv merged 1 commit intocontainerd:mainfrom
Tim-Zhang:fix-loop-dev-musl

Conversation

@Tim-Zhang
Copy link
Member

Fixes: #413

Fixes: containerd#413

Signed-off-by: Tim Zhang <tim@hyper.sh>
@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