Skip to content

Conversation

@javiermanzano
Copy link

No description provided.

@javiermanzano javiermanzano mentioned this pull request Apr 13, 2015
Copy link
Contributor

Choose a reason for hiding this comment

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

can you please change this to 22

@hiteshsondhi88
Copy link
Contributor

This PR will affect the current users, what do you think about creating polymorphic methods and leave the current users unaffected ?

@JovinPJ
Copy link

JovinPJ commented Apr 16, 2015

How can i get these changes? i have the same issue with spaces in filepath.
I am using Android Studio, and in 'dependencies' using,
compile 'com.github.hiteshsondhi88.libffmpeg:FFmpegAndroid:0.2.5'

hiteshsondhi88 added a commit that referenced this pull request May 10, 2015
Array of strings on ShellCommand to allow paths with spaces
@hiteshsondhi88 hiteshsondhi88 merged commit 4967cc6 into cropsly:master May 10, 2015
@rafaelekol
Copy link

I also need this commit, can you advice how to import this changes to our project.

@JovinPJ
Copy link

JovinPJ commented May 23, 2015

Thanks hiteshsondhi88, worked for me :)

@ab-helly
Copy link

@JovinPJ how did you get these changes with gradle?

@JovinPJ
Copy link

JovinPJ commented Jun 11, 2015

@ab-helly
please check the link http://hiteshsondhi88.github.io/ffmpeg-android-java/
Actually, what i've done is downloaded the '.aar' and included in my lib folder
and added the dependency like
'compile(name:'FFmpegAndroid', ext:'aar')'.
'FFmpegAndroid' was name of my aar file
There is also option like ,
compile 'com.github.hiteshsondhi88.libffmpeg:FFmpegAndroid:0.2.5' for adding dependency

One drawback ,i noticed is that the apk size become very large(>22mb).

Thanks

@ab-helly
Copy link

@JovinPJ thanks, aar should work for me. 0.2.5 still doesn't contain these changes

@bramgiessen
Copy link

How can I use this commit?

@mattsff
Copy link

mattsff commented Jul 16, 2015

I downloaded aar file and it's not working yet, do you know how can I get it ? Only compiling the fork? Thanks!

@JovinPJ
Copy link

JovinPJ commented Jul 17, 2015

please let me know , what is your issue.

ffmpeg.execute(cmd, new ExecuteBinaryResponseHandler() {

}

here 'cmd' is a string array.

did you able to compile this?

Thanks

@mattsff
Copy link

mattsff commented Jul 17, 2015

I could do it donwloading all project and put it as module. I couldn't through gradle or AAR file.
It works now, thanks !

@kschick
Copy link

kschick commented Aug 27, 2015

@JovinPJ How did you get this change from the aar? The aar linked to from that site is still version 0.2.5.

@silentsilas
Copy link

Yeah, the current .aar in this repository is using an old version. You'll have to pull down the repository and open it in Android Studio, then run the assembleRelease gradle script via the project root's gradle file.

screen shot 2016-03-29 at 3 18 42 pm

If it runs successfully, you'll find the updated .aar at build/outputs/aar/FFmpegAndroid-release.aar

I personally had to undo the String[] changes in this commit and swapped out the FFmpeg binaries with the more recent ones from PR #70

@JovinPJ
Copy link

JovinPJ commented Jul 27, 2016

Hi ,
sorry for the delay, if you give me your mail id i'll send the aar file

@lekhanhduybkhn
Copy link

Hi JovinPJ there is my email [email protected]. Could you send me arr file. so I try download aar and gradle but file name with space is don't work for me

@symphonyrecords
Copy link

WhiteSpace in file path

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.