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

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: