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

feat(android): allow building exoplayer from source #3932

Merged

Conversation

freeboub
Copy link
Collaborator

Summary

Add configuration and documentation to be able to build from media3 / Exoplayer source code

Motivation

I had to do a small workaround in exoplayer source code to develop a new feature/poc

Changes

  • Add configuration to build from source
  • document the feature
  • implement it in the sample. It is not enabled by default, but can be activated with only 2 gradle script uncommenting

Test plan

Easy to test by change the sample, @KrzysztofMoch do you think I need to enable 1 dedicated build from source in the github CI ? My issue with that is that sample will need to clone media3 source code, it will highly increase folder size for developement environment ...

freeboub and others added 27 commits May 5, 2024 21:06
…ideo

# Conflicts:
#	android/src/main/java/com/brentvatne/exoplayer/ReactExoplayerViewManager.java
#	src/Video.tsx
…fix/avoidVideoResizingFlickering

# Conflicts:
#	.github/ISSUE_TEMPLATE/bug-report.yml
* fix: ensure player doesn't start when view is unmounted
@freeboub
Copy link
Collaborator Author

@KrzysztofMoch This one is ready for review !

@KrzysztofMoch KrzysztofMoch changed the title Feat/allow building from exoplayer source feat(android): allow building exoplayer from source Jun 25, 2024
Copy link
Collaborator

@KrzysztofMoch KrzysztofMoch left a comment

Choose a reason for hiding this comment

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

Looks good to me, left one small comment

examples/basic/android/app/build.gradle Show resolved Hide resolved
examples/basic/android/app/build.gradle Show resolved Hide resolved
@freeboub
Copy link
Collaborator Author

Looks good to me, left one small comment

Great, thank you, so let's merge this one ! 👍

@freeboub freeboub merged commit a7d834a into TheWidlarzGroup:master Jun 25, 2024
4 checks passed
@YangJonghun
Copy link
Collaborator

@freeboub
Looks great to me! sorry for the late confirmation.
By the way, do you have any plans to bump up a media3 version?

@freeboub
Copy link
Collaborator Author

For media3 version, I tried few month ago, I had some issue regarding to gradle plugin. But you can force the version in the gradle file with following value. The react native version used in sample was not compatible with media3...

RNVideo_media3Version=1.1.1

Notice also that in my thumbnail poc, I use master branch of media3 so version 1.3.1 shall work (no blocking point on API change).

@YangJonghun
Copy link
Collaborator

@freeboub
I asked because it was a target version issue and I wanted to insert the add-on from 1.2.0 if the library raised the default.

@freeboub
Copy link
Collaborator Author

@YangJonghun I am not sure to understand exactly what you want do ... Upgrading locally to 1.2.0 doesn't ?

@YangJonghun
Copy link
Collaborator

YangJonghun commented Jun 27, 2024

@freeboub
I'd like to contribute for add CMCD prop.
However, this function require 1.2.0 or later version of media3, which is difficult to do in the current default version.

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

3 participants