play-services-plugins icon indicating copy to clipboard operation
play-services-plugins copied to clipboard

Support third party licenses

Open joostfunkekupper opened this issue 3 years ago • 24 comments

This PR is based on the open #143 which hasn't been updated in a while. I've also included some instructions on how to manually add third party licenses that were not picked up through Maven.

joostfunkekupper avatar Jun 24 '21 04:06 joostfunkekupper

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Jun 24 '21 04:06 google-cla[bot]

@googlebot I signed it!

joostfunkekupper avatar Jun 24 '21 04:06 joostfunkekupper

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Jun 24 '21 04:06 google-cla[bot]

Hi @ganadist, would you be interested in approving the use of your commits? I'm really hoping to get your changes approved and merged, since it's been very useful in our production apps.

joostfunkekupper avatar Jun 24 '21 05:06 joostfunkekupper

Any chance that you can add the dependency management of the play services like in LicensesTask.addGooglePlayServiceLicenses(...)?

rekire avatar Jun 26 '21 20:06 rekire

Any chance that you can add the dependency management of the play services like in LicensesTask.addGooglePlayServiceLicenses(...)?

Could you elaborate? The Google Play Services licenses are already being added by this library.

joostfunkekupper avatar Jun 30 '21 22:06 joostfunkekupper

I mean that any dependency can use that way to add own licenses with those two files. It should be simple to check if those two files exists in any dependency and not exclusive for the play services dependencies.

rekire avatar Jul 01 '21 05:07 rekire

By the way I found a better way for what I'm aiming I created the PR #191 that will take the name directly from the POM file. Never mind my request. :-)

rekire avatar Jul 03 '21 13:07 rekire

By the way I found a better way for what I'm aiming I created the PR #191 that will take the name directly from the POM file. Never mind my request. :-)

Yeah right, nice.

joostfunkekupper avatar Jul 06 '21 04:07 joostfunkekupper

@joostfunkekupper you need to update/rebase your PR. Mine was merged in the meantime. You might should add also one or more tests to make sure that everything works as expected.

rekire avatar Aug 27 '21 08:08 rekire

Thanks @rekire for letting me know. I'll try and get onto that asap.

joostfunkekupper avatar Aug 31 '21 10:08 joostfunkekupper

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Sep 07 '21 02:09 google-cla[bot]

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Oct 06 '21 04:10 google-cla[bot]

Hello @ganadist. Sorry to bug you again about this, but could you reply with @googlebot I consent. to this PR so that the google-cla bot can approve your commits to be part of this PR.

joostfunkekupper avatar Oct 06 '21 04:10 joostfunkekupper

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Oct 06 '21 04:10 google-cla[bot]

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

information_source Googlers: Go here for more info.

@googlebot I consent.

ganadist avatar Oct 06 '21 04:10 ganadist

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Oct 06 '21 04:10 google-cla[bot]

Sorry @ganadist, your reply should only include @googlebot I consent. and nothing else :) The bot isn't very smart

joostfunkekupper avatar Oct 06 '21 05:10 joostfunkekupper

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Oct 06 '21 05:10 google-cla[bot]

@googlebot I consent.

ganadist avatar Oct 06 '21 05:10 ganadist

Hi @caller9, this PR is ready to be reviewed. Documentation and test case are included.

joostfunkekupper avatar Oct 06 '21 07:10 joostfunkekupper

Hi @caller9, any update on this PR? I would really appreciate if the changes from @joostfunkekupper and @ganadist get merged soon. Thanks!

cc @PaulRashidi

juliankotrba avatar Dec 14 '21 20:12 juliankotrba

Hey guys, thanks for this important feature. I would really appreciate a release as https://github.com/cookpad/LicenseToolsPlugin was deprecated in favor of this license plugin. It's open for quite some time and licenses are a legal requirement. So I think it would help a lot of people to provide this feature soon because alternatives are deprecated. Especially hybrid apps use licenses outside the normal scope and need to show them, too.

mobilemarines avatar Feb 09 '23 08:02 mobilemarines

Hey guys, thanks for this important feature. I would really appreciate a release as https://github.com/cookpad/LicenseToolsPlugin was deprecated in favor of this license plugin. It's open for quite some time and licenses are a legal requirement. So I think it would help a lot of people to provide this feature soon because alternatives are deprecated. Especially hybrid apps use licenses outside the normal scope and need to show them, too.

@mobilemarines I recommend using AboutLibraries. It's fully Open Source (unlike this plugin), it's actively maintained, and provides the exact same functionality as the play services plugin.

wseemann avatar Feb 14 '23 05:02 wseemann