This is an automated email from the ASF dual-hosted git repository.
janc pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-core.git
The following commit(s) were added to refs/heads/master by this push:
new 544f20e38 ci: Don't create temp folder for RAT checks
544f20e38 is described below
commit 544f20e383f092a9626932566ef004457405b949
Author: Szymon Janc <[email protected]>
AuthorDate: Wed Oct 22 13:19:10 2025 +0200
ci: Don't create temp folder for RAT checks
Now that RAT is properly handling full file paths we don't have
to create temporary folder with added files as check can be done
on whole tree.
---
.github/check_license.py | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/.github/check_license.py b/.github/check_license.py
index 8145ce81d..bd41e6991 100755
--- a/.github/check_license.py
+++ b/.github/check_license.py
@@ -26,7 +26,6 @@ import sys
import shutil
from pathlib import Path
-tmp_folder = "/tmp/check_license/"
licenses_url = "https://www.apache.org/legal/resolved.html";
LICENSE_FILE = "LICENSE"
RAT_EXCLUDES_FILE = ".rat-excludes"
@@ -96,13 +95,12 @@ def check_license_files(license_entries: set[str]) ->
list[str]:
def run_rat_check(files_diff: list[str]) -> list[str]:
result = []
- for cfg_fname in files_diff:
- os.makedirs(str(Path(tmp_folder + cfg_fname).parent), 0o755, True)
- shutil.copy(cfg_fname, tmp_folder + cfg_fname)
- rat_out = run_cmd_no_check(f"java -jar apache-rat.jar --input-exclude-std
GIT --input-exclude-parsed-scm GIT --input-exclude-file .rat-excludes
--output-style .github/rat_report.xsl -- . {tmp_folder} | grep \"^ ! \"")
+ rat_out = run_cmd_no_check(f"java -jar apache-rat.jar --input-exclude-std
GIT --input-exclude-parsed-scm GIT --input-exclude-file .rat-excludes
--output-style .github/rat_report.xsl -- . | grep \"^ ! \"")
if rat_out:
for entry in rat_out:
- result.append(entry.strip(' !').replace(tmp_folder, '\t\t'))
+ entry = entry.strip(' ! /')
+ if entry in files_diff:
+ result.append(entry)
return result
