bchapuis commented on PR #920: URL: https://github.com/apache/incubator-baremaps/pull/920#issuecomment-2597788704
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, which is released under the LGPL license and which is 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). -- 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]
