Skip to content

fix(generate_licenses): add missing licenses for ffmpeg and openh264#176

Merged
davidliu merged 1 commit intowebrtc-sdk:m125_releasefrom
harshs-dyte:hs/missing-licenses
Jul 25, 2025
Merged

fix(generate_licenses): add missing licenses for ffmpeg and openh264#176
davidliu merged 1 commit intowebrtc-sdk:m125_releasefrom
harshs-dyte:hs/missing-licenses

Conversation

@harshs-dyte
Copy link
Copy Markdown
Contributor

Doing a custom build with the following command line fails the build during the license generation step because these components were added to the build without their licenses being declared here.

python tools_webrtc/ios/build_ios_libs.py --build_config release --arch arm64 x64 --extra-gn-args ios_enable_code_signing=false is_component_build=false rtc_include_tests=false is_debug=false enable_dsyms=false rtc_libvpx_build_vp9=true use_goma=false rtc_enable_symbol_export=true rtc_build_examples=false rtc_use_h264=true use_rtti=true use_custom_libcxx=false

@harshs-dyte
Copy link
Copy Markdown
Contributor Author

harshs-dyte commented Jun 30, 2025

I see there's a patch in the webrtc-build repo adding licenses, this should probably go there instead?

@harshs-dyte
Copy link
Copy Markdown
Contributor Author

Anything else I need to do to allow this to be merged?

@davidliu
Copy link
Copy Markdown
Contributor

@harshs-dyte hey, we just updated to m137 and switched branches. Would it be possible to open the PR against m137_release?

@harshs-dyte
Copy link
Copy Markdown
Contributor Author

@harshs-dyte hey, we just updated to m137 and switched branches. Would it be possible to open the PR against m137_release?

Sure thing.

@davidliu davidliu merged commit db3ba7a into webrtc-sdk:m125_release Jul 25, 2025
@harshs-dyte harshs-dyte deleted the hs/missing-licenses branch July 25, 2025 08:17
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.

3 participants