On Thu, Apr 21, 2022 at 01:30:29PM +0200, Henrik Riomar wrote:
> On Thu, Apr 21, 2022 at 1:11 PM Richard W.M. Jones <rjones@xxxxxxxxxx> wrote:
> >
> > On Thu, Apr 21, 2022 at 12:54:42PM +0200, Henrik Riomar wrote:
>
> > > I now note that commit b7cc3d93a613ef6d0ac5ccd6e32cc3d66e057243, that
> > > is part of v1.22 introduced two bashisms:
> > >
> > > possible bashism in ./virt-what line 119 (echo -e):
> > > │
> > > if ( { echo -e "GET /latest/meta-datainstance/instance-type
> > > HTTP/1.0\r\nHost: 100.100.100.200\r\n\r" │
> > > >&3; grep -sq 'ebm' <&3 ; } 3<> /dev/tcp/100.100.100.200/80 ) 2>/dev/null ; then │
> > > possible bashism in ./virt-what line 119 (/dev/(tcp|udp)):
> > > │
> > > if ( { echo -e "GET /latest/meta-datainstance/instance-type
> > > HTTP/1.0\r\nHost: 100.100.100.200\r\n\r" │
> > > >&3; grep -sq 'ebm' <&3 ; } 3<> /dev/tcp/100.100.100.200/80 ) 2>/dev/null ; then
> > >
> > > Can this be fixed? as that is the only bashisms in virt-what.
> >
> > Can you send a patch to fix it?
>
> The echo -e should be a simple replace with printf "%b\n", but the
> code using file descriptor 3 and the bash feature to open a tcp
> socket, needs some more work,
> it can be replaced with nc, but the problem is that I have no Alibaba
> Cloud VM to test a fix on.
Or just change the first line of the script to #!/bin/bash, so it
doesn't rely on the default shell impl.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
