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

Add Annex F: History, Motivation and Rational #1008

Closed
wants to merge 19 commits into from

Conversation

bact
Copy link
Collaborator

@bact bact commented Jul 25, 2024

  • Using text and 4 figures from Bob's Word doc 2024-07-04

Notes

  • SPDX is stylized as "Software Package Data eXchange" and "System Package Data eXchange" in this annex (same as in the new Introduction, as the last two paragraphs of this annex are largely similar to the Intro).
    • While in other instances of the spec, they are written as "Software Package Data Exchange" and "System Package Data Exchange".
    • Do we like to change both of them to a regular spelling?
  • Links. These two links at the end of the doc may need an update for 3.0.1:

    The SPDX 3.0 ontology is available at:
    https://github.com/spdx/spdx-spec/tree/development/v3.0/ontology

    The SPDX 3.0 specification is available as web pages at:
    https://spdx.github.io/spdx-spec/v3.0/

bact added 3 commits July 25, 2024 12:56
Using text and 4 figures from Bob's Word doc 2024-07-04

Signed-off-by: Arthit Suriyawongkul <[email protected]>
Signed-off-by: Arthit Suriyawongkul <[email protected]>
Signed-off-by: Arthit Suriyawongkul <[email protected]>
@bact bact mentioned this pull request Jul 25, 2024
69 tasks
Copy link

@bobmartin3000 bobmartin3000 left a comment

Choose a reason for hiding this comment

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

LGTM

@bact
Copy link
Collaborator Author

bact commented Aug 5, 2024

Note the links to model, ontology, and spec at Lines 136-143. These may need to recheck before publication.

@goneall goneall added this to the 3.0.1 milestone Aug 9, 2024
Copy link
Member

@goneall goneall left a comment

Choose a reason for hiding this comment

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

Since this is the history from an ISO perspective, we should start with SPDX history going back to 1.0 then, more importantly, through the previous ISO submission then adding this additional information.

I wonder if this chapter is even needed for the ISO submission since our prior submission did not include this level of detail. If it isn't required, I would recommend removing it.

bact and others added 2 commits August 9, 2024 19:52
@goneall
Copy link
Member

goneall commented Aug 9, 2024

Since this is the history from an ISO perspective, we should start with SPDX history going back to 1.0 then, more importantly, through the previous ISO submission then adding this additional information.

I wonder if this chapter is even needed for the ISO submission since our prior submission did not include this level of detail. If it isn't required, I would recommend removing it.

In reading further, it looks like the SPDX history is there, but much later in the document. Perhaps reordering the history to start more at the beginning may fix the problem.

@kestewart
Copy link
Contributor

Met with @bact, @goneall and decided this needs to be redone/renumbered, and depends on #992 being applied first.

Discussion is this isn't essential for the .pdf, and can be handled in a cover letter to OMG with the .pdf; and if we want to include it for ISO submission; probably needs to be reworked as described above.

Copy link
Contributor

@kestewart kestewart left a comment

Choose a reason for hiding this comment

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

No longer Annex F (if we end up including). Needs discussion.

@goneall
Copy link
Member

goneall commented Aug 10, 2024

Also note that the annex numbering needs to change and the mkdocs.yaml file needs to be updated.

@kestewart
Copy link
Contributor

I'm fine with this going to OMG. This needs the Annex F references updated.

bact and others added 2 commits August 11, 2024 22:34
Co-authored-by: Kate Stewart <[email protected]>
Signed-off-by: Arthit Suriyawongkul <[email protected]>
Copy link
Contributor

@kestewart kestewart left a comment

Choose a reason for hiding this comment

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

Approve this as long as added to mkdocs

Copy link
Member

@goneall goneall left a comment

Choose a reason for hiding this comment

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

LGTM

@bact bact mentioned this pull request Aug 11, 2024
@bact
Copy link
Collaborator Author

bact commented Aug 11, 2024

Please use #1053 instead

@bact bact closed this Aug 12, 2024
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

Successfully merging this pull request may close these issues.

5 participants