On Thu, Jun 20, 2024 at 12:10:32PM GMT, Nilay Shroff wrote: > > > On 6/20/24 12:05, Daniel Wagner wrote: > > On Wed, Jun 19, 2024 at 10:55:02PM GMT, Nilay Shroff wrote: > >> + truncate -s "${NVME_IMG_SIZE}" "$(_nvme_def_file_path).$i" > >> + uuid="$(uuidgen -r)" > > > > This adds a new dependency on an external tool. It should be mentioned > > in the README and added to the list of tools to check for: > > _check_dependencies(). Alternatively you could skip the test if the tool > > is not available. Anyway the rest looks good. > > > The "uuidgen" is part of util-linux package and I saw that it's already mentioned > as one of the required packages for blktest: https://github.com/osandov/blktests Ah, I just grepped for uuidgen and there is no other user. So all is good.