2019/3/9 10:46, Laurent Vivier:
> On 06/03/2019 05:49, Unai Martinez-Corral wrote:
> > -QEMU_PATH=/usr/local/bin
> > -
> > +QEMU_PATH="${QEMU_PATH:-/usr/local/bin}"
> > +QEMU_SUFFIX="${QEMU_SUFFIX:-}"
> > QEMU_CREDENTIAL="${QEMU_CREDENTIAL:-no}"
> > QEMU_PERSISTENT="${QEMU_PERSISTENT:-no}"
> >
> > -QEMU_SUFFIX=""
>
> Same question as for PATCH 3/10: why?
Same context. Moreover, in this case envvars already existed and were
hardcoded. I don't think it does any harm to let a external caller set
the defaults.
> And I think the usage text should be formatted differently to be cleared and
> looks
> like more the one of qemu-XXX:
Agree. I will update when the remaining patches are reviewed, since 6
out of 10 are affected by this change.
Unai
