Reformat the code to comply with Ruff's check #340

Signed-off-by: Chin Yeung Li <[email protected]>

Author: chinyeungli

build_rpm_docker.py

@@ -26,60 +26,77 @@
 
 def build_rpm_with_docker():
     # Load the pyproject.toml file
-    with open('pyproject.toml') as f:
-        project = toml.load(f)['project']
+    with open("pyproject.toml") as f:
+        project = toml.load(f)["project"]
 
-    pkg_name = project['name']
+    pkg_name = project["name"]
     # Insert "python3-"" prefix that follows a common convention for Python RPMs
     rpm_name = f"python3-{pkg_name.lower()}"
 
     docker_cmd = [
-        'docker', 'run', '--rm',
-        '-v', f"{os.getcwd()}:/workspace",
-        '-w', '/workspace',
-        'fedora:42',
-        '/bin/bash', '-c',
+        "docker",
+        "run",
+        "--rm",
+        "-v",
+        f"{os.getcwd()}:/workspace",
+        "-w",
+        "/workspace",
+        "fedora:42",
+        "/bin/bash",
+        "-c",
         f"""set -ex
-            # Install All build dependencies
-            dnf install -y rpm-build python3-devel python3-setuptools python3-wheel python3-build python3-toml
+# Install All build dependencies
+dnf install -y rpm-build python3-devel python3-setuptools python3-wheel python3-build python3-toml
 
-            # Build the wheel
-            python3 -m build --wheel
+# Build the wheel
+python3 -m build --wheel
 
-            # Get the wheel file name
-            WHEEL_FILE=$(ls dist/*.whl)
-            if [ -z "$WHEEL_FILE" ]; then
-                echo "Error: No wheel file found in dist/." >&2
-                exit 1
-            fi
-            WHEEL_FILENAME=$(basename "$WHEEL_FILE")
+# Get the wheel file name
+WHEEL_FILE=$(ls dist/*.whl)
+if [ -z "$WHEEL_FILE" ]; then
+    echo "Error: No wheel file found in dist/." >&2
+    exit 1
+fi
+WHEEL_FILENAME=$(basename "$WHEEL_FILE")
 
-            # Keep RPM version as is for sorting
-            RPM_VERSION="{project['version'].replace("-dev", "~dev")}"
+# Keep RPM version as is for sorting
+RPM_VERSION="{project["version"].replace("-dev", "~dev")}"
 
-            # Creates the standard directory structure required by rpmbuild
-            mkdir -p dist/rpmbuild/{{BUILD,RPMS,SOURCES,SPECS,SRPMS}}
-            mv "$WHEEL_FILE" dist/rpmbuild/SOURCES/
+# Creates the standard directory structure required by rpmbuild
+mkdir -p dist/rpmbuild/{{BUILD,RPMS,SOURCES,SPECS,SRPMS}}
+mv "$WHEEL_FILE" dist/rpmbuild/SOURCES/
 
-            # Get the changelog date
-            CHANGELOG_DATE=$(date '+%a %b %d %Y')
+# Get the changelog date
+CHANGELOG_DATE=$(date '+%a %b %d %Y')
 
-            # Generate spec file with correct deps
-            cat > dist/rpmbuild/SPECS/{rpm_name}.spec << EOF
+# Generate spec file with correct deps
+cat > dist/rpmbuild/SPECS/{rpm_name}.spec << EOF
 Name:           {rpm_name}
 Version:        $RPM_VERSION
 Release:        1%{{?dist}}
-Summary:        {project.get('description', 'Automate open source license compliance and ensure supply chain integrity')}
-
-License:        {project.get('license', 'AGPL-3.0-only')}
-URL:            {project.get('urls', '').get('Homepage', 'https://github.com/aboutcode-org/dejacode')}
+Summary:        {
+            project.get(
+                "description",
+                "Automate open source license complianceand ensure supply chain integrity",
+            )
+        }
+
+License:        {project.get("license", "AGPL-3.0-only")}
+URL:            {
+            project.get("urls", "").get("Homepage", "https://github.com/aboutcode-org/dejacode")
+        }
 Source0:        "$WHEEL_FILENAME"
 
 BuildArch:      noarch
 BuildRequires:  python3-devel python3-setuptools python3-wheel python3-build python3-toml
 
 %description
-{project.get('description', 'Automate open source license compliance and ensure supply chain integrity')}
+{
+            project.get(
+                "description",
+                "Automate open source license compliance and ensuresupply chain integrity",
+            )
+        }
 
 %prep
 
@@ -93,22 +110,26 @@ def build_rpm_with_docker():
 %{{python3_sitelib}}/*
 
 %changelog
-* $CHANGELOG_DATE {project.get('authors', [{}])[0].get('name', 'nexB Inc.')} - $RPM_VERSION-1
-- {project.get('urls', '').get('Changelog', 'https://github.com/aboutcode-org/dejacode/blob/main/CHANGELOG.rst')}
+* $CHANGELOG_DATE {project.get("authors", [{}])[0].get("name", "nexB Inc.")} - $RPM_VERSION-1
+- {
+            project.get("urls", "").get(
+                "Changelog", "https://github.com/aboutcode-org/dejacode/blob/main/CHANGELOG.rst"
+            )
+        }
 EOF
 
-            # Build the RPM
-            rpmbuild --define "_topdir /workspace/dist/rpmbuild" -bb dist/rpmbuild/SPECS/{rpm_name}.spec
+# Build the RPM
+rpmbuild --define "_topdir /workspace/dist/rpmbuild" -bb dist/rpmbuild/SPECS/{rpm_name}.spec
 
-            # Fix permissions for Windows host
-            chmod -R u+rwX dist/rpmbuild
-        """
+# Fix permissions for Windows host
+chmod -R u+rwX dist/rpmbuild
+""",
     ]
 
     try:
-        subprocess.run(docker_cmd, check=True)
+        subprocess.run(docker_cmd, check=True, shell=False)  # noqa: S603
         # Verify the existance of the .rpm
-        rpm_file = next(Path('dist/rpmbuild/RPMS/noarch').glob('*.rpm'), None)
+        rpm_file = next(Path("dist/rpmbuild/RPMS/noarch").glob("*.rpm"), None)
         if rpm_file:
             print(f"\nSuccess! RPM built: {rpm_file}")
         else:
@@ -119,9 +140,9 @@ def build_rpm_with_docker():
         sys.exit(1)
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     # Check if "docker" is available
-    if not shutil.which('docker'):
+    if not shutil.which("docker"):
         print("Error: Docker not found. Please install Docker first.", file=sys.stderr)
         sys.exit(1)