busbey commented on a change in pull request #3836:
URL: https://github.com/apache/hbase/pull/3836#discussion_r746919206
##########
File path: hbase-resource-bundle/src/main/resources/supplemental-models.xml
##########
@@ -2251,4 +2251,22 @@ Copyright (c) 2007-2017 The JRuby project
</licenses>
</project>
</supplement>
+ <supplement>
+ <project> <!-- v3.4.0 comes in via HADOOP-17317 -->
+ <groupId>dnsjava</groupId>
+ <artifactId>dnsjava</artifactId>
+ <version>3.4.0</version>
+ <name>dnsjava</name>
+ <licenses>
+ <license>
+ <name>BSD 2-Clause License</name>
+ <url>https://opensource.org/licenses/BSD-2-Clause</url>
+ <distribution>repo</distribution>
+ <comments>
+ Copyright (c) 1998-2011, Brian Wellington.
+ </comments>
Review comment:
the BSD 2 Clause license is not in the set of licenses where we
aggregate all the dependencies provided with a single copy of the license text.
(something that happens in
hbase-resource-bundle/src/main/resources/META-INF/LICENSE.vm )
in such cases, we rely on the `comments` field here to provide the entirety
of what we'll put into the generated LICENSE file. so this needs to include
both this copyright statement and the complete text of the BSD 2 clause.
essentially everything from the LICENSE file on the dependency:
https://github.com/dnsjava/dnsjava/blob/v3.4.0/LICENSE
it's the same kind of thing we had to do for jamon-runtime for the MPL.
--
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]