critiqjo / key-mon

Automatically exported from code.google.com/p/key-mon
https://github.com/scottkirkwood/key-mon
Apache License 2.0
97 stars 8 forks source link

OptionException #85

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. update back to older revision which does not recognize new feature

 ~/p/key-mon $ src/key-mon
Traceback (most recent call last):
  File "src/key-mon", line 3, in <module>
    km.main()
  File "/home/livibetter/p/key-mon/src/keymon/key_mon.py", line 798, in main
    opts.read_ini_file('~/.config/key-mon/config')
  File "/home/livibetter/p/key-mon/src/keymon/options.py", line 336, in read_ini
_file
    self.parse_ini(fi)
  File "/home/livibetter/p/key-mon/src/keymon/options.py", line 315, in parse_in
i
    raise OptionException('Unknown option %r in section [%s]' % (name, section))

keymon.options.OptionException: Unknown option 'backgroundless' in section [ui]

What is the expected output? What do you see instead?

In my opinion, it should print out a warning message and still keep that entry 
intact. I noticed if I just removed the `raise`, the Unknown option would be 
removed from configuration file, I guess it's saved from parsed options which 
are all valid/recognized options, therefore Unknown option is sort of being 
discarded.

This should only affect developers since they are the only people would go back 
to the future to find the doc but unfortunately the vehicle ran out of power 
and had to wait for next lightning.

Anyway, this is a minor issue, fixing it would have better convenience.

Original issue reported on code.google.com by livibet...@gmail.com on 11 Aug 2011 at 5:50

GoogleCodeExporter commented 9 years ago

Original comment by livibet...@gmail.com on 11 Aug 2011 at 5:50

GoogleCodeExporter commented 9 years ago
This issue was closed by revision 8902cb479c57.

Original comment by livibet...@gmail.com on 19 Aug 2011 at 10:16