Attention is currently required from: daniel, fixeria, laforge.

View Change

2 comments:

• File contrib/csv-to-pgsql.py:

  • Patch Set #8, Line 37: PQSQL

    PGSQL or PSQL are usual acronyms for postgres. […]

    Done

  • Patch Set #8, Line 261: str(Path.home()) + "/.osmocom/pysim/card_data_pqsql.cfg"

    Not critical, but I guess the clean/pythonic way would be to use `os.path. […]

    Thanks, having the user name in the helptext is indeed something we should avoid. I remember having problems with resolving ~, but os.path.expanduser() works well when I apply it on the path when accessing opts.pqsql.

    I have also fixed this for pySim-shell.py

To view, visit change 41508. To unsubscribe, or for help writing mail filters, visit settings.