diff options
author | Seth Michael Larson <seth@python.org> | 2024-04-30 15:05:05 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-04-30 15:05:05 (GMT) |
commit | 72dae53e09a5344bf4922d934a34a2fa48a11c86 (patch) | |
tree | 0d19db06ea818ab8f2fb659519bb78c3445e9eb3 /Tools | |
parent | 9a75d56d5d9fdffb6ce9d83ede98486df238102d (diff) | |
download | cpython-72dae53e09a5344bf4922d934a34a2fa48a11c86.zip cpython-72dae53e09a5344bf4922d934a34a2fa48a11c86.tar.gz cpython-72dae53e09a5344bf4922d934a34a2fa48a11c86.tar.bz2 |
gh-116122: Add SBOM generation to PCbuild/build.bat (GH-116138)
Diffstat (limited to 'Tools')
-rw-r--r-- | Tools/build/generate_sbom.py | 21 |
1 files changed, 18 insertions, 3 deletions
diff --git a/Tools/build/generate_sbom.py b/Tools/build/generate_sbom.py index 5c1851f..258b58c 100644 --- a/Tools/build/generate_sbom.py +++ b/Tools/build/generate_sbom.py @@ -4,13 +4,13 @@ import re import hashlib import json import glob -import pathlib +from pathlib import Path, PurePosixPath, PureWindowsPath import subprocess import sys import urllib.request import typing -CPYTHON_ROOT_DIR = pathlib.Path(__file__).parent.parent.parent +CPYTHON_ROOT_DIR = Path(__file__).parent.parent.parent # Before adding a new entry to this list, double check that # the license expression is a valid SPDX license expression: @@ -119,9 +119,16 @@ def filter_gitignored_paths(paths: list[str]) -> list[str]: # 1 means matches, 0 means no matches. assert git_check_ignore_proc.returncode in (0, 1) + # Paths may or may not be quoted, Windows quotes paths. + git_check_ignore_re = re.compile(r"^::\s+(\"([^\"]+)\"|(.+))\Z") + # Return the list of paths sorted git_check_ignore_lines = git_check_ignore_proc.stdout.decode().splitlines() - return sorted([line.split()[-1] for line in git_check_ignore_lines if line.startswith("::")]) + git_check_not_ignored = [] + for line in git_check_ignore_lines: + if match := git_check_ignore_re.fullmatch(line): + git_check_not_ignored.append(match.group(2) or match.group(3)) + return sorted(git_check_not_ignored) def get_externals() -> list[str]: @@ -238,12 +245,20 @@ def create_source_sbom() -> None: ) for path in paths: + + # Normalize the filename from any combination of slashes. + path = str(PurePosixPath(PureWindowsPath(path))) + # Skip directories and excluded files if not (CPYTHON_ROOT_DIR / path).is_file() or path in exclude: continue # SPDX requires SHA1 to be used for files, but we provide SHA256 too. data = (CPYTHON_ROOT_DIR / path).read_bytes() + # We normalize line-endings for consistent checksums. + # This is a rudimentary check for binary files. + if b"\x00" not in data: + data = data.replace(b"\r\n", b"\n") checksum_sha1 = hashlib.sha1(data).hexdigest() checksum_sha256 = hashlib.sha256(data).hexdigest() |