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

Make FFmpeg working on Android platform #1672

Closed
wants to merge 4 commits into from
Closed

Make FFmpeg working on Android platform #1672

wants to merge 4 commits into from

Conversation

cv47
Copy link
Contributor

@cv47 cv47 commented May 5, 2013

Require @oioitff media branch
Prepare for @oioitff media branch merge to master.

@raven02
Copy link
Contributor

raven02 commented May 5, 2013

@oioitff , for windows , are you linking to lib*.a or *.lib ? I can compile and link but it crash when loading game .

@oioitff
Copy link
Contributor

oioitff commented May 5, 2013

@raven02 Those should be both OK.
Do you make a release build? Maybe you need to set

  /OPT:NOREF

for Linker in Visual Studio, otherwise it would crash.

@hrydgard
Copy link
Owner

hrydgard commented May 5, 2013

I will test and probably merge this tonight, no time right now.

@raven02
Copy link
Contributor

raven02 commented May 5, 2013

Yes , i'm making a release win32 build with VC+ 2010 express.Wondering where to set this /OPT:NOREF option :)

@raven02
Copy link
Contributor

raven02 commented May 5, 2013

@cv47 , nice to see it . btw , i think we don't need this as well

$(SRC)/Core/HW/audioPlayer.cpp \
$(SRC)/Core/HW/mediaPlayer.cpp \

@cv47
Copy link
Contributor Author

cv47 commented May 5, 2013

@raven02 How to play the video without mediaPlayer.cpp?
Compile failed without mediaPlayer.cpp

@raven02
Copy link
Contributor

raven02 commented May 5, 2013

I think they are targeting to merge atrac3 audio first since CG/sceMpeg one is still somehow depending on dshow and at3tool . Therefore , we'll keep using the original sceMpeg.cpp without those USE_FFMPEG/USE_DSHOW at this moment.

Once your android static lib commited then atrac3 audio should start working for Andriod build .

@cv47
Copy link
Contributor Author

cv47 commented May 5, 2013

@raven02 Okay, I got it

atrac3 audio should start working for Andriod build
@raven02
Copy link
Contributor

raven02 commented May 5, 2013

@cv47 .Thanks

@hrydgard
Copy link
Owner

hrydgard commented May 5, 2013

I tried to run ./android-build.sh , and after setting up the paths, it just gets stuck on the configure stage. It sits there with a perl.exe going 100% on a core forever. I'm using mingw shell. Has anyone seen this?

@hrydgard
Copy link
Owner

hrydgard commented May 5, 2013

Installed some more mingw stuff, now it proceeds but ends with the following:

License: LGPL version 2.1 or later
Creating config.mak and config.h...
libavutil/avconfig.h is unchanged

WARNING: /c/AndroidNDK/toolchains/arm-linux-androideabi-4.4.3/prebuilt/windows-x86_64/bin/arm-linux-androideabi-pkg-config not found, library detection may fail.
WARNING: Compiler does not indicate floating-point ABI, guessing soft.
library.mak:107: *** missing separator.  Stop.
library.mak:107: *** missing separator.  Stop.

Strange..

@hrydgard
Copy link
Owner

hrydgard commented May 5, 2013

OK I've figured it out, unknown pointed me to https://ffmpeg.org/trac/ffmpeg/ticket/1209

Sorry for the delay but I want to try to shrink the library by removing codecs we don't need before I merge this. I will do that tomorrow night.

@hrydgard
Copy link
Owner

hrydgard commented May 6, 2013

Ok so I didn't have time, tomorrow it is :P

@cv47
Copy link
Contributor Author

cv47 commented May 7, 2013

@hrydgard I got compile error with current FFmepg module, but compile success on original FFmpeg 1.2 library.....strange

@hrydgard
Copy link
Owner

hrydgard commented May 8, 2013

I have fixed this up to work great now. However the .a files are still too big to check in (which we need for the buildbot) so I will work a little on shrinking them before merging this.

Will be done soon.

@hrydgard
Copy link
Owner

hrydgard commented May 8, 2013

I've merged this now!

@hrydgard hrydgard closed this May 8, 2013
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

4 participants