Skip to content

create_data_tar: collect NOTICE and LICENSE files locally from source repo#33

Merged
Keerthi Gowda (keerthi-go) merged 1 commit into
mainfrom
feat/local-notice-collect
Jul 2, 2026
Merged

create_data_tar: collect NOTICE and LICENSE files locally from source repo#33
Keerthi Gowda (keerthi-go) merged 1 commit into
mainfrom
feat/local-notice-collect

Conversation

@keerthi-go

@keerthi-go Keerthi Gowda (keerthi-go) commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Generate consolidated NOTICE file with all the NOTICE and LICENSE files:

  • Add helper_scripts/notice_and_license.py:
    • collect_notice(): uses git ls-files to find all tracked NOTICE and LICENSE files in the source repo and concatenates them; adds per-file section headers (FILE: <path>) when multiple files are found; non-fatal if no files exist
    • fetch_license_qcom2(): downloads LICENSE.qcom-2 from meta-qcom; non-fatal on network failure (warns and skips)
    • strip_doc_dirs(): removes usr/share/doc/<pkg>/ from extracted debs

Comment thread .github/workflows/stale-issues.yaml Fixed
… repo

Replace the Qualcomm notice-gen API with a fully local approach:

- Add helper_scripts/notice_and_license.py:
  - collect_notice(): uses 'git ls-files' to find all tracked NOTICE
    and LICENSE files in the source repo and concatenates them; adds
    per-file section headers (FILE: <path>) when multiple files are
    found; non-fatal if no files exist (warns and skips)
  - fetch_license_qcom2(): downloads LICENSE.qcom-2 from meta-qcom;
    non-fatal on network failure (warns and skips)
  - strip_doc_dirs(): removes usr/share/doc/<pkg>/ from extracted debs

- Update create_data_tar.py:
  - Call collect_notice() instead of the notice-gen API
  - Drop --_notice-project / --_notice-revision hidden args and the
    resolve_git_context() plumbing that passed them into Docker

Signed-off-by: Keerthi Gowda <kbalehal@qti.qualcomm.com>
@keerthi-go Keerthi Gowda (keerthi-go) merged commit 32931cc into main Jul 2, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants