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

NuGet: Use license expression #36421

Merged
merged 8 commits into from
Sep 27, 2022
Merged

NuGet: Use license expression #36421

merged 8 commits into from
Sep 27, 2022

Conversation

vanillajonathan
Copy link
Contributor

Use a SPDX license expression to specify MIT instead of pointing to the LICENSE file.

Reference: https://docs.microsoft.com/en-us/nuget/reference/nuspec#license

@julien-deramond
Copy link
Member

julien-deramond commented May 23, 2022

Thanks for the PR. Got a few questions here.

Atm when we go to https://www.nuget.org/packages/bootstrap/ we have a "License info" link.

I suppose that with this PR we would have we would have the equivalent of what we can see in https://www.nuget.org/packages/Newtonsoft.Json/13.0.2-beta1 with a "MIT license". If we click on this MIT link we can see that some information are missing.

Wouldn't be the same with your proposal or does it use automatically IDK <authors> and/or <copyright> info from the *.nuspec files? How do we add for example what we currently have in our license? for example:

Copyright (c) 2011-2021 Twitter, Inc.
Copyright (c) 2011-2021 The Bootstrap Authors

Seems interesting to use the SPDX format in order to see what kind of license is used directly in the NuGet Gallery (this value is maybe used as well by 3rd party tools as well) but I'd say it's important that we can keep all existing info.

@XhmikosR Any thoughts? Have things like that been already tried out?

@vanillajonathan
Copy link
Contributor Author

I don't know. But this is the way other packages do it too, including Microsoft:
Example https://www.nuget.org/packages/System.Net.Http.Json and https://www.nuget.org/packages/Microsoft.EntityFrameworkCore/

@XhmikosR
Copy link
Member

/CC @supergibbs who is handling the NuGet packages

@XhmikosR XhmikosR changed the title Use license expression NuGet: Use license expression May 24, 2022
@supergibbs
Copy link
Contributor

supergibbs commented May 25, 2022

Looks good to me, except the license file would change. The copyright info is displayed on nuget.org which is just the dates right now. If you want to show:

Copyright (c) 2011-2022 Twitter, Inc.
Copyright (c) 2011-2022 The Bootstrap Authors

we can update the .nuspec files accordingly

@mdo
Copy link
Member

mdo commented Sep 1, 2022

We should simplify to The Bootstrap Authors for everywhere IMO—we can drop the Twitter mention after ~10 years of them having nothing to do with it lol.

Copy link
Contributor

@supergibbs supergibbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't officially approve, but looks good to me

@supergibbs
Copy link
Contributor

@vanillajonathan Actually noticed something, .nuspec files should remove <file src="LICENSE.txt" target="" />

@vanillajonathan
Copy link
Contributor Author

@supergibbs Done!

@XhmikosR
Copy link
Member

Not sure if we need to backport this to v4-dev.

@XhmikosR XhmikosR merged commit f84d82a into twbs:main Sep 27, 2022
@vanillajonathan vanillajonathan deleted the patch-1 branch September 27, 2022 19:49
@supergibbs supergibbs mentioned this pull request Oct 25, 2022
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants