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

Sergey Bronnikov sergeyb at tarantool.org
Thu Mar 25 12:45:21 MSK 2021


Hello,

On 24.03.2021 23:25, Alexander Turenko wrote:
>> 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?

We suppose that coredump may contain a sensitive data

and it would be wrong to recommend to share an archive on a public file 
hosting (at least without encryption).

>>   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