On Sun, Sep 25, 2016 at 11:36:01AM -0600, Jorge Natz wrote: > ---------- Forwarded message ---------- > From: Jorge Natz <jorgenatzdev@xxxxxxxxx> > Date: Sat, Sep 24, 2016 at 10:38 AM > Subject: Re: [PATCH] usb-serial: translate ezusb_convert > To: Greg KH <greg@xxxxxxxxx> > > > > > On 09/23/2016 01:59 AM, Greg KH wrote: > > > Perl is much more portable than you think, and installed more than you > > think :) > > There are lightweight systems (think busybox based) that would benefit from > this change, as many of these systems do not have a Perl interpreter > installed, and installing one would pose an inconvenience due to the size > and dependency requirements of the interpreter. Granted, this is a small > difference, but why not? Because you don't do conversion of a firmware binary file on a "lightweight" system? It's nothing anyone normally runs at all. > > 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. > > The only reason that 'ezusb_convert.pl' is written inside the patch is > for the need to remove this script. Farther down in the patch, it adds > the script 'ezusb_convert.sh'. > > For more reinforced arguments as to why Perl should be removed from > the build process, see: > https://lkml.org/lkml/2009/1/2/20 But this script isn't part of the build process. When was the last time you used this script? thanks, greg k-h -- 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
