[Tarantool-patches] [PATCH] tools: implement toolchain for crash artefacts

Alexander Turenko alexander.turenko at tarantool.org
Wed Mar 24 23:25:29 MSK 2021


> 1. in a private conversation we decided to remove mention of public resource
> in a message "http://transfer.sh".

Can you (or Igor) share a reason?

>  TOOL=$(basename "$0")
> +#TOOLPATH="$( cd -- "$(dirname "$0")" >/dev/null 2>&1 ; pwd -P )"
> +TOOLPATH=$(realpath -s $0)

'realpath' is not necessarily available. Actually I saw that it may be
not available on Mac OS, but I didn't saw such situation on Linux.
However realpath from busybox does not support -s option (verified on
Alpine 3.10).

My opinion: it is better to don't extend the list of external
dependencies in this case.

I would also highlight that it is quite usual to use just $0 in a usage
message:

 | $ cat --help | head -n 1
 | Usage: cat [OPTION]... [FILE]...
 |
 | $ /bin/cat --help | head -n 1
 | Usage: /bin/cat [OPTION]... [FILE]...


More information about the Tarantool-patches mailing list