Include "zip" as required apt dependency in bootstrap error message

[chausner]

When running bootstrap.sh on a new Ubuntu installation, it currently fails with the message:

Could not find unzip. Please install it (and other dependencies) with:
sudo apt-get install curl unzip tar

The mentioned dependencies are enough to bootstrap vcpkg but when actually installing packages, it turns out that also zip is required:

...
-- Configuring x64-linux-dbg
-- Configuring x64-linux-rel
-- Building x64-linux-dbg
-- Building x64-linux-rel
-- Installing: /mnt/d/vcpkg/packages/rtaudio_x64-linux/share/rtaudio/README.md
-- Installing: /mnt/d/vcpkg/packages/rtaudio_x64-linux/share/rtaudio/copyright
-- Performing post-build validation
-- Performing post-build validation done
sh: 1: zip: not found

So I suggest to also include zip in the error message in the bootstrap script.

[strega-nil]

I think this is a good change; however, it does kinda suck that the error message is sh: 1: zip: not found. I'll open an issue for this error message. If you're interested, we'd love the help!

[ihnorton]

Is zip still required / required on linux? Related (possibly dumb) question: where is it used... I grep'd and I think I only see the usage of tar, but not zip.

[edit]:

Disabled the tool check for zip, and yes it is still required:

-- Performing post-build validation
warning: Failed to compress folder "/root/TileDB/external/vcpkg/packages/liblzma_x64-linux": error: zip failed with exit code: (127).
sh: zip: command not found

the usage comes from here: https://github.com/microsoft/vcpkg-tool/blob/2c5416ae0793584f2baf456ee13182af1cd138b5/src/vcpkg/binarycaching.cpp#L348