codetree doesn't handle badly formed options in URLs

[Tom Haddon]

If you have a URL that you're passing to codetree such as:

postgresql cs:postgresql?overwrite

You get a traceback as follows:

Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/codetree/config.py", line 92, in __init__
    lineno=fileinput.filelineno())
  File "/usr/lib/python2.7/dist-packages/codetree/config.py", line 24, in from_raw_line
    url, D.source_options = cls.parse_source(source)
  File "/usr/lib/python2.7/dist-packages/codetree/config.py", line 35, in parse_source
    options.update(dict((opt.split("="),)))
ValueError: dictionary update sequence element #0 has length 1; 2 is required

Codetree should give a more informative error, highlighting the line it's having problems parsing.

[Tim Kuhlman (timkuhlman)]

Note that the line 'postgresql cs:postgresql?overwrite' actually creates a different error as it tries to send that to the charm store. The error above is from 'postgresql cs:postgresql;overwrite'