CycloneDX / cyclonedx-property-taxonomy

A taxonomy of all official CycloneDX property namespaces and names
https://cyclonedx.github.io/cyclonedx-property-taxonomy/
Apache License 2.0
14 stars 29 forks source link

docs: clarify namespace in ABNF #48

Open jkowalleck opened 1 year ago

jkowalleck commented 1 year ago

i found the docs confusing.

in the current ABNF it reads

property-name = 1*(namespace ":") name
; ...

the problem i saw: the namespace is used multiple time, so if i registered a namespace, how would i use it multiple times?

Here is a fix.

coderpatros commented 1 year ago

i found the docs confusing.

in the current ABNF it reads

property-name = 1*(namespace ":") name
; ...

the problem i saw: the namespace is used multiple time, so if i registered a namespace, how would i use it multiple times?

Here is a fix.

Maybe it should be reworded instead? Instead of namespace maybe namespace-segment? The current syntax is to differentiate between namespaces, sub-namespaces, and the name of the property within the namespace. Thoughts?

jkowalleck commented 1 year ago

re: https://github.com/CycloneDX/cyclonedx-property-taxonomy/pull/48#issuecomment-1483692202

then why not call these things exactly like that? how do you like te following?

property-name =  name
property-name =/ namespace ":" 1*(sub-namespace ":") name

namespace = 1*namechar
sub-namespaces = 1*namechar
name = 1*namechar

; ...