Skip to content

bitbake: Remove source directory variable in Whinlatter#324

Merged
robwoolley merged 1 commit intoros-infrastructure:masterfrom
rcwoolley:workdir-fix
Aug 12, 2025
Merged

bitbake: Remove source directory variable in Whinlatter#324
robwoolley merged 1 commit intoros-infrastructure:masterfrom
rcwoolley:workdir-fix

Conversation

@rcwoolley
Copy link

Styhead requires that S = "${WORKDIR}" be replaced with S = "${UNPACKDIR}", however S = "${WORKDIR}/git" is still supported.

Whinlatter no longer supports setting S = "${WORKDIR}/git".

See the Yocto Project release notes for details.

Styhead requires that S = "${WORKDIR}" be replaced with
S = "${UNPACKDIR}", however S = "${WORKDIR}/git" is still
supported.

Whinlatter no longer supports setting S = "${WORKDIR}/git".

See the Yocto Project release notes for details.

Signed-off-by: Rob Woolley <rob.woolley@windriver.com>
Copy link
Contributor

@robwoolley robwoolley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Styhead

https://docs.yoctoproject.org/next/migration-guides/migration-5.1.html#workdir-changes
S = "${WORKDIR}" becomes S = "${UNPACKDIR}"

Note: "S = "${WORKDIR}/git" and also deeper paths continue to work as expected without changes."

Whinlatter

https://docs.yoctoproject.org/next/migration-guides/migration-5.3.html#workdir-changes

"S = ${WORKDIR}/something no longer supported"

" S setting in recipes can and should be removed."

when "git" as the source checkout location is hardcoded "in SRC_URI, replace with BB_GIT_DEFAULT_DESTSUFFIX, otherwise replace with BP."

How to make those adjustments without tedious manual editing
https://docs.yoctoproject.org/next/migration-guides/migration-5.3.html#how-to-make-those-adjustments-without-tedious-manual-editing

@robwoolley robwoolley merged commit 392ca5d into ros-infrastructure:master Aug 12, 2025
20 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.

2 participants