Re: [PATCH] usb-serial: translate ezusb_convert

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Sep 23, 2016 at 09:59:57AM +0200, Greg KH wrote:
> On Thu, Sep 22, 2016 at 05:54:25PM -0600, Jorge Natz wrote:
> > 
> > ezusb_convert is a script used to convert an Intel HEX firmware file
> > into a set of C arrays for use in header files. Increase the
> > portability of this script by translating it from Perl to a
> > POSIX shell script. Perl is not was widely installed on UNIX machines as
> > is a POSIX-conforming shell, therefore portability of this script, and
> > therefore a small part of the build, should be increased.
> 
> Perl is much more portable than you think, and installed more than you
> think :)

Well, sometimes I have to compile kernel on Windows host with no cygwin,
where Perl is not installed.

> I don't really see the need for this change at all, and oddly, you leave
> the file name with a ".pl" at the end.

I see ezusb_convert.sh helper script filename in the patch...

> So thanks for the patch, but I don't think we need it.

I can also certainly live without it and I'm aware that there are a few
other perl scripts left in tree, but I consider this idea usefull.

	ladis (speaking for negligible fraction of users)
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux