Skip to content
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

Merged
merged 6 commits into from
Sep 19, 2024
Merged

Conversation

dierberg
Copy link
Contributor

@dierberg dierberg commented Sep 9, 2024

No description provided.

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>
Copy link
Contributor

@OliverFendt OliverFendt left a 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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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

Copy link
Contributor

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

Copy link
Contributor

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

Copy link
Contributor

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

Copy link
Contributor

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>
Copy link
Contributor

@OliverFendt OliverFendt left a 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
Copy link
Contributor

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
Copy link
Contributor

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-
Copy link
Contributor

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.

Copy link
Contributor

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

Copy link
Contributor

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

Copy link
Contributor

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

Copy link
Contributor

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>
Copy link
Contributor

@OliverFendt OliverFendt left a 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

@OliverFendt OliverFendt merged commit 83ed3e8 into Open-Source-Compliance:main Sep 19, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants