@@ -71,6 +71,12 @@ <h1 class="title">Module <code>mbed_tools_ci_scripts.spdx_report.spdx_package</c
7171 uuid: str
7272
7373
74+ def _set_package_copyright(file: SpdxFile, package: Package) -> None:
75+ """Sets the copyright field of a package based on file copyright."""
76+ if file.copyright:
77+ package.cr_text = determine_spdx_value(file.copyright)
78+
79+
7480class SpdxPackage:
7581 """SPDX package.
7682
@@ -187,7 +193,7 @@ <h1 class="title">Module <code>mbed_tools_ci_scripts.spdx_report.spdx_package</c
187193 """
188194 return self._package_info.metadata.description
189195
190- def get_spdx_files(self) -> Optional[List[" SpdxFile" ]]:
196+ def get_spdx_files(self) -> Optional[List[SpdxFile]]:
191197 """Gets package's files SPDX description.
192198
193199 Returns:
@@ -224,7 +230,6 @@ <h1 class="title">Module <code>mbed_tools_ci_scripts.spdx_report.spdx_package</c
224230 """
225231 package = Package()
226232 package.check_sum = Algorithm("SHA1", str(NoAssert()))
227- package.verif_code = NoAssert()
228233 package.cr_text = NoAssert()
229234 package.name = determine_spdx_value(self.name)
230235 package.version = determine_spdx_value(self.version)
@@ -244,12 +249,13 @@ <h1 class="title">Module <code>mbed_tools_ci_scripts.spdx_report.spdx_package</c
244249 for file in files:
245250 package.add_file(file.generate_spdx_file())
246251 package.add_lics_from_file(License.from_identifier(str(determine_spdx_value(file.licence))))
247- package.cr_text = file.copyright
248- package.verif_code = package.calc_verif_code()
252+ _set_package_copyright( file, package)
253+ package.verif_code = determine_spdx_value( package.calc_verif_code() )
249254 else:
250255 # Has to generate a dummy file because of the following rule in SDK:
251256 # - Package must have at least one file
252257 dummy_file = SpdxFile(Path(UNKNOWN), self._package_info.root_dir, self.licence)
258+ package.verif_code = NoAssert()
253259 package.add_file(dummy_file.generate_spdx_file())
254260 package.add_lics_from_file(License.from_identifier(str(determine_spdx_value(dummy_file.licence))))
255261 return package</ code > </ pre >
@@ -429,7 +435,7 @@ <h2 id="attributes">Attributes</h2>
429435 """
430436 return self._package_info.metadata.description
431437
432- def get_spdx_files(self) -> Optional[List[" SpdxFile" ]]:
438+ def get_spdx_files(self) -> Optional[List[SpdxFile]]:
433439 """Gets package's files SPDX description.
434440
435441 Returns:
@@ -466,7 +472,6 @@ <h2 id="attributes">Attributes</h2>
466472 """
467473 package = Package()
468474 package.check_sum = Algorithm("SHA1", str(NoAssert()))
469- package.verif_code = NoAssert()
470475 package.cr_text = NoAssert()
471476 package.name = determine_spdx_value(self.name)
472477 package.version = determine_spdx_value(self.version)
@@ -486,12 +491,13 @@ <h2 id="attributes">Attributes</h2>
486491 for file in files:
487492 package.add_file(file.generate_spdx_file())
488493 package.add_lics_from_file(License.from_identifier(str(determine_spdx_value(file.licence))))
489- package.cr_text = file.copyright
490- package.verif_code = package.calc_verif_code()
494+ _set_package_copyright( file, package)
495+ package.verif_code = determine_spdx_value( package.calc_verif_code() )
491496 else:
492497 # Has to generate a dummy file because of the following rule in SDK:
493498 # - Package must have at least one file
494499 dummy_file = SpdxFile(Path(UNKNOWN), self._package_info.root_dir, self.licence)
500+ package.verif_code = NoAssert()
495501 package.add_file(dummy_file.generate_spdx_file())
496502 package.add_lics_from_file(License.from_identifier(str(determine_spdx_value(dummy_file.licence))))
497503 return package</ code > </ pre >
@@ -791,7 +797,6 @@ <h2 id="returns">Returns</h2>
791797 """
792798 package = Package()
793799 package.check_sum = Algorithm("SHA1", str(NoAssert()))
794- package.verif_code = NoAssert()
795800 package.cr_text = NoAssert()
796801 package.name = determine_spdx_value(self.name)
797802 package.version = determine_spdx_value(self.version)
@@ -811,12 +816,13 @@ <h2 id="returns">Returns</h2>
811816 for file in files:
812817 package.add_file(file.generate_spdx_file())
813818 package.add_lics_from_file(License.from_identifier(str(determine_spdx_value(file.licence))))
814- package.cr_text = file.copyright
815- package.verif_code = package.calc_verif_code()
819+ _set_package_copyright( file, package)
820+ package.verif_code = determine_spdx_value( package.calc_verif_code() )
816821 else:
817822 # Has to generate a dummy file because of the following rule in SDK:
818823 # - Package must have at least one file
819824 dummy_file = SpdxFile(Path(UNKNOWN), self._package_info.root_dir, self.licence)
825+ package.verif_code = NoAssert()
820826 package.add_file(dummy_file.generate_spdx_file())
821827 package.add_lics_from_file(License.from_identifier(str(determine_spdx_value(dummy_file.licence))))
822828 return package</ code > </ pre >
@@ -833,7 +839,7 @@ <h2 id="returns">Returns</h2>
833839< summary >
834840< span > Expand source code</ span >
835841</ summary >
836- < pre > < code class ="python "> def get_spdx_files(self) -> Optional[List[" SpdxFile" ]]:
842+ < pre > < code class ="python "> def get_spdx_files(self) -> Optional[List[SpdxFile]]:
837843 """Gets package's files SPDX description.
838844
839845 Returns:
0 commit comments