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

Generate git version and include that in source output #46

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

raharper
Copy link
Contributor

@raharper raharper commented Mar 8, 2024

The current GIT_VERSION value only works if you build from a git repo. Once we release on github, the tarballs that one can download are not git repos and the git command to describe the version fail.

Generate the git version value and write to a file. In source tarball builds, the value will come from the file rather than git.

  • Add 'show-version' as makefile target
  • Add 'show-version' to github workflow
  • Adjust make-tarball tool to insert the git-version file

Fixes: #45

@raharper raharper requested a review from hallyn March 8, 2024 18:43
@raharper
Copy link
Contributor Author

raharper commented Mar 8, 2024

Hrm, this focal environment is getting a security violation with its efi shell:

BdsDxe: failed to start Boot0002 "EFI Internal Shell" from Fv(7CB8BDC9-F8EB-4F34-AAEA-3EE4AF6516A1)/FvFile(7C04A583-9E3E-4F1C-AD65-E05268D0B4D1): Security Violation

I don't get that on a local focal system.

@raharper
Copy link
Contributor Author

raharper commented Mar 8, 2024

@raharper raharper force-pushed the feat/include-version-in-source-v2 branch from aaac3f4 to 4d247aa Compare March 18, 2024 16:49
The current GIT_VERSION value only works if you build from a git
repo.  Once we release on github, the tarballs that one can download
are not git repos and the git command to describe the version fail.

Generate the git version value and write to a file.  In source tarball
builds, the value will come from the file rather than git.

- Add 'show-version' as makefile target
- Add 'show-version' to github workflow
- Adjust make-tarball tool to insert the git-version file

Signed-off-by: Ryan Harper <[email protected]>
…debug

- Drop ovmf down to restore use of uefi-shell in secureboot mode
- Update collect-firmwares to use correct ovmf version
- Only print serial log if args verbose is set
@raharper raharper force-pushed the feat/include-version-in-source-v2 branch from 7f44a2d to 3666982 Compare March 18, 2024 18:54
@hallyn hallyn merged commit 9768251 into puzzleos:main Mar 18, 2024
1 check 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.

building from release tarballs will miss GIT_VERSION in CFLAGS
2 participants