path: root/support
diff options
authorGravatar Thomas Petazzoni <thomas.petazzoni@free-electrons.com>2016-08-09 12:25:11 +0200
committerGravatar Thomas Petazzoni <thomas.petazzoni@free-electrons.com>2016-08-09 12:25:11 +0200
commit34b1bdeba40bf303dc8821421745d5b38a729984 (patch)
tree2b0cd587789820390ed545e8c942ce93f6830202 /support
parent466934bb5c35856df22ff0c56021c38ee372685c (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')
1 files changed, 1 insertions, 1 deletions
diff --git a/support/scripts/size-stats b/support/scripts/size-stats
index 81d96caa12..0ddcc07902 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)