iustin / bakonf

A very simple tool designed to make backups of the configuration files

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Example bakonf.yml produces error

jose1711 opened this issue · comments

configs: /etc/bakonf/conf.d/*.yml

configs entry in config file is treated as a list which causes the example config file to produce an error:

Traceback (most recent call last):
  File "/usr/bin/bakonf", line 1079, in <module>
    main()
  File "/usr/bin/bakonf", line 1070, in main
    real_main()
  File "/usr/bin/bakonf", line 1063, in real_main
    bm = BackupManager(options)
  File "/usr/bin/bakonf", line 706, in __init__
    self._parseconf(options.configfile)
  File "/usr/bin/bakonf", line 765, in _parseconf
    tlist = self._get_extra_sources(filename, config)
  File "/usr/bin/bakonf", line 732, in _get_extra_sources
    with open(fname) as stream:
IsADirectoryError: [Errno 21] Is a directory: '/'

In order to fix this the line needs to be changed to a single-item list, e. g. like this:

configs:
- /etc/bakonf/conf.d/*.yml

I'll try to make a release will both fixes and also the pending commits sometimes next weekend or so. Thanks!