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

Cannot list themes with --list-themes #54

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. key-mon --list-themes
2. key-mon --theme=apple

Please provide any additional information below.
The first command lists nothing, yet I can select a theme nevertheless with the 
second command.

The reason may be in this code:
--snip--
  if opts.list_themes:
    print _('Available themes:')
    theme_dir = os.path.join(os.path.dirname(__file__), 'themes')
    for entry in sorted(os.listdir(theme_dir)):
      try:
        parser = SafeConfigParser()
        parser.read(os.path.join(theme_dir, entry, 'config'))
        desc = parser.get('theme', 'description')
        print '%s: %s' % (entry, desc)
      except:
        pass
    raise SystemExit()
--snip--

There is no file or content in my themes directory containing "config". Or am I 
missing something?

Original issue reported on code.google.com by mbst...@googlemail.com on 20 Sep 2010 at 1:26

GoogleCodeExporter commented 9 years ago
Thanks, it should be a simple fix.

Original comment by scottaki...@gmail.com on 20 Sep 2010 at 2:28

GoogleCodeExporter commented 9 years ago
Problem with MANIFEST.in not including config files.
Fixed in repository.

Original comment by scottaki...@gmail.com on 21 Sep 2010 at 1:54

GoogleCodeExporter commented 9 years ago
This works perfectly now in key-mon 1.4.1.

Thanks for the fix!

Original comment by mbst...@googlemail.com on 22 Sep 2010 at 11:53