Skip to content

Conversation

@JorgeGzm
Copy link
Contributor

@JorgeGzm JorgeGzm commented Aug 15, 2025

Summary

Adds the zip package to the Docker CI environment.

This change is specifically needed because the xedge library was recently added to nuttx-apps, which runs a bash script that uses the zip package to create their web pages.
PR: #16665

Impact

Enables CI builds that require zip compression functionality.

Testing

Docker image must be created or updated based on the modified Dockerfile that installs the zip package during image creation. To verify if zip is installed and working, run the following command in the CI Docker container:

root@c9f70d04fa19:/workspace/nuttx# zip --help
Copyright (c) 1990-2008 Info-ZIP - Type 'zip "-L"' for software license.
Zip 3.0 (July 5th 2008). Usage:
zip [-options] [-b path] [-t mmddyyyy] [-n suffixes] [zipfile list] [-xi list]
  The default action is to add or replace zipfile entries from list, which
  can include the special name - to compress standard input.
  If zipfile and list are omitted, zip compresses stdin to stdout.
  -f   freshen: only changed files  -u   update: only changed or new files
  -d   delete entries in zipfile    -m   move into zipfile (delete OS files)
  -r   recurse into directories     -j   junk (don't record) directory names
  -0   store only                   -l   convert LF to CR LF (-ll CR LF to LF)
  -1   compress faster              -9   compress better
  -q   quiet operation              -v   verbose operation/print version info
  -c   add one-line comments        -z   add zipfile comment
  -@   read names from stdin        -o   make zipfile as old as latest entry
  -x   exclude the following names  -i   include only the following names
  -F   fix zipfile (-FF try harder) -D   do not add directory entries
  -A   adjust self-extracting exe   -J   junk zipfile prefix (unzipsfx)
  -T   test zipfile integrity       -X   eXclude eXtra file attributes
  -y   store symbolic links as the link instead of the referenced file
  -e   encrypt                      -n   don't compress these suffixes
  -h2  show more help

@github-actions github-actions bot added Area: Tooling Area: CI Size: XS The size of the change in this PR is very small labels Aug 15, 2025
@xiaoxiang781216 xiaoxiang781216 merged commit 433b717 into apache:master Aug 16, 2025
4 checks passed
@JorgeGzm JorgeGzm deleted the docker_add_zip_packet branch August 16, 2025 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: CI Area: Tooling Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants