Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

changes the means to compute the version for packages #499

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

cmickeyb
Copy link
Contributor

@cmickeyb cmickeyb commented Oct 4, 2024

Change the way we compute the version so that github workflows can use this

Version information is kept in the file VERSION. The file contains lines for each version from the most recent to the most distant. Each tab separated line contains a version (of the form v?[0-9]+.[0-9]+.[0-9]+) and the hash of a git commit that represents that version. Each line may also have a short description separated by a tab.

When a new version is committed, only the VERSION file should be modified and the commit should be tagged (annotated tag) with the version number.

…e this

Version information is kept in the file VERSION. The file contains
lines for each version from the most recent to the most distant. Each
tab separated line contains a version (of the form
v?[0-9]+.[0-9]+.[0-9]+) and the hash of a git commit that represents
that version. Each line may also have a short description separated by
a tab.

When a new version is committed, only the VERSION file should be
modified and the commit should be tagged (annotated tag) with the
version number.

This also required some changes to the github workflow to pull
the full history of the repository (otherwise the patch level of
the version cannot be computed).

Signed-off-by: Mic Bowman <[email protected]>
Note that trivy notes errors about missing USER declarations. The
final images that are built, pdo_ccf, pdo_services, and pdo_client
all have users defined.

Signed-off-by: Mic Bowman <[email protected]>
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.

2 participants