-
Notifications
You must be signed in to change notification settings - Fork 20
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
FOSSC-152 tcpdump #159
FOSSC-152 tcpdump #159
Conversation
Signed-off-by: Sascha Dierberg <dierberg@dresearch-fe.de>
Signed-off-by: Sascha Dierberg <dierberg@dresearch-fe.de>
Signed-off-by: Sascha Dierberg <dierberg@dresearch-fe.de>
Signed-off-by: Sascha Dierberg <dierberg@dresearch-fe.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome @dierberg, I only have some comments
FileChecksum: SHA1: b5c3b35efe65b923279ceafe7b7661ba58e2bee7 | ||
FileChecksum: SHA256: 325aea7021c5af417af18d5668a31803a2975a44ac1498cffa067ab643d7784e | ||
FileChecksum: MD5: a0108e35191e0b2fa6b6ecfe3717e3e8 | ||
LicenseConcluded: LicenseRef-BSD-4-Clause-7a1f5e62d532027306e027c62e7da7dc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The license conclusion is correct, the license requires an acknowledgment:
This product includes software developed by the Kungliga Tekniska Högskolan and its contributors." please add this in the acknowledgement section, which will then be the FileAttribution.
The acknowledgments requirements regarding the Berkeley University are irrelevant
FileChecksum: SHA1: 58e8545fda3963b4a1995daee48364592991a31b | ||
FileChecksum: SHA256: 0777c56ecd5d7117c5e886104f7c2e58679f8e5767fb39373895efb28c114f84 | ||
FileChecksum: MD5: 42642a3baf42ad21986a88a03eb7215c | ||
LicenseConcluded: LicenseRef-BSD-4-Clause-7a1f5e62d532027306e027c62e7da7dc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see my comment in addrtostr.c
FileChecksum: SHA1: 5dddf7fecbf83050adcc9899eb5aa00ae477de34 | ||
FileChecksum: SHA256: 22cd6f9a00e1b8cf7c19c8c609cd16b69d7c78f125b10e7759c55989a2f099be | ||
FileChecksum: MD5: 3ef1a80472a553781a0016e6fc8517dc | ||
LicenseConcluded: LicenseRef-BSD-4-Clause-b0908a45eefa29a3686c0b8cd7811de4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The conclusion is correct, the license requires an acknowledgment:
This product includes software developed by the NetBSD Foundation, Inc. and its contributors.
Please add this in the acknowledgment section
FileChecksum: SHA1: 54fc0c0b58299a154002e1e633ea04921b1b5baf | ||
FileChecksum: SHA256: d61917ee173bb13e1dc285ac2c6fb40f2f4e143331c7b25e3bbb7484eaaeb131 | ||
FileChecksum: MD5: 0ce98659b95b4f006f08d9ab09cf5d44 | ||
LicenseConcluded: LicenseRef-BSD-4-Clause-15a6ffa42c54da1397de5147c310520f AND LicenseRef-BSD-4-Clause-Shortened-e96ddab5f8fffe47de17b5812d71176a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The conclusion is correct, the BSD-4-Clause instance requires an acknowledgment, please add it:
This product includes software developed by Yen Yen Lim and North Dakota State University
FileChecksum: SHA1: 1424350f9cb4eeef42da0b897747491b20e35b1c | ||
FileChecksum: SHA256: c5a8d6de936589748a181111b514bd4393ce7ec647df1fc15941ffacf0b66bf8 | ||
FileChecksum: MD5: b261dff3f15cf19a89b8d3443ef6bd2b | ||
LicenseConcluded: LicenseRef-BSD-4-Clause-Shortened-e96ddab5f8fffe47de17b5812d71176a AND LicenseRef-Permission-Notice-3a2bf8a18b88925f22a9130d4cf995f0 AND LicenseRef-warranty-disclaimer-88394e5be8e0e24e8312d81013827b8f |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the LicenseRef-Permission-Notice-3a2bf8a18b88925f22a9130d4cf995f0 AND LicenseRef-warranty-disclaimer-88394e5be8e0e24e8312d81013827b8f is a dual license statement:
Permission to use, copy, and modify this software with or without fee
is hereby granted, provided that this entire notice is included in
all copies of any software which is or includes a copy or
modification of this software.
You may use this code under the GNU public license if you so wish. Please
contribute changes back to the authors under this freer than GPL license
so that we may further the use of strong encryption without limitations to
all.
THIS SOFTWARE IS BEING PROVIDED "AS IS", WITHOUT ANY EXPRESS OR
IMPLIED WARRANTY. IN PARTICULAR, NONE OF THE AUTHORS MAKES ANY
REPRESENTATION OR WARRANTY OF ANY KIND CONCERNING THE
MERCHANTABILITY OF THIS SOFTWARE OR ITS FITNESS FOR ANY PARTICULAR
PURPOSE.
The overall license is
LicenseRef-BSD-4-Clause-Shortened-e96ddab5f8fffe47de17b5812d71176a AND (GPL-1.0-or-later OR Permission-Notice)
FileChecksum: SHA1: 2c82d77ca8631e9fc51f59f06715980aafed999e | ||
FileChecksum: SHA256: b6752c230fcc29bb30c785f5dddd6e881517695c82953ad82e7af3cd2bec3dba | ||
FileChecksum: MD5: fa8e7659ce06db01279bc69a53eb3340 | ||
LicenseConcluded: LicenseRef-BSD-3-Clause-499bc2ee779fcca8f326572ae23b93fd OR LicenseRef-GPL-1.0-only |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the conclusion in this case is GPL-1.0-or-later not "-only" because the GPL says if no version number of GPL is specified the user is free to choose one among the official released GPL versions, due to this we always conclude GPL-1.0-or-later in all such cases
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check my comments in the tag value file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check my comments in the tag value file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check my comments in the tag value file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check my comments in the tag value file
Signed-off-by: Sascha Dierberg <sascha.dierberg@iris-sensing.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, there are only some minor remaining comments
FileChecksum: SHA1: 83e58b6d926f9f383dc7d62d14102a7c5e97ac34 | ||
FileChecksum: SHA256: 79e44b230637814b30f6046e8683a377984855b6682d4610783ad2aa3ab87ce2 | ||
FileChecksum: MD5: b55c318c4d11e665f0386349edc07f5c | ||
LicenseConcluded: LicenseRef-BSD-3-Clause |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The conclusion is not fully correct. The information in the file is:
This is based on code in the CMake 3.12.4 FindPkgConfig.cmake,
which is "Distributed under the OSI-approved BSD 3-Clause License."
when you look at:
https://github.com/Kitware/CMake/blob/master/Modules/FindPkgConfig.cmake
you see the following:
Distributed under the OSI-approved BSD 3-Clause License. See accompanying
file Copyright.txt or https://cmake.org/licensing for details.
if you then surf to the given link you will be displayed with:
CMake is distributed under the OSI-approved BSD 3-clause License. Please see its Copyright.txt for details. The CMake source tree also contains some third-party components with compatible licenses.
Copyright.txt is a link, when you click on it you will see:
https://gitlab.kitware.com/cmake/cmake/raw/master/Copyright.txt
Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions
are met:
-
Redistributions of source code must retain the above copyright
notice, this list of conditions and the following disclaimer. -
Redistributions in binary form must reproduce the above copyright
notice, this list of conditions and the following disclaimer in the
documentation and/or other materials provided with the distribution. -
Neither the name of Kitware, Inc. nor the names of Contributors
may be used to endorse or promote products derived from this
software without specific prior written permission.
THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
"AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
(INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
Please conclude this license text for the file and add in the license comment the search, including the date 16th of Sept 2024
@@ -1239,6 +1229,13 @@ FOR A PARTICULAR PURPOSE. | |||
|
|||
Dual-license | |||
|
|||
License LicenseRef-BSD-4-Clause-Shortened-e96ddab5f8fffe47de17b5812d71176a AND |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please change this to:
License: GPL-1.0-or-later OR Permission-Notice
The BSD-4-Clause is not to be considered for the dual licensing, nevertheless the overall license of the file is
BSD-4-Clause-Shortened-e96ddab5f8fffe47de17b5812d71176a AND (GPL-1.0-or-later OR Permission-Notice)
|
||
LicenseRef-Dual-license | ||
|
||
To the extent that files may be licensed under either LicenseRef-BSD-4-Clause- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change this to:
To the extent that files may be licensed under either GPL-1.0-or-later OR Permission-Notice, in this context,
Permission-Notice has been chosen. This choice shall not restrict the freedom of other users to select either
GPL-1.0-or-later OR Permission-Notice licenses. For convenience, all license texts are provided.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please have a look at my comments in the Disclosure file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please have a look at my comments in the Disclosure file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please have a look at my comments in the Disclosure file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please have a look at my comments in the Disclosure file
Signed-off-by: Sascha Dierberg <sascha.dierberg@iris-sensing.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, thank you very much @dierberg
No description provided.