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

empty config file #5

Open
N0W3N opened this issue Feb 17, 2020 · 0 comments
Open

empty config file #5

N0W3N opened this issue Feb 17, 2020 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@N0W3N
Copy link
Owner

N0W3N commented Feb 17, 2020

I have discovered an issue with the install.py file when you try to create the config.ini

When config.ini already exists, but is either empty or doesn't contain the required headers, it can't continue and terminates the process.

with open('config.ini', 'r') as cfg: path = **config.get('Path of ffmpeg', 'path')**

config.get raises an error if the given section (e.g. Path of ffmpeg) doesn't exist.

This issue could be resolved with adding an exception-clause into the code block, since ConfigParser provides great error handlings by itself.
I'll delay this part of issue, since this code block is already redundant and will be replaced with a more efficient class.

@N0W3N N0W3N added the bug Something isn't working label Feb 17, 2020
@N0W3N N0W3N self-assigned this Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant