Skip to content

refactor: extract create-provenance to a script#163

Closed
keerthi-go wants to merge 1 commit into
mainfrom
refactor/create-provenance-script
Closed

refactor: extract create-provenance to a script#163
keerthi-go wants to merge 1 commit into
mainfrom
refactor/create-provenance-script

Conversation

@keerthi-go
Copy link
Copy Markdown
Contributor

Summary

Moves the 73-line Create provenance file inline shell block out of qcom-release-reusable-workflow.yml into scripts/create-provenance.sh.

The workflow step becomes a single line:

- name: Create provenance file
  env:
    DISTRO_CODENAME: ${{ inputs.suite }}
    DEBIAN_BRANCH: ${{ inputs.debian-branch }}
    UPSTREAM_REPO: ${{ vars.UPSTREAM_REPO_GITHUB_NAME }}
    PKG_VERSION: ${{ steps.changelog.outputs.version }}
    PKG_REPO: ${{ github.repository }}
  run: ./qcom-build-utils/scripts/create-provenance.sh

No behaviour change

All logic is identical — supports both source packages and prebuilt binary packages (upstream.conf). All env vars are unchanged. Script validates required vars at startup with :${VAR:?}.

Move the 'Create provenance file' inline shell block
(73 lines) into scripts/create-provenance.sh.

The workflow step is now a single line:
  run: ./qcom-build-utils/scripts/create-provenance.sh

All env vars (DISTRO_CODENAME, DEBIAN_BRANCH,
UPSTREAM_REPO, PKG_VERSION, PKG_REPO) are unchanged.
No behaviour change.

Signed-off-by: Keerthi Gowda <kbalehal@qti.qualcomm.com>
@keerthi-go
Copy link
Copy Markdown
Contributor Author

Superseded by #165 which combines both provenance scripts into one PR.

@keerthi-go keerthi-go closed this May 28, 2026
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.

1 participant