Re: [pve-devel] [PATCH storage] storage: add bzip2 support

2024-08-06 Thread Maximiliano Sandoval
Stoiko Ivanov writes: > In general I think the addition of bz2 for ISOs makes sense (if only for > OPNSense's sake) - Thanks for addressing this! > > You want to add bzip2 as dependency to debian/control > we probably should also add lzop, zstd, gzip to the dependencies > although they are pulle

Re: [pve-devel] [PATCH storage] storage: add bzip2 support

2024-08-04 Thread Christian Ebner
On 8/2/24 18:41, Stoiko Ivanov wrote: [2] https://opnsense.org/download/ src/PVE/Storage.pm | 3 +++ src/PVE/Storage/Plugin.pm | 2 +- src/test/archive_info_test.pm | 6 -- src/test/list_volumes_test.pm | 11 ++- src/test/parse_volname_tes

Re: [pve-devel] [PATCH storage] storage: add bzip2 support

2024-08-02 Thread Stoiko Ivanov
In general I think the addition of bz2 for ISOs makes sense (if only for OPNSense's sake) - Thanks for addressing this! You want to add bzip2 as dependency to debian/control we probably should also add lzop, zstd, gzip to the dependencies although they are pulled in as dependency of pve-manger (d

[pve-devel] [PATCH storage] storage: add bzip2 support

2024-08-02 Thread Maximiliano Sandoval
This was requested at [1]. [1] https://bugzilla.proxmox.com/show_bug.cgi?id=5267 Suggested-by: Stoiko Ivanov Signed-off-by: Maximiliano Sandoval --- For the sake of not having a commit where tests fail, this is all in one commit. Please tell me if I should split it. The idea is to expose this