Bug 3073 - Fetcher can still get into a circular mirror loop
Summary: Fetcher can still get into a circular mirror loop
Status: RESOLVED FIXED
Alias: None
Product: BitBake
Classification: Build System, Metadata & Runtime
Component: bitbake (show other bugs)
Version: unspecified
Hardware: All Multiple
: Undecided normal
Target Milestone: ---
Assignee: Paul Eggleton
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2012-09-07 11:24 UTC by Paul Eggleton
Modified: 2012-09-28 17:20 UTC (History)
3 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Eggleton 2012-09-07 11:24:21 UTC 
If you have an entry in MIRRORS which points to itself with a slash appended e.g. the following gnutls entry which we are about to remove:

ftp://ftp.gnutls.org/pub/gnutls ftp://ftp.gnutls.org/pub/gnutls/ \n \

If matched, such entries result in repeated looping adding a slash to the path each time until the stack space is exhausted. We were only not hitting this issue with current master because poky rev 188dff3daf048e369bcab0d1db2964490e43d734 changed SRC_URI in gnutls.inc to use GNU_MIRROR so this bogus mirror entry is not be used.
Comment 2 Kevin Strasser 2012-09-10 05:53:15 UTC 
Would it be possible to backport this patch to denzil?
Comment 3 Paul Eggleton 2012-09-14 15:10:02 UTC 
Sure, I've just sent a patch marked for denzil to remove the bogus mirror entry. I was going to do the same for the fetcher patch, but that doesn't apply cleanly and isn't absolutely needed if the mirrors entry is removed.
Comment 4 Paul Eggleton 2012-09-28 17:20:02 UTC 
The fix to remove the bogus entry has now been merged into the denzil branch as well:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?h=denzil&id=2fab410f2e0c25157bce6e9f5b2416338116ad5f