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

Build failure with ffmpeg git master #179

Closed
orbea opened this issue Jan 18, 2018 · 5 comments
Closed

Build failure with ffmpeg git master #179

orbea opened this issue Jan 18, 2018 · 5 comments

Comments

@orbea
Copy link
Contributor

orbea commented Jan 18, 2018

Expected behavior

visulaboyadvance-m should build with ffmpeg.

Actual behavior

visualboyadvance-m does not build with the latest ffmpeg mater due to API changes.

make[2]: *** [CMakeFiles/vbamcore.dir/build.make:231: CMakeFiles/vbamcore.dir/src/common/ffmpeg.cpp.o] Error 1

Full build log - https://pastebin.com/4z3bvTzE

Steps to reproduce the behavior

  1. Build ffmpeg from a recent snapshot or their git master
  2. Try building visualboyadvance-m with the new ffmpeg.

Build details

linux
gcc-7.2.0
ffmpeg-2018.01.08_3f234a0

@rkitover rkitover added the bug label Jan 18, 2018
@rkitover
Copy link
Collaborator

Yeah actually the ffmpeg stuff has been horribly broken anyway, all it could do was record sound clips, the whole thing needs to be fixed.

@orbea
Copy link
Contributor Author

orbea commented Jan 18, 2018

I can't say I used it at all to begin with so I will just use -DENABLE_FFMPEG=OFF for now, but I'll leave this issue open since it seems worth fixing.

@rkitover rkitover added the build label Jan 25, 2018
pvdabeel pushed a commit to pvdabeel/gentoo that referenced this issue May 21, 2019
Doesn't support ffmpeg-4. Portage takes a long time to report the
conflict and the feature is half broken anyway.

Bug: visualboyadvance-m/visualboyadvance-m#179
Signed-off-by: James Le Cuirot <chewi@gentoo.org>
@darktjm
Copy link

darktjm commented May 22, 2019

When I wrote it it worked fine for both video and audio. I don't want to maintain code that actively violates my coding and UI standards, though (among other things). When I was still maintaining it, I paid close attention to the change logs and tried to also make it backwards compatible as much as possible, but all patches to the ffmpeg code dropped that as well. The only reason this even popped up on my radar is that I've stopped caring about my own, working version and just went ahead and installed gentoo's package (but never use; I've pretty much stopped playing emulated games), and noticed that ffmpeg is now masked.

@rkitover
Copy link
Collaborator

@darktjm if you would be so kind as to share your working version with us, we will not ask you to maintain it.

@orbea
Copy link
Contributor Author

orbea commented Oct 5, 2019

It builds now and should work so I am closing this.

@orbea orbea closed this as completed Oct 5, 2019
@rkitover rkitover added the fixed label Oct 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants