CalvinKirs commented on PR #920: URL: https://github.com/apache/incubator-baremaps/pull/920#issuecomment-2597839172
> The part of Planetiler integrated into Apache Baremaps is a fast, memory-mapped data structure solely released under the Apache License, which is unrelated to the software mentioned in their NOTICE file. Therefore, I have integrated the reference to Planetiler directly into our main LICENSE file and left the copyright in all the adapted files. I applied the same approach to all the code adapted from third party projects (e.g. Palantir Stream). In the last release, we also added a notice/ and notice-binary/ directory to carry these files in the release.↳ > > I may be wrong, but I think adding appending this NOTICE to our main NOTICE may create confusion. For instance, the Planetile NOTICE file mentions GeoTools (LGPL) and EPSG (proprietary), which are are incompatible with the Apache License. > > https://github.com/apache/incubator-baremaps/blob/c8115159415f72c0e8df222f442f0c09a52387ed/LICENSE#L219 > > We spent a significant amount of time finding alternatives and rewriting code from scratch to completely remove these incompatible dependencies in Apache Baremaps. Leaving these references feels like a footgun and may give people who discover the project the impression that we rely on Apache-incompatible code. I’d be happy to get your opinion on this and of the other mentors in this discussion (cc @julianhyde, @bdelacretaz, @desruisseaux). ``` We should include only this. This product contains code from Planetiler project ## Licenses Planetiler licensed under the Apache license, Version 2.0 Copyright 2021 Michael Barry and Planetiler Contributors. ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
