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

Add Apache-2.0 license and some note to README.md. #9

Merged
merged 4 commits into from
Oct 7, 2021

Conversation

cloudwebrtc
Copy link
Member

@voluntas @davidzhao @hiroshihorie I added a patch for the missing Apache-2.0 License. I have clarified my contribution. If you have any questions, please comment below.

@cloudwebrtc
Copy link
Member Author

Perhaps we should also include an explanation in the binary package that part of the patched code uses the license in this link.
https://github.com/shiguredo-webrtc-build/webrtc-build#%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9

@cloudwebrtc
Copy link
Member Author

cloudwebrtc commented Oct 5, 2021

I think that in addition to declaring the source of these PRs in the patched repo, when releasing the binary, we still need to specify the part authorized by Apache-2.0 in the LICENSE file in the released binary file.

It looks like https://github.com/webrtc-sdk/android/blob/main/Licenses/WEBRTC.md#abseil-cpp

@voluntas Do you have any advice?

@cloudwebrtc cloudwebrtc force-pushed the fix/license-issue-for-shiguredo-patchs branch from b03d89e to b939fc5 Compare October 6, 2021 06:24
README.md Outdated Show resolved Hide resolved
NOTICE Show resolved Hide resolved
@voluntas
Copy link

voluntas commented Oct 6, 2021

I think that in addition to declaring the source of these PRs in the patched repo, when releasing the binary, we still need to specify the part authorized by Apache-2.0 in the LICENSE file in the released binary file.

This recognition is correct.

@voluntas
Copy link

voluntas commented Oct 6, 2021

For clarity, we have prepared a NOTICE file. It is a good idea to explicitly post the copyright written here.

https://github.com/shiguredo-webrtc-build/webrtc-build/blob/master/NOTICE

Copy link
Member

@davidzhao davidzhao left a comment

Choose a reason for hiding this comment

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

lgtm

README.md Outdated Show resolved Hide resolved
@cloudwebrtc cloudwebrtc force-pushed the fix/license-issue-for-shiguredo-patchs branch from 412cf6c to 74cdceb Compare October 7, 2021 01:19
@cloudwebrtc cloudwebrtc force-pushed the fix/license-issue-for-shiguredo-patchs branch from 74cdceb to df4cc8a Compare October 7, 2021 01:24
@cloudwebrtc
Copy link
Member Author

Okay, modified, please check the latest commit.

@voluntas
Copy link

voluntas commented Oct 7, 2021

LGTM

@cloudwebrtc cloudwebrtc merged commit 5d2d3c8 into main Oct 7, 2021
@cloudwebrtc cloudwebrtc deleted the fix/license-issue-for-shiguredo-patchs branch October 7, 2021 01:47
davidliu pushed a commit to davidliu/webrtc that referenced this pull request Feb 8, 2022
* Add Apache-2.0 license and some notes to README.md.
* Add user link.
* update.
davidliu pushed a commit to davidliu/webrtc that referenced this pull request Feb 9, 2022
* Add Apache-2.0 license and some notes to README.md.
* Add user link.
* update.
davidliu pushed a commit that referenced this pull request Feb 10, 2022
* Add Apache-2.0 license and some notes to README.md.
* Add user link.
* update.
davidliu pushed a commit that referenced this pull request Jul 17, 2022
* Add Apache-2.0 license and some notes to README.md.
* Add user link.
* update.
davidliu pushed a commit that referenced this pull request Jul 21, 2022
* Add Apache-2.0 license and some notes to README.md.
* Add user link.
* update.
cloudwebrtc added a commit that referenced this pull request Jan 18, 2023
* Add Apache-2.0 license and some notes to README.md.
* Add user link.
* update.
cloudwebrtc added a commit that referenced this pull request Jun 5, 2023
* Add Apache-2.0 license and some notes to README.md.
* Add user link.
* update.
cloudwebrtc added a commit that referenced this pull request Jun 12, 2023
* Add Apache-2.0 license and some notes to README.md.
* Add user link.
* update.

Updated readme detailing changes from original (#42)

Adding membrane framework (#51)

Co-authored-by: David Zhao <[email protected]>
@cloudwebrtc cloudwebrtc mentioned this pull request Jun 12, 2023
cloudwebrtc added a commit that referenced this pull request Jul 13, 2023
* Add Apache-2.0 license and some notes to README.md.
* Add user link.
* update.

Updated readme detailing changes from original (#42)

Adding membrane framework (#51)

Co-authored-by: David Zhao <[email protected]>
cloudwebrtc added a commit that referenced this pull request May 20, 2024
* Add Apache-2.0 license and some notes to README.md.
* Add user link.
* update.

Updated readme detailing changes from original (#42)

Adding membrane framework (#51)

Co-authored-by: David Zhao <[email protected]>

Updated readme (#83)
@cloudwebrtc cloudwebrtc mentioned this pull request May 21, 2024
cloudwebrtc added a commit that referenced this pull request Jun 12, 2024
Use M125 as the latest version and migrate historical patches to m125

Patches Group:

## 1. Update README.md
b6c65fc
* Add Apache-2.0 license and some note to README.md. (#9)
* Updated readme detailing changes from original (#42)
* Adding membrane framework (#51)
* Updated readme (#83)

## 2. Audio Device Optimization
7454824
* allow listen-only mode in AudioUnit, adjust when category changes
(#2)
* release mic when category changes
(#5)
* Change defaults to iOS defaults
(#7)
* Sync audio session config
(#8)
* feat: support bypass voice processing for iOS.
(#15)
* Remove MacBookPro audio pan right code
(#22)
* fix: Fix can't open mic alone when built-in AEC is enabled.
(#29)
* feat: add audio device changes detect for windows.
(#41)
* fix Linux compile (#47)
* AudioUnit: Don't rely on category switch for mic indicator to turn off
(#52)
* Stop recording on mute (turn off mic indicator)
(#55)
* Cherry pick audio selection from m97 release
(#35)
* [Mac] Allow audio device selection
(#21)
* RTCAudioDeviceModule.outputDevice / inputDevice getter and setter
(#80)
* Allow custom audio processing by exposing AudioProcessingModule
(#85)
* Expose audio sample buffers for Android
(#89)
* feat: add external audio processor for android.
(#103)
* android: make audio output attributes modifiable
(#118)
* Fix external audio processor sample rate calculation
(#108)
* Expose remote audio sample buffers on RTCAudioTrack
(#84)
* Fix memory leak when creating audio CMSampleBuffer
#86

## 3. Simulcast/SVC support for iOS/Android.
b0b9fe9
    
- Simulcast support for iOS SDK (#4)
- Support for simulcast in Android SDK (#3)
- include simulcast headers for mac also (#10)
- Fix simulcast using hardware encoder on Android (#48)
- Add scalabilityMode support for AV1/VP9. (#90)

## 4. Android improvements.
9aaaab5
- Start/Stop receiving stream method for VideoTrack (#25)
- Properly remove observer upon deconstruction (#26)
- feat: Expose setCodecPreferences/getCapabilities for android. (#61)
- fix: add WrappedVideoDecoderFactory.java. (#74)

## 5. Darwin improvements
a13ea17
- [Mac/iOS] feat: Add RTCYUVHelper for darwin. (#28)
- Cross-platform `RTCMTLVideoView` for both iOS / macOS (#40)
- rotationOverride should not be assign (#44)
- [ObjC] Expose properties / methods required for AV1 codec support
(#60)
- Workaround: Render PixelBuffer in RTCMTLVideoView (#58)
- Improve iOS/macOS H264 encoder (#70)
- fix: fix video encoder not resuming correctly upon foregrounding
(#75).
- add PrivacyInfo.xcprivacy to darwin frameworks. (#112)
- Add NSPrivacyCollectedDataTypes key to xcprivacy file (#114)
- Thread-safe `RTCInitFieldTrialDictionary` (#116)
- Set RTCCameraVideoCapturer initial zoom factor (#121)
- Unlock configuration before starting capture session (#122)

## 6. Desktop Capture for macOS.
841d78f
- [Mac] feat: Support screen capture for macOS. (#24) (#36)
- fix: Get thumbnails asynchronously. (#37)
- fix: Use CVPixelBuffer to build DesktopCapture Frame, fix the crash
caused by non-CVPixelBuffer frame in RTCVideoEncoderH264 that cannot be
cropped. (#63)
- Fix the crash when setting the fps of the virtual camera. (#62)

## 7. Frame Cryptor Support.
fc08745
- feat: Frame Cryptor (aes gcm/cbc). (#54)
- feat: key ratchet/derive. (#66)
- fix: skip invalid key when decryption failed. (#81)
- Improve e2ee, add setSharedKey to KeyProvider. (#88)
- add failure tolerance for framecryptor. (#91)
- fix h264 freeze. (#93)
- Fix/send frame cryptor events from signaling thread (#95)
- more improvements for E2EE. (#96)
- remove too verbose logs (#107)
- Add key ring size to keyProviderOptions. (#109)

## 8. Other improvements.
eed6c8a
- Added yuv_helper (#57)
- ABGRToI420, ARGBToI420 & ARGBToRGB24 (#65)
- more yuv wrappers (#87)
- Fix naming for yuv helper (#113)
- Fix missing `RTC_OBJC_TYPE` macros (#100)

---------

Co-authored-by: Hiroshi Horie <[email protected]>
Co-authored-by: David Zhao <[email protected]>
Co-authored-by: davidliu <[email protected]>
Co-authored-by: Angelika Serwa <[email protected]>
Co-authored-by: Théo Monnom <[email protected]>
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