These postponed/lazy operations should only used when necessary, so +=/=+/.=/=. should be preferred for non-conditional operations.
Some cases are required so this will need careful auditing so only do a few files at a time before sending for review to the list. Thanks. Randy
For a newcomer, the description might be daunting. It would be better to describe better perhaps with an example of what types of substitutions are needed.
Chris can you please provide some examples and guidance?
Chris won't have time to work on this.
Bulk move from 4.99 or 0.00 to 5.99
Hi all, I'm a newcomer and would like to support for this bugfix. In my understanding, we will need to replace ":prepend/:append" to "+=/=+/.=/=.". Any suspicious case should be reported to this Bugzilla. Is my understanding correct?
Loc, Welcome to Yocto! As was mentioned earlier, some cases of this are required so this will need careful auditing so only do a few files at a time before sending for review to the list. Arbitrarily, you could start with: poky.git on master ❯ rg :append meta/recipes-support/a* | wc -l 12 ❯ rg :prepend meta/recipes-support/a* | wc -l 3 No additional bugzilla issues need to be opened unless someone requests one in the email list review. "A journey of a thousand miles begins with a single step" and sometimes you start out in the wrong direction! ;-) Make a small change, do some testing, send it out for review and iterate. Thanks.
Hi Randy Can you please confirm is this issue is still valid? if yes, Then i would like to work on this.. Thanks for the inspiring quote! Thanks Sanjay:)
As I was saying before, start small and simple and learn from feedback. Ross Burton was saying that he'd be careful about what appends get changed so why don't you do 2 or 3 of the simpler appends from the list below, test them locally and send it to the list for review and autobuilder testing. ❯ rg "DEPEND.*:append =" meta/recipes* | rg -v "@" meta/recipes-extended/baremetal-example/baremetal-helloworld_git.bb:DEPENDS:qemux86:append = " nasm-native" meta/recipes-extended/less/less_643.bb:RDEPENDS:${PN}-ptest:append = " perl-module-getopt-std perl-module-cwd locale-base-en-us" meta/recipes-graphics/waffle/waffle_1.7.2.bb:DEPENDS:append = " python3" meta/recipes-gnome/librsvg/librsvg_2.57.1.bb:BASEDEPENDS:append = " cargo-native" meta/recipes-support/ptest-runner/ptest-runner_2.4.3.bb:RDEPENDS:${PN}:append = " pstree" meta/recipes-support/libgpg-error/libgpg-error_1.48.bb:RDEPENDS:${PN}-ptest:append = " make bash" meta/recipes-bsp/grub/grub-efi_2.12.bb:DEPENDS:append = " grub-native" meta/recipes-core/sysvinit/sysvinit_3.04.bb:DEPENDS:append = " update-rc.d-native base-passwd virtual/crypt" meta/recipes-core/initscripts/initscripts_1.0.bb:DEPENDS:append = " update-rc.d-native" I've assigned the bug to Sanjay but feel free to co-ordinate the work with others if it turns out that there are lots of changes to review and tests rather than just mechanically search and replace.
Sanjay, do you plan to work on this enhancement this fall? If so please move it to 5.2-M2. Thanks, Randy
Sanjay moved to unassigned, Take back if you work on it.