diff options
author | Dave Rodgman <dave.rodgman@arm.com> | 2023-10-02 17:19:51 +0100 |
---|---|---|
committer | Dave Rodgman <dave.rodgman@arm.com> | 2023-10-02 17:19:51 +0100 |
commit | 65d8ec1444542a905e8209e0438646d80bb28473 (patch) | |
tree | 756aa940eb7fc10766ab22ef1d720c8622b0286a | |
parent | 3901e2ef92eba8571d12c0531833e79bee406970 (diff) | |
download | mbedtls-65d8ec1444542a905e8209e0438646d80bb28473.zip mbedtls-65d8ec1444542a905e8209e0438646d80bb28473.tar.gz mbedtls-65d8ec1444542a905e8209e0438646d80bb28473.tar.bz2 |
Move check into list_files_to_merge
Signed-off-by: Dave Rodgman <dave.rodgman@arm.com>
-rwxr-xr-x | scripts/assemble_changelog.py | 23 |
1 files changed, 14 insertions, 9 deletions
diff --git a/scripts/assemble_changelog.py b/scripts/assemble_changelog.py index 44fe011..e808101 100755 --- a/scripts/assemble_changelog.py +++ b/scripts/assemble_changelog.py @@ -448,18 +448,24 @@ def list_files_to_merge(options): """List the entry files to merge, oldest first. "Oldest" is defined by `EntryFileSortKey`. + + Also check for required .txt extension """ - files_to_merge = glob.glob(os.path.join(options.dir, '*.txt')) + files_to_merge = glob.glob(os.path.join(options.dir, '*')) + + # Ignore 00README.md + readme = os.path.join(options.dir, "00README.md") + if readme in files_to_merge: + files_to_merge.remove(readme) + + # Identify files without the required .txt extension + bad_files = [x for x in files_to_merge if not x.endswith(".txt")] + if bad_files: + raise FilePathError(bad_files) + files_to_merge.sort(key=EntryFileSortKey) return files_to_merge -def check_extensions(options): - files = glob.glob(os.path.join(options.dir, '*')) - files = {x for x in files if not x.endswith(".txt")} - files.discard("ChangeLog.d/00README.md") - if files: - raise FilePathError(files) - def merge_entries(options): """Merge changelog entries into the changelog file. @@ -469,7 +475,6 @@ def merge_entries(options): Write the new changelog to options.output. Remove the merged entries if options.keep_entries is false. """ - check_extensions(options) with open(options.input, 'r', encoding='utf-8') as input_file: changelog = ChangeLog(input_file, TextChangelogFormat) files_to_merge = list_files_to_merge(options) |