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 to stackage #24

Open
expipiplus1 opened this issue Jan 21, 2016 · 6 comments
Open

Add to stackage #24

expipiplus1 opened this issue Jan 21, 2016 · 6 comments

Comments

@expipiplus1
Copy link
Contributor

It would be super to have this package in stackage.

Instructions are here: https://github.com/fpco/stackage/blob/master/MAINTAINERS.md#adding-a-package

@acowley
Copy link
Owner

acowley commented Jan 21, 2016

I'd really like to fix up the CI tests before adding to Stackage. The issue is that the ffmpeg libraries available on Travis are ancient, so I currently just fake all the missing C definitions so that GHC can at least compile the package. This served as a minimal smoke test for building the package for a while, but the C stubs fell out of date at some point, and I haven't fixed them yet.

There is an issue open somewhere for Travis to whitelist current ffmpeg, but there hasn't been any movement on it for a while. The next step is to look at the Travis log and see if we can just define the necessary constants ourselves.

@expipiplus1
Copy link
Contributor Author

Is it not possible to installed a more up to date version of ffmpeg as part of the CI build? I'm assuming no.

@acowley
Copy link
Owner

acowley commented Jan 21, 2016

Not in the new containerized infrastructure, unless the source is white listed. One option is to build ffmpeg ourselves and cache the build directory, but the procedure for doing so (included here in the ffmpeg-light repo for Ubuntu) may still require an apt source that isn't already white listed. I'll have a look when at a computer. If you have a chance to investigate, let me know.

@acowley
Copy link
Owner

acowley commented Jan 21, 2016

Here's the issue to let travis containers pull in a current ffmpeg.

@ghost
Copy link

ghost commented May 30, 2018

Did you have to put in a request to add ffmpeg to hackage server?

@acowley
Copy link
Owner

acowley commented May 31, 2018

Thank you for reminding me about this @kanishka-azimi! I fixed the CI situation recently, but had forgotten about this PR. I've made a PR on stackage.

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

No branches or pull requests

2 participants