uptane / uptane-standard

standard for Uptane
https://uptane.github.io
Other
37 stars 31 forks source link

Version Updates, SHALL Preference, and Added definitions #215

Closed jhdalek55 closed 3 years ago

jhdalek55 commented 3 years ago

I added a note that the compliance word for the Standard is to be SHALL, not MUST and updated to the reference links for the standard and deployment documents to 1.2.0

jhdalek55 commented 3 years ago

The second commit on this PR adds the actual RFC 2119 definitions to the Standard, as well as the caution against the use of imperatives from that document. This was done in response to a discussion at our 8/3/21 Uptane Standards meeting.

jhdalek55 commented 3 years ago

@tkfu @iramcdonald, others...please review and merge if OK.....

jhdalek55 commented 3 years ago

Second call...can we get eyes on this and an approval?

iramcdonald commented 3 years ago

Hi Lois,

I am fine with adding the RFC 2119 definitions of the conformance terms.

However, I am completely defeated by GitHub. I cannot find issue #215 at all (as open or closed). I cannot find a link to the actual text that is changed.

Sorry, but I continue to find GitHub much less than ideal...

Cheers,

Ira McDonald (Musician / Software Architect)

Chair - SAE Trust Anchors and Authentication TF Co-Chair - TCG Trusted Mobility Solutions WG

Co-Chair - TCG Metadata Access Protocol SG

*Chair - Linux Foundation Open Printing WGSecretary - IEEE-ISTO Printer Working GroupCo-Chair - IEEE-ISTO PWG Internet Printing Protocol WGIETF Designated Expert - IPP & Printer MIBBlue Roof Music / High North Inchttp://sites.google.com/site/blueroofmusic http://sites.google.com/site/blueroofmusichttp://sites.google.com/site/highnorthinc http://sites.google.com/site/highnorthincmailto: @. @.>(permanent) PO Box 221 Grand Marais, MI 49839 906-494-2434*

On Thu, Sep 9, 2021 at 12:53 PM Lois Anne DeLong @.***> wrote:

Second call...can we get eyes on this and an approval?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/uptane/uptane-standard/pull/215#issuecomment-916270602, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE33UO432SRIQLJI6COYR73UBDRA5ANCNFSM5CKMHWAQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

jhdalek55 commented 3 years ago

Sorry @iramcdonald. It's not my favorite thing either.

This link should take you to the side by side comparison of text so you can see what was added https://github.com/uptane/uptane-standard/pull/215/files

Lois

iramcdonald commented 3 years ago

Hi Lois,

Thanks - I reviewed the changes - looks good to me.

Cheers,

Ira McDonald (Musician / Software Architect)

Chair - SAE Trust Anchors and Authentication TF Co-Chair - TCG Trusted Mobility Solutions WG

Co-Chair - TCG Metadata Access Protocol SG

*Chair - Linux Foundation Open Printing WGSecretary - IEEE-ISTO Printer Working GroupCo-Chair - IEEE-ISTO PWG Internet Printing Protocol WGIETF Designated Expert - IPP & Printer MIBBlue Roof Music / High North Inchttp://sites.google.com/site/blueroofmusic http://sites.google.com/site/blueroofmusichttp://sites.google.com/site/highnorthinc http://sites.google.com/site/highnorthincmailto: @. @.>(permanent) PO Box 221 Grand Marais, MI 49839 906-494-2434*

On Mon, Sep 13, 2021 at 10:33 AM Lois Anne DeLong @.***> wrote:

Sorry @iramcdonald https://github.com/iramcdonald. It's not my favorite thing either.

This link should take you to the side by side comparison of text so you can see what was added https://github.com/uptane/uptane-standard/pull/215/files

Lois

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/uptane/uptane-standard/pull/215#issuecomment-918255575, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE33UO3MTGA2UPBQYWLADO3UBYDSZANCNFSM5CKMHWAQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

jhdalek55 commented 3 years ago

Thanks @iramcdonald . Can someone with merge privileges take care of this?

jhdalek55 commented 3 years ago

Thanks, @mnm678