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

LicenseCompareHelper: BSD-2-Clause license text not recognised #214

Closed
pmonks opened this issue Nov 2, 2023 · 2 comments · Fixed by #221
Closed

LicenseCompareHelper: BSD-2-Clause license text not recognised #214

pmonks opened this issue Nov 2, 2023 · 2 comments · Fixed by #221

Comments

@pmonks
Copy link
Collaborator

pmonks commented Nov 2, 2023

The following BSD-2-Clause license text is not being recognised by org.spdx.utility.compare.LicenseCompareHelper.isStandardLicenseWithinText().

Note: tested with Spdx-Java-Library v1.1.8.

BSD 2-Clause License

Copyright (c) 2023, Peter Monks

Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions are met:

1. Redistributions of source code must retain the above copyright notice, this
   list of conditions and the following disclaimer.

2. 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.

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.

Note: I have a sneaking suspicion (without any evidence) that this issue, along with #213, #184, and #182 might all be symptoms of the same issue with template-based matching. I'm still capturing more examples as I find them though, as that gives us more test cases to make sure it's fully fixed.

@goneall
Copy link
Member

goneall commented Nov 2, 2023

@pmonks thanks for collecting the issues. I'm going to be traveling for 3 weeks, so it may be a while before I can research the issues.

@pmonks
Copy link
Collaborator Author

pmonks commented Nov 2, 2023

Absolutely no problem! I may be able to take a closer look in the coming weeks myself (not that I really know that code very well yet) - I'm just wrapping up a decent chunk of work, so it's a good time to start looking at something else.

goneall added a commit that referenced this issue Dec 6, 2023
Fixes #214
Fixes #213

Signed-off-by: Gary O'Neall <gary@sourceauditor.com>
goneall added a commit that referenced this issue Dec 8, 2023
Fixes #214
Fixes #213

Signed-off-by: Gary O'Neall <gary@sourceauditor.com>
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 a pull request may close this issue.

2 participants