Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3.2.0a4: dangling commas in service lists #46

Closed
matthewwall opened this issue Jul 6, 2015 · 1 comment
Closed

3.2.0a4: dangling commas in service lists #46

matthewwall opened this issue Jul 6, 2015 · 1 comment

Comments

@matthewwall
Copy link
Contributor

  1. clean install
  2. check weewx.conf: no comma at end of archive_services
  3. wee_extension --install forecast.tgz
  4. wee_extension --uninstall forecast
  5. archive_services now has a dangling comma

also, data_services starts out with a dangling comma. the code should be smart enough to deal with dangling commas, but we should not put them in weewx.conf in the first place and whatever wee_* utilities process weewx.conf should not leave them.

@tkeffer
Copy link
Contributor

tkeffer commented Jul 6, 2015

A dangling comma is actually ConfigObj's way of signalling that the contents are a list. To work around this, we developed utility weeutil.weeutil.option_as_list()

@tkeffer tkeffer closed this as completed Jul 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants