-
-
Notifications
You must be signed in to change notification settings - Fork 425
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
Support for third_party_licenses files from Google Play Services libraries #824
Comments
@arekolek thank you for the report. I've had a look into the format they use in the past, and we may not benefit from the complexity of supporting it. It is possibly worth to identify which additional libraries were found, and why the |
Thanks for the reply I'm pretty sure of the 17 libraries I mentioned, none was in the According to google/play-services-plugins#217 (comment) they look only at I linked above to the part of their plugin code where they handle the latter: https://github.com/google/play-services-plugins/blob/45ec9c690672f0ff6f86e6221ef64e204b10a228/oss-licenses-plugin/src/main/groovy/com/google/android/gms/oss/licenses/plugin/LicensesTask.groovy#L168 The good thing is that it seems that they follow the same convention in all their libraries, so there shouldn't be a need to handle each Google library separately. |
Also to make it clear: I wouldn't want to include the
|
Thank you for the additional references. I will need to spend some more time to get a better overview of what the effort may be to add support to handle this. Currently I can't give a clear answer if this is something we'll be able to offer or when we may be able to offer it. Given your research on the matter already, possibly this is something you'd like to contribute? |
I started working on something that would let me add these as described in https://github.com/mikepenz/AboutLibraries#libraries just so I don't lose these licenses (for the most part it was a rewrite of part of the Later I might be able to check out how this plugin works and try to integrate that into it, I'll post here if I do I see two issues already:
|
I've noticed the similar issue some time ago. Namely, okhttp depends on the Public Suffix List, which itself is licensed under MPL but is not listed in POM. It is even not a jvm library. The license is distributed by including the txt file (by default it is present in the the root dir inside APK/AAB). More info: #502 (and resources linked there). |
To follow up on this ticket. Did you end up writing a new plugin for the oss handling? IS there something to look at which we may be able to integrate back in? Thanks a lot |
I have this gradle plugin in my I hoped to just create a PR to this repo, but didn't have the opportunity to look more into how it works. So I'll ask if I can just share my current plugin with you so you can have a look |
Yes please @arekolek, can't promise anything. However I'd like to have a look and see what I can do. |
@mikepenz you can see the code here https://github.com/arekolek/third-party-licenses Note there's also a separate branch, with a proof-of-concept for detecting license names |
Thank you very much. It will probably take me a bit to get to it. Really appreciate it! |
About this issue
Can this plugin also include libraries depended upon by the Google Play services libraries, like the Google
oss-licenses-plugin
does?In my case, compared to
AboutLibraries
,oss-licenses-plugin
included additional 17 libraries because of Google Play services, and I'd prefer not to have to maintain it semi-manually and have to keep usingoss-licenses-plugin
to extract these licenses.From what I was able to find out,
.aar
files for Google libraries includethird_party_licenses.json
andthird_party_licenses.txt
files that are parsed to extract these libraries and their licenses. Unfortunately in this case there is no "artifact ID" that would correspond to theuniqueId
from AboutLibraries, only the name.As an example of how this looks in the
.aar
file: https://stackoverflow.com/a/74617118/1916449Details
Checklist
The text was updated successfully, but these errors were encountered: