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

Add ffmpeg link for windows #1287

Merged
merged 1 commit into from Feb 26, 2021
Merged

Add ffmpeg link for windows #1287

merged 1 commit into from Feb 26, 2021

Conversation

cashlo
Copy link
Contributor

@cashlo cashlo commented Feb 22, 2021

Installing gstreamer and pygobject on windows is a lot of pain, it turns out installing ffmpeg is a lot easier. Adding a link for ffmpeg and I hope it will save someone some pain.

Reference Issue

What does this implement/fix? Explain your changes.

Any other comments?

Installing gstreamer and pygobject on windows is a lot of pain, it turns out installing ffmpeg is a lot easier. Adding a link for ffmpeg and I hope it will save someone some pain.
@bmcfee bmcfee added the documentation Issues relating to docstrings, examples, and documentation build label Feb 22, 2021
@bmcfee
Copy link
Member

bmcfee commented Feb 22, 2021

Thanks for this, but I'm a bit puzzled. The link we had (gstreamer.freedesktop.org) already included binaries, so why do we need a second link?

@cashlo
Copy link
Contributor Author

cashlo commented Feb 23, 2021

I just couldn't get pygobject to install on Windows, but for ffmpeg I downloaded the binaries and added them to path and that was it. Am I missing something? The new link is for ffmpeg.

@bmcfee bmcfee added this to the 0.8.1 milestone Feb 23, 2021
@bmcfee
Copy link
Member

bmcfee commented Feb 23, 2021

Ah, sorry about that -- I read too quickly. This LGTM, thanks!

@bmcfee bmcfee merged commit 8d9a1de into librosa:main Feb 26, 2021
@cashlo cashlo deleted the patch-1 branch March 8, 2021 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issues relating to docstrings, examples, and documentation build
Development

Successfully merging this pull request may close these issues.

None yet

2 participants