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

Minor changes to support Mac OSX (tested on 10.10) #35

Merged
merged 2 commits into from Jan 20, 2019

Conversation

jensp
Copy link
Contributor

@jensp jensp commented Oct 1, 2015

These commits should fix untrunc for OSX. I don't really know how to program and I found the header file on a github gist, but it's in the public domain and the content seems simple enough. Maybe this will help some poor souls stuck on Apple's OS.

@spetzreborn
Copy link

Verified to still build on linux.

@RSully
Copy link

RSully commented Nov 22, 2015

This may be helpful: https://gist.github.com/panzi/6856583

@mkoderer
Copy link

mkoderer commented Apr 1, 2016

Fix works fine for me. Can we merge it?

Copy link

@ezhes ezhes left a comment

Choose a reason for hiding this comment

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

These changes allowed me to build on OSX Sierra! I'd approve them. I did have to make this change in file.cpp to get it to build

@slhck
Copy link

slhck commented Jan 11, 2017

Actually, for OS X, you could make it easier by installing ffmpeg through Homebrew. Then, all that's needed is a -L/usr/local/lib flag for g++.

If you get the error error: use of undeclared identifier 'avcodec_alloc_frame', you have to replace all occurrences of avcodec_alloc_frame with av_frame_alloc in the source.

EDIT: This is the case for the libav libraries of FFmpeg 3.0, which is bundled with Homebrew.

@alejandroiglesias
Copy link

@slhck your instructions worked very well. Thanks!

@Letme
Copy link

Letme commented Sep 12, 2017

Can we merge that in?

@ponchio ponchio merged commit 99dbc35 into ponchio:master Jan 20, 2019
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.

None yet

9 participants