Attention is currently required from: daniel, fixeria, laforge.
13 comments:
Commit Message:
remove […]
Done
File contrib/csv-to-pgsql.py:
Patch Set #4, Line 16: # cmd2 >= 2.3.0 has deprecated the bg/fg in favor of Bg/Fg :(
Other than `YELLOW`, `cmd2` API is not used in this script. […]
ufortunately the API of the PySimLogger only accepts those CMD2 enums. To make the use of the PySimLogger easier in stand-alone programs like this I have added some flexibility to it (see the patch before this one). Now we can just pass an ANSI color code like you suggest.
Patch Set #4, Line 225: log.info("CSV file: %s" % opts.csv)
BTW, for logging API it's recommended to use lazy format-sting evaluation: […]
Done
Patch Set #4, Line 227: if not csv_file:
Does `open()` return `None` at all? It raises an exception on error.
Done
File docs/card-key-provider.rst:
Patch Set #4, Line 25: CardKeyProviderCsv
```suggestion […]
Done
may
Done
```suggestion […]
Done
Patch Set #4, Line 293: CardKeyProviderCsv
```suggestion […]
Done
File pySim-shell.py:
Patch Set #4, Line 1186: card_key_provider_register(CardKeyProviderPgsql(opts.pqsql, column_keys))
That means you can use both providers at the same time, right?
Yes, this is intentional. Users may want to use a local CSV file in addition. However, one of the two will probably the usual case.
File pySim/card_key_provider.py:
column_names = ""
for f in fields:
column_names += f.lower() + ", "
column_names = column_names[:-2]
This can be done with join: […]
Done
Patch Set #4, Line 229: cur.execute("SELECT column_name FROM information_schema.columns where table_name = '%s';" % t)
yeah. […]
Done
Patch Set #4, Line 242: break;
cosmetic: semicolon is not needed here
Done
File setup.py:
Patch Set #4, Line 37: psycopg2-binary
Not blocking here, but do we really want this as a required dependency? This list is already quite l […]
What if we leave it out and mention it in the manual that it is needed when someone wants to use the CardKeyProviderPgsql? Would that be an option? There shouldn't be any problems until someone tries to actually use CardKeyProviderPgsql.
To view, visit change 41508. To unsubscribe, or for help writing mail filters, visit settings.