Skip to content
This repository has been archived by the owner on Aug 27, 2022. It is now read-only.

Propose changing LicenseReference Class to the ExtractedLicensingInfo Class used by SPDX #17

Open
goneall opened this issue Jan 9, 2020 · 3 comments

Comments

@goneall
Copy link
Collaborator

goneall commented Jan 9, 2020

Propose changing LicenseReference Class to the SPDX ExtractedLicensingInfo class
Properties:
licenseId [1] – string
comment [0..1] – string
name [0..1] – string
seeAlso [*] – URL
extratedText [1] - string

@CASTResearchLabs
Copy link
Collaborator

isn't it just an attribute name issue more than a structural issue?
LicenseReference has urls <-> seeAlso, expression <-> extractedText, name, identifier <-> licenseId
Comments could generally be handled as annotations for any Element

@goneall
Copy link
Collaborator Author

goneall commented Jan 14, 2020

@CASTResearchLabs After reviewing the model again, I agree with the exception of adding extratedText. Extracted text is semantically different from expression used in other parts of the document. The extractedText would hold the exact text of the license or notice as found in the artifact(s).

@goneall
Copy link
Collaborator Author

goneall commented Jan 14, 2020

I updated the spreadsheet with the proposed field name changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants