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

No crossRef on an extractedLicenseInfo? #120

Closed
armintaenzertng opened this issue Sep 29, 2022 · 2 comments
Closed

No crossRef on an extractedLicenseInfo? #120

armintaenzertng opened this issue Sep 29, 2022 · 2 comments

Comments

@armintaenzertng
Copy link

According to spec, an extractedLicenseInfo has the property LicenseCrossReference which I assumed to be implemented by the class CrossRef.
extractedLicenseInfo has no setter for a CrossRef object, though, only for a string collection seeAlso. Which for current purposes is totally fine as the intent to provide a URL is still taken care of. I just wondered if seeAlso should actually be a (collection of) CrossRefs

@goneall
Copy link
Member

goneall commented Oct 1, 2022

@armintaenzertng - the seeAlso collection maps directly to the tag LicenseCrossReference.

CrossRef is actually for a different purpose. It is used to record more detail about the the URL's listed in the Listed Licenses web page (e.g. if they are live). It should only be used for listed licenses today.

There is an issue in the spec repo to document these fields: spdx/spdx-spec#46

For the purposes of extractedLicenseInfo I suggest just using the seeAlso collection.

@armintaenzertng
Copy link
Author

ok then, if seeAlso is fine, I just got confused due to the different naming conventions in the spec and the code.

Thanks for clarifying! :)

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

No branches or pull requests

2 participants