laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/pysim/+/15500 )

Change subject: pySim-prog.py: use more expressive error message on CSV read 
failure
......................................................................

pySim-prog.py: use more expressive error message on CSV read failure

When the CSV file fails to read the error message is just "Error reading
parameters". Lets make clear that this error is related to a problem
with the CSV file

Change-Id: If285c1fbf7d285f512b573040f1b8983e4e3087e
---
M pySim-prog.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/pySim-prog.py b/pySim-prog.py
index 55634a5..2387986 100755
--- a/pySim-prog.py
+++ b/pySim-prog.py
@@ -660,7 +660,7 @@
                        imsi = opts.imsi
                cp = read_params_csv(opts, imsi=imsi, iccid=iccid)
        if cp is None:
-               print "Error reading parameters\n"
+               print "Error reading parameters from CSV file!\n"
                return 2
        print_parameters(cp)


--
To view, visit https://gerrit.osmocom.org/c/pysim/+/15500
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: If285c1fbf7d285f512b573040f1b8983e4e3087e
Gerrit-Change-Number: 15500
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-MessageType: merged

Reply via email to