Commit ecace27
committed
fix: don't symlink buildkit-cni documentation files to bin/
The nerdctl-full tarball was incorrectly creating symlinks for all
files in libexec/cni/, including documentation files like README.md
and LICENSE. This resulted in non-executable files appearing in bin/
as buildkit-cni-README.md and buildkit-cni-LICENSE.
Add executable and regular file checks to the symlink creation loop
to filter out non-executable files. The fix uses [ -x "$f" ] to check
for execute permission and [ -f "$f" ] to ensure it's a regular file,
so only actual CNI plugin binaries are symlinked.
Tested: bin/ file count reduced from 46 to 44 files (removed 2 doc
symlinks). All 18 CNI plugin executables still correctly symlinked.
Fixes containerd#4553
Signed-off-by: Sadique Azmi <[email protected]>1 parent 2db28f3 commit ecace27
1 file changed
+1
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
167 | 167 | | |
168 | 168 | | |
169 | 169 | | |
170 | | - | |
| 170 | + | |
171 | 171 | | |
172 | 172 | | |
173 | 173 | | |
| |||
0 commit comments