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

[apps][minizip] add minizip app & 增加 minizip 应用 #31

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

WCX1024979076
Copy link

增加minizip应用和repo,目前在qemu-virt64-aarch64功能上测试通过,测试流程如下。

msh />cd tmp
msh /tmp>echo test a.txt
msh /tmp>echo test b.txt
msh /tmp>minizip -o -9 target.zip a.txt b.txt
msh /tmp>MiniZip 1.01b, demo of zLib + Zip package written by Gilles Vollant
more info at http://www.winimage.com/zLibDll/minizip.html

creating target.zip

msh /tmp>rm a.txt b.txt
msh /tmp>miniunz -l target.zip
msh /tmp>MiniUnz 1.01b, demo of zLib + Unz package written by Gilles Vollant
more info at http://www.winimage.com/zLibDll/minizip.html

target.zip opened
 Length  Method   Size  Ratio   Date    Time   CRC-32     Name
 ------  ------   ----  -----   ----    ----   ------     ----
      4  Defl:X       6 150%  01-01-50  00:00  d87f7e0c   a.txt
      4  Defl:X       6 150%  01-01-50  00:00  d87f7e0c   b.txt

msh /tmp>miniunz -x target.zip
msh /tmp>MiniUnz 1.01b, demo of zLib + Unz package written by Gilles Vollant
more info at http://www.winimage.com/zLibDll/minizip.html

target.zip opened
 extracting: a.txt
 extracting: b.txt

@BernardXiong
Copy link
Member

感觉似乎只需要在repo中加入即可,不需要在apps下添加。

@WCX1024979076
Copy link
Author

感觉似乎只需要在repo中加入即可,不需要在apps下添加。

这个倒是可以把minizip_test.c给删掉,minizip.cminiunz.c可以给终端msh提供压缩和解压的命令。

@mysterywolf
Copy link
Member

minizip 已经是rt-thread的软件包了

@WCX1024979076
Copy link
Author

minizip 已经是rt-thread的软件包了

这个确实在RT-Thread-packages里实现了minizip,用户可以在scons --menuconfig里选择使用RT-Thread-packakges/minizip软件包。我这里的PR是在userapps利用xmake来实现minizip,给用户多一种选择可以选择不重新编译rtthread.bin的方式来实现使用minizip。但是确实会有一些冗余,理论上讲RT-Thread-packages里面实现的用户软件包都可以迁移到userapps中,包括vt100zlibminizip等等。

@BernardXiong
Copy link
Member

BernardXiong commented Aug 3, 2023

minizip 已经是rt-thread的软件包了

userapps里的和rt-thread的软件包关系不大

@BernardXiong
Copy link
Member

从整体上看,并不希望在apps文件夹下加入无数的应用,这里面的应该更多的是示例性质的。然后相关的应用或库,都以repo方式存在吧。

@WCX1024979076
Copy link
Author

从整体上看,并不希望在apps文件夹下加入无数的应用,这里面的应该更多的是示例性质的。然后相关的应用或库,都以repo方式存在吧。

可以的,那我整理一下代码并测试一下。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants