<?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>13533</bug_id>
          
          <creation_ts>2019-09-17 00:22:00 +0000</creation_ts>
          <short_desc>Devtool finish on _git package with SRCPV in PV points to wrong WORKDIR</short_desc>
          <delta_ts>2026-10-26 17:20:04 +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>Scripts and Tools</component>
          <version>3.0</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>IN PROGRESS REVIEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>6.2 M3</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jaewon Lee">jaewon</reporter>
          <assigned_to name="David Reyna">david.reyna</assigned_to>
          <cc>alex.kanavin</cc>
    
    <cc>bluelightning</cc>
    
    <cc>bruce.ashfield</cc>
    
    <cc>ccasciato</cc>
    
    <cc>randy.macleod</cc>
    
    <cc>richard.purdie</cc>
    
    <cc>ross.burton</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Don&apos;t know</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>84987</commentid>
    <comment_count>0</comment_count>
    <who name="Jaewon Lee">jaewon</who>
    <bug_when>2019-09-17 00:22:00 +0000</bug_when>
    <thetext>Taking blktrace package as an example,

First do a devtool modify blktrace,

workdir while devtool workspace exists for blktrace will be for ex. blktrace/1.2.0+git999-r0  because in externalsrc.bbclass if  SRCPV is in PV, SRCPV is changed to 999. 

but in devtool finish, this data store: rd = parse_recipe(config, tinfoil, args.recipename, True)

will not reflect that change if you do a rd.getVar(&apos;WORKDIR&apos;), SRCPV will not be changed to 999, so you can point to build artifacts in devtool finish using WORKDIR.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>84988</commentid>
    <comment_count>1</comment_count>
    <who name="Jaewon Lee">jaewon</who>
    <bug_when>2019-09-17 00:42:31 +0000</bug_when>
    <thetext>After investigation, its due to filter_workspace being default true for parse_recipe definition in devtool/__init__.py

for modify I get the reasoning for filtering out devtool workspace, but is there a reason for filtering out devtool workspace for finish?
if I turn filtering off for the parse_recipe call in finish, it behaves as expected

we need WORKDIR to be correct during finish. just as an example, this ccmake patch where is looking for generated cfg or cmake files and this wont work in cases where SRCPV is in PV

https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=89c4424d6d875e102ca324d3b81426b4466dccc5</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>85036</commentid>
    <comment_count>2</comment_count>
    <who name="Jaewon Lee">jaewon</who>
    <bug_when>2019-09-20 16:43:35 +0000</bug_when>
    <thetext>Patch sent on mailing list
[OE-core][master][PATCH] standard.py: Not filtering devtool workspace for devtool finish</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>85037</commentid>
    <comment_count>3</comment_count>
    <who name="Jaewon Lee">jaewon</who>
    <bug_when>2019-09-20 18:10:05 +0000</bug_when>
    <thetext>Also if this change gets merged, we should take another look at devtool cfg generation because right now it gets written to $S

https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=73bf792c858752008fe174840dad1171eee86635

quote from the commit:
&quot;${S}/.config will be a symlink to ${B}/.config. We need to do this as
    devtool is not able to access ${B} because ${B} is set in a .bbappend in
    the workspace layer which is not parsed by devtool itself.&quot;

but we should align with bitbake method of writing to B if devtool can access correct $B</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>85561</commentid>
    <comment_count>4</comment_count>
    <who name="Alexander Kanavin">alex.kanavin</who>
    <bug_when>2019-11-18 13:11:39 +0000</bug_when>
    <thetext>Please note:
http://lists.openembedded.org/pipermail/openembedded-core/2019-November/289311.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>86829</commentid>
    <comment_count>5</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2020-03-26 15:12:31 +0000</bug_when>
    <thetext>Please retest with the latest tinfoil datastore connector changes.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90174</commentid>
    <comment_count>6</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2021-04-29 15:28:06 +0000</bug_when>
    <thetext>ping</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90943</commentid>
    <comment_count>7</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2021-07-15 15:09:41 +0000</bug_when>
    <thetext>Any update?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>95789</commentid>
    <comment_count>8</comment_count>
    <who name="David Reyna">david.reyna</who>
    <bug_when>2023-06-22 09:42:40 +0000</bug_when>
    <thetext>I have a new intern to help understand and resolve this issue.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>