package.PrimaryPackagePurpose should be []string

[puerco]

The field to capture the PrimaryPurpose of packages should be a list of strings instead of a single string as it has a cardinality of many.

It is even een noted in the comments:

tools-golang/spdx/v2/v2_3/package.go

Lines 117 to 120 in 7f95b01

	// 7.24: Primary Package Purpose
	// Cardinality: optional, one or many
	// Allowed values: APPLICATION, FRAMEWORK, LIBRARY, CONTAINER, OPERATING-SYSTEM, DEVICE, FIRMWARE, SOURCE, ARCHIVE, FILE, INSTALL, OTHER
	PrimaryPackagePurpose string `json:"primaryPackagePurpose,omitempty"`

Since this is a breaking change I wonder what would be the best way to fix it 🤔

[goneall]

@puerco - I think this was an error in the spec - I believe the cardinality should be 0..1 not 0..*

[goneall]

Reference: spdx/spdx-spec#794 spdx/spdx-spec#797

[puerco]

Ooh I just noticed, it was weird to find out the 0..many cardinality, I think I've written more than one tool that accepts it.

I saw that Keith already fixed the spec for a future release, do you think we should correct the 2.3 published spec too?

(I'll close this issue)