conda-forge / intel_repack-feedstock

A conda-smithy repository for intel_repack.
BSD 3-Clause "New" or "Revised" License
6 stars 19 forks source link

Use LicenseRef-IntelSimplifiedSoftware as SPDX identifier #67

Closed xhochy closed 6 months ago

xhochy commented 6 months ago

This allows us to differentiate the license of these packages from other proprietary Intel packages with a different license.

Checklist

conda-forge-webservices[bot] commented 6 months ago

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.

Here's what I've got...

For recipe:

PaulKMueller commented 6 months ago

I checked out the raw license files for the different builds locally. Intel Simplified Software License (Version October 2022) should indeed be the applicable license for all of them 👍 .

oleksandr-pavlyk commented 6 months ago

Should we expected "IntelSimplifiedSoftware" to be present in https://github.com/spdx/license-list-data/blob/main/json/licenses.json or is it a conda-specific token?

beckermr commented 6 months ago

We should not expect it to be there. The LicenseRef- syntax indicates a custom token.

xhochy commented 6 months ago

I'm also happy to change it to something else, as this can be an arbitrary string. I'm mostly interested in having a string that separates it from other Intel licenses.

napetrov commented 6 months ago

One note - one of the reasons that this license is not SPDX conformant - it's not stable license and get ongoing working updates - so current versioning is important.

So LicenseRef-IntelSimplifiedSoftware is fine as long as there is no goal for differentiating between particular versions

xhochy commented 6 months ago

Seems like it would be better to include the version then, too? Should I add a 2022 suffix?

napetrov commented 6 months ago

Seems like it would be better to include the version then, too? Should I add a 2022 suffix?

or even like Oct2022 - as technically there might be multiple updates during year. Hopefully this will not happen

xhochy commented 6 months ago

@conda-forge-admin please rerender

xhochy commented 6 months ago

Updated to the dated-license and CI has passed, I hope this is ready for a merge now.