bootstrap icon indicating copy to clipboard operation
bootstrap copied to clipboard

NuGet: Use license expression

Open vanillajonathan opened this issue 3 years ago • 7 comments

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

vanillajonathan avatar May 22 '22 23:05 vanillajonathan

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?

julien-deramond avatar May 23 '22 05:05 julien-deramond

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/

vanillajonathan avatar May 23 '22 08:05 vanillajonathan

/CC @supergibbs who is handling the NuGet packages

XhmikosR avatar May 24 '22 08:05 XhmikosR

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

supergibbs avatar May 25 '22 19:05 supergibbs

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.

mdo avatar Sep 01 '22 19:09 mdo

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

supergibbs avatar Sep 13 '22 08:09 supergibbs

@supergibbs Done!

vanillajonathan avatar Sep 13 '22 09:09 vanillajonathan

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

XhmikosR avatar Sep 27 '22 19:09 XhmikosR