Re: [PATCH] REGTESTS: replace "which" with POSIX "command"

2020-09-26 Thread Илья Шипицин
On Sat, Sep 26, 2020, 1:39 PM Willy Tarreau  wrote:

> On Sat, Sep 26, 2020 at 01:21:22PM +0500,  ??? wrote:
> > Fedora docker also comes without "find" utility
> >
> > ## Gathering tests to run
> ##
> > ./scripts/run-regtests.sh: line 131: find: command not found
> > ## Starting vtest ##
>
> Isn't it worth filing a bug report to whoever maintains that image ?
> Because quite frankly, a UNIX-like system without the ubiquitous
> "find" command is kind of pointless. It cannot even be a matter of
> place at this point, since even busybox provides a working one if
> they need.
>

I'll try to rewrite "find" usage to fail if it's missing


> Willy
>


Re: [PATCH] REGTESTS: replace "which" with POSIX "command"

2020-09-26 Thread Willy Tarreau
On Sat, Sep 26, 2020 at 01:21:22PM +0500,  ??? wrote:
> Fedora docker also comes without "find" utility
> 
> ## Gathering tests to run ##
> ./scripts/run-regtests.sh: line 131: find: command not found
> ## Starting vtest ##

Isn't it worth filing a bug report to whoever maintains that image ?
Because quite frankly, a UNIX-like system without the ubiquitous
"find" command is kind of pointless. It cannot even be a matter of
place at this point, since even busybox provides a working one if
they need.

Willy



Re: [PATCH] REGTESTS: replace "which" with POSIX "command"

2020-09-26 Thread Илья Шипицин
сб, 26 сент. 2020 г. в 13:14, Willy Tarreau :

> Hi Ilya,
>
> On Sat, Sep 26, 2020 at 11:58:48AM +0500,  ??? wrote:
> > Hello,
> >
> > I've found that "socat" was not properly detected under Fedora docker
> image.
>
> Thanks, now applied.
>
> > (maybe we should introduce "set -e" as well)
>
> This could be an idea, indeed. But overall any error will be spotted one
> way or another.
>


Fedora docker also comes without "find" utility

## Gathering tests to run ##
./scripts/run-regtests.sh: line 131: find: command not found
## Starting vtest ##
Testing with haproxy version: 2.3-dev5
No tests found that meet the required criteria
[root@fbf7e85bfd94 haproxy]# echo $?
0
[root@fbf7e85bfd94 haproxy]#


>
> Willy
>


Re: [PATCH] REGTESTS: replace "which" with POSIX "command"

2020-09-26 Thread Willy Tarreau
Hi Ilya,

On Sat, Sep 26, 2020 at 11:58:48AM +0500,  ??? wrote:
> Hello,
> 
> I've found that "socat" was not properly detected under Fedora docker image.

Thanks, now applied.

> (maybe we should introduce "set -e" as well)

This could be an idea, indeed. But overall any error will be spotted one
way or another.

Willy