-
Notifications
You must be signed in to change notification settings - Fork 724
Conform BSD-2-Clause and BSD-3-Clause text to SPDX #9813
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
Conversation
|
Two considerations:
|
ulysses4ever
left a comment
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.
Perfetto, grazie!
glyn
left a comment
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.
Thanks for this. LGTM.
|
Merge-me? |
`cabal init` text for BSD-2-Clause and BSD-3-Clause licence differed slightly from the one published at SPDX. [1] [2] This caused some problems to users when dealing with licence-recognition software. [3] [1] https://spdx.org/licenses/BSD-2-Clause.html [2] https://spdx.org/licenses/BSD-3-Clause.html [3] https://discourse.haskell.org/t/non-standard-license-generated-by-stack-new/9059
b732fa8 to
1e86730
Compare
|
Confirmed working on Windows. There are however some very small nitpicks:
|
|
Thanks, it makes sense, I will make a follow up patch to this. |
|
I wonder about "automated tests not needed", given that it was possible for the text to deviate from what was required. |
|
@mergify backport 3.12 |
✅ Backports have been created
|
|
(I will address both of your comments when I have time, if you don’t do it before me!) |
Conform BSD-2-Clause and BSD-3-Clause text to SPDX (backport #9813)
cabal inittext for BSD-2-Clause and BSD-3-Clause licence differed slightly from the one published at SPDX. [1] [2]This caused some problems to users when dealing with licence-recognition software. [3]
[1] https://spdx.org/licenses/BSD-2-Clause.html
[2] https://spdx.org/licenses/BSD-3-Clause.html
[3] https://discourse.haskell.org/t/non-standard-license-generated-by-stack-new/9059
Include the following checklist in your PR:
Tests have been added. (Ask for help if you don’t know how to write them! Ask for an exemption if tests are too complex for too little coverage!)automated tests not neededQA Notes
cabal initand a series of ⏎ (it should select a BSD licence)LICENSEfile does not contain the sentence “All rights reserved.”.