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

Fix "ValueError" when Parsing Data && add recginzing "ffmpeg.exe" on Windows OS #128

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

iWangJiaxiang
Copy link

My PR has following changes:

except (ValueError, IndexError):   
  • Add recginzing "ffmpeg.exe" on Windows OS. Now you do not need any operation about ffmpeg.exe on Windows. The codes are as fsollowed:
def ffmpeg_check():
    """
    Return the ffmpeg executable name. "null" returned when no executable exsits.
    """
    if which("ffmpeg"):
        return "ffmpeg"
    elif which("ffmpeg.exe"):
        return "ffmpeg.exe"
    else:
        return None

Enjoy! :D

@BingLingGroup
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants