<?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>11407</bug_id>
          
          <creation_ts>2017-04-24 23:05:10 +0000</creation_ts>
          <short_desc>devtool should warn user of prior download after reset</short_desc>
          <delta_ts>2025-02-14 16:48:09 +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>devtools / tool chain</component>
          <version>2.4</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>normal</bug_severity>
          <target_milestone>5.99</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Brian Kurle">brianx.j.kurle</reporter>
          <assigned_to name="Unassigned">unassigned</assigned_to>
          <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
    
    <cc>randy.macleod</cc>
    
    <cc>sgw</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>72804</commentid>
    <comment_count>0</comment_count>
    <who name="Brian Kurle">brianx.j.kurle</who>
    <bug_when>2017-04-24 23:05:10 +0000</bug_when>
    <thetext>devtool reset [-a|&lt;recipe&gt;] will warn user that the sources/&lt;recipe&gt; will be left behind.  Moreover, subsequent issuance of a &apos;devtool add&apos; will error out on the prior sources/ directory.

However, if the sources/ is removed, and a new &apos;devtool add&apos; with a corrected URI for the recipe is given, it will use the prior &quot;cached&quot; downloaded version with no failure or warning.

E.g.,

$ devtool add --version 2.4.2 mbedtls https://tls.mbed.org/download/start/mbedtls-2.4.2-apache.tgz
# (downloads a landing page, not the desired tarball)

$ devtool reset -a
NOTE: Cleaning sysroot for recipe mbedtls...
NOTE: Leaving source tree /home/bjkurlex/mbedtls/dev2/workspace/sources/mbedtls as-is; if you no longer need it then please delete it manually

An additional NOTE: here to warn the user that a prior version of the downloaded file (whether valid or not) is left as well.  

Otherwise, it is is assumed (even after removing the aforementioned sources/ directory) that a new &quot;corrected&quot; add will work.

# issuing &quot;corrected&quot; URI
$ devtool add --version 2.4.2 mbedtls https://tls.mbed.org/download/mbedtls-2.4.2-apache.tgz

This will fail and isn&apos;t obvious to a new user that the prior downloaded version is negating the corrected URI even though the sources directory previously warned was removed.

Another approach is to cache the URI associated with the download and if the URI changes (but the file does not), then it should be redownloaded.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75219</commentid>
    <comment_count>1</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2017-07-21 08:35:05 +0000</bug_when>
    <thetext>FWIW I have fixed an aspect of this which is handling of URLs that return an HTML page:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=047d37633b2d451ee4f53c635d3f48b9b0732e44

The underlying issue of a previously downloaded file with the same name is still present however.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>93588</commentid>
    <comment_count>2</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2022-06-23 15:13:07 +0000</bug_when>
    <thetext>A real bug...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>96805</commentid>
    <comment_count>3</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2023-10-24 14:24:21 +0000</bug_when>
    <thetext>Bulk move from 4.99 or 0.00 to 5.99</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>100985</commentid>
    <comment_count>4</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2025-02-14 16:46:27 +0000</bug_when>
    <thetext>bulk change to add Saul&apos;s non-WR address.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>101020</commentid>
    <comment_count>5</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2025-02-14 16:48:09 +0000</bug_when>
    <thetext>Bulk change: Remove Saul&apos;s old WR address.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>