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

#243 Fix Error! Unhandled error (execa) #385

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

Conversation

olegshulyakov
Copy link

@olegshulyakov olegshulyakov commented Jun 18, 2022

Fix #243, #245, #294, #346, #363, #365, #377.

@ghost
Copy link

ghost commented Jun 18, 2022

Really poor form to take commits from #375 without credit.

@karikamiya
Copy link

@olegshulyakov can you publish binaries on your fork with these changes built in?

@StefanLobbenmeier
Copy link
Contributor

Feel free to use the release in my fork: https://github.com/StefanLobbenmeier/youtube-dl-gui/releases

but the Windows build is broken at the moment

@olegshulyakov
Copy link
Author

@karikamiya What OS do you need them to?
You can build them by yourself if needed. Just grab main branch from my fork and follow steps in GitHub workflow. Or if you already use VSCode there is a build task to make at as simple as can.

@W1nKille3r
Copy link

Windows 22H2
Same error

Doesn't appear at 20H2

Copy link

@StoppedwummSites StoppedwummSites left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything looks right, code is working

@olegshulyakov
Copy link
Author

@jely2002 Please take a look

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.

Error! Unhandled error (execa).
5 participants