Re: [PATCH 1/8] tests/docker: Fix alpine dockerfile

2022-07-28 Thread Thomas Huth
On 28/07/2022 08.52, Thomas Huth wrote: On 27/07/2022 18.36, Lucas Mateus Castro(alqotel) wrote: Currently the run script uses 'readlink -e' but the image only has the busybox readlink, this commit add the coreutils package which contains the readlink with the '-e' option. Signed-off-by: Lucas

Re: [PATCH 1/8] tests/docker: Fix alpine dockerfile

2022-07-28 Thread Thomas Huth
On 27/07/2022 18.36, Lucas Mateus Castro(alqotel) wrote: Currently the run script uses 'readlink -e' but the image only has the busybox readlink, this commit add the coreutils package which contains the readlink with the '-e' option. Signed-off-by: Lucas Mateus Castro(alqotel) --- tests/docke

Re: [PATCH 1/8] tests/docker: Fix alpine dockerfile

2022-07-27 Thread Lucas Mateus Martins Araujo e Castro
On 27/07/2022 14:09, Daniel P. Berrangé wrote: On Wed, Jul 27, 2022 at 01:36:25PM -0300, Lucas Mateus Castro(alqotel) wrote: Currently the run script uses 'readlink -e' but the image only has the busybox readlink, this commit add the coreutils package which contains the readlink with the '-e'

Re: [PATCH 1/8] tests/docker: Fix alpine dockerfile

2022-07-27 Thread Daniel P . Berrangé
On Wed, Jul 27, 2022 at 01:36:25PM -0300, Lucas Mateus Castro(alqotel) wrote: > Currently the run script uses 'readlink -e' but the image only has the > busybox readlink, this commit add the coreutils package which > contains the readlink with the '-e' option. Use of 'readlink' is discouraged in f