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

fsl-kernel-localversion: fix SRCREV_machine and AUTOREV use cases #1696

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

MaxKrummenacher
Copy link
Contributor

This would need backport to kirkstone too. The commit it fixes is currently a backport pull request for kirkstone: #1694

Depending on how one uses the Yocto kernel classes the kernels hash
is defined in either "SRCREV_machine" or "SRCREV". If "SRCREV_machine"
is in use, "SRCREV" stays at its bitbake default "INVALID".

If the "SRCREV_machine" or "SRCREV" is set to "AUTOREV" that value is
replaced by "AUTOINC".

If using "SRCREV_machine" and/or "AUTOREV" do_kernel_localversion fails

| run.do_kernel_localversion:158 exit 128 from 'head=`git --git-dir=.../.git rev-parse --verify --short INVALID 2> /dev/null`'

Cope with both use cases.

Fixes: 4153739 ("classes: make localversion classes deterministic")
Signed-off-by: Max Krummenacher <[email protected]>
@otavio otavio merged commit 92a1b94 into Freescale:master Dec 4, 2023
Copy link
Contributor

github-actions bot commented Dec 4, 2023

Backport failed for kirkstone, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally.

git fetch origin kirkstone
git worktree add -d .worktree/backport-1696-to-kirkstone origin/kirkstone
cd .worktree/backport-1696-to-kirkstone
git checkout -b backport-1696-to-kirkstone
ancref=$(git merge-base 0fc1504fb3c8721f80182f5ec54644efe1f1e842 16a356ef0d5d62090e84530e125ea6952994aaeb)
git cherry-pick -x $ancref..16a356ef0d5d62090e84530e125ea6952994aaeb

Copy link
Contributor

github-actions bot commented Dec 4, 2023

Successfully created backport PR #1697 for kirkstone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants