Bug#913001: live-wrapper: Handle firmware list more robustly by using proper whitespace split
Contro: tag -1 pending
On Mon, Nov 05, 2018 at 07:50:37PM +0000, Witold Baryluk wrote:
>Package: live-wrapper
>Version: 0.7
>Severity: normal
>Tags: patch
>
>Hi,
>
>Patch against current master branch in attachement.
>
>"
> Fix splitting of firmare package names
>
> split(' ') literally splits on a single space:
>
> 'firmware-a firmware-b'.split(' ') == ['firmware-a', '', 'firmware-b']
>
> Use split() that automatically removes empty elements (equivelent to
> strip + remove if empty), and splits on tabs and newlines.
>
> This makes human provided arguments be handled more robustly.
>"
Apologies for the delay. I've just pushed this change into git
now. It'll also get picked up by our builds from ths point. (Glad I
checked - they were still trying to pull from alioth!)
--
Steve McIntyre, Cambridge, UK. steve@einval.com
"Since phone messaging became popular, the young generation has lost the
ability to read or write anything that is longer than one hundred and sixty
characters." -- Ignatios Souvatzis
Reply to: