<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>2951</bug_id>
          
          <creation_ts>2012-08-16 01:37:37 +0000</creation_ts>
          <short_desc>Review and remove unnecessary uses of _prepend/_append</short_desc>
          <delta_ts>2026-09-10 15:15:00 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>OE-Core</product>
          <component>core</component>
          <version>unspecified</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard> </status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>6.99</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Christopher Larson">kergoth</reporter>
          <assigned_to name="Unassigned">unassigned</assigned_to>
          <cc>bluelightning</cc>
    
    <cc>koansoftware</cc>
    
    <cc>loc.nguyen.wt</cc>
    
    <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
    
    <cc>nayakksanjay</cc>
    
    <cc>randy.macleod</cc>
    
    <cc>ross.burton</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>24079</commentid>
    <comment_count>0</comment_count>
    <who name="Christopher Larson">kergoth</who>
    <bug_when>2012-08-16 01:37:37 +0000</bug_when>
    <thetext>These postponed/lazy operations should only used when necessary, so +=/=+/.=/=. should be preferred for non-conditional operations.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90604</commentid>
    <comment_count>1</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2021-06-10 15:14:20 +0000</bug_when>
    <thetext>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</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90961</commentid>
    <comment_count>2</comment_count>
    <who name="Marco Cavallini">koansoftware</who>
    <bug_when>2021-07-20 07:19:31 +0000</bug_when>
    <thetext>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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91057</commentid>
    <comment_count>3</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2021-07-29 15:36:30 +0000</bug_when>
    <thetext>Chris can you please provide some examples and guidance?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91227</commentid>
    <comment_count>4</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2021-08-19 15:09:23 +0000</bug_when>
    <thetext>Chris won&apos;t have time to work on this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>96935</commentid>
    <comment_count>5</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2023-10-24 14:24:46 +0000</bug_when>
    <thetext>Bulk move from 4.99 or 0.00 to 5.99</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97219</commentid>
    <comment_count>6</comment_count>
    <who name="Loc Nguyen">loc.nguyen.wt</who>
    <bug_when>2023-11-08 03:58:55 +0000</bug_when>
    <thetext>Hi all, I&apos;m a newcomer and would like to support for this bugfix. In my understanding, we will need to replace &quot;:prepend/:append&quot; to &quot;+=/=+/.=/=.&quot;. Any suspicious case should be reported to this Bugzilla. Is my understanding correct?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97270</commentid>
    <comment_count>7</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2023-11-16 20:09:08 +0000</bug_when>
    <thetext>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.
&quot;A journey of a thousand miles begins with a single step&quot; and sometimes you start out in the wrong direction! ;-)  Make a small change, do some testing, send it out for review and iterate.
Thanks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98509</commentid>
    <comment_count>8</comment_count>
    <who name="K Sanjay Nayak">nayakksanjay</who>
    <bug_when>2024-03-17 16:01:17 +0000</bug_when>
    <thetext>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:)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98534</commentid>
    <comment_count>9</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2024-03-19 18:18:12 +0000</bug_when>
    <thetext>As I was saying before, start small and simple and learn from feedback.

Ross Burton was saying that he&apos;d be careful about what appends get changed so why don&apos;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 &quot;DEPEND.*:append =&quot; meta/recipes*  | rg -v &quot;@&quot;
meta/recipes-extended/baremetal-example/baremetal-helloworld_git.bb:DEPENDS:qemux86:append = &quot; nasm-native&quot;
meta/recipes-extended/less/less_643.bb:RDEPENDS:${PN}-ptest:append = &quot; perl-module-getopt-std perl-module-cwd locale-base-en-us&quot;
meta/recipes-graphics/waffle/waffle_1.7.2.bb:DEPENDS:append = &quot; python3&quot;
meta/recipes-gnome/librsvg/librsvg_2.57.1.bb:BASEDEPENDS:append = &quot; cargo-native&quot;
meta/recipes-support/ptest-runner/ptest-runner_2.4.3.bb:RDEPENDS:${PN}:append = &quot; pstree&quot;
meta/recipes-support/libgpg-error/libgpg-error_1.48.bb:RDEPENDS:${PN}-ptest:append = &quot; make bash&quot;
meta/recipes-bsp/grub/grub-efi_2.12.bb:DEPENDS:append = &quot; grub-native&quot;
meta/recipes-core/sysvinit/sysvinit_3.04.bb:DEPENDS:append = &quot; update-rc.d-native base-passwd virtual/crypt&quot;
meta/recipes-core/initscripts/initscripts_1.0.bb:DEPENDS:append = &quot; update-rc.d-native&quot;

I&apos;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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99637</commentid>
    <comment_count>10</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2024-09-10 18:56:49 +0000</bug_when>
    <thetext>Sanjay, do you plan to work on this enhancement this fall?
If so please move it to 5.2-M2. Thanks, Randy</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>106795</commentid>
    <comment_count>11</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2026-09-10 15:15:00 +0000</bug_when>
    <thetext>Sanjay moved to unassigned, Take back if you work on it.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>