diff options
| author | Thomas Petazzoni <thomas.petazzoni@free-electrons.com> | 2016-08-09 12:25:11 +0200 |
|---|---|---|
| committer | Thomas Petazzoni <thomas.petazzoni@free-electrons.com> | 2016-08-09 12:25:11 +0200 |
| commit | 34b1bdeba40bf303dc8821421745d5b38a729984 (patch) | |
| tree | 2b0cd587789820390ed545e8c942ce93f6830202 /support | |
| parent | 466934bb5c35856df22ff0c56021c38ee372685c (diff) | |
size-stats: fix bug when parsing file names with comma
The size-stats script fails when the usb_modeswitch_data is enabled,
because this package installs files that contain commas in their
name. However, the size-stats script also uses comma as a separator for
its CSV files, causing a "ValueError: too many values to unpack" in:
pkg, fpath = l.split(",")
Fix this by splitting only the two fields that need to be split.
The bug was reported by Matthias <porto.rio@gmx.net>, who also suggested
a fix.
Fixes bug #9136.
Reported-by: Matthias <porto.rio@gmx.net>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Diffstat (limited to 'support')
| -rwxr-xr-x | support/scripts/size-stats | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/support/scripts/size-stats b/support/scripts/size-stats index 81d96caa1..0ddcc0790 100755 --- a/support/scripts/size-stats +++ b/support/scripts/size-stats @@ -66,7 +66,7 @@ def build_package_dict(builddir): filesdict = {} with open(os.path.join(builddir, "build", "packages-file-list.txt")) as filelistf: for l in filelistf.readlines(): - pkg, fpath = l.split(",") + pkg, fpath = l.split(",", 1) # remove the initial './' in each file path fpath = fpath.strip()[2:] fullpath = os.path.join(builddir, "target", fpath) |