]> code.ossystems Code Review - openembedded-core.git/commit
bitbake/fetcher: Deal with a ton of different bugs
authorRichard Purdie <rpurdie@linux.intel.com>
Tue, 19 Oct 2010 11:30:21 +0000 (12:30 +0100)
committerRichard Purdie <rpurdie@linux.intel.com>
Tue, 19 Oct 2010 11:47:45 +0000 (12:47 +0100)
commit05c8ee60f164bec5a34e432312c0e65e927b40cf
tree9a4ac23d7e58259ecaaebd11eded27c36c4604be
parent1077021f7010388e889d85630b13c4c7901d7718
bitbake/fetcher: Deal with a ton of different bugs

The more we try and patch up the fetcher code, the more things break. The
code blocks in question are practically unreadable and are full of corner
cases where fetching could fail. In summary the issues noticed included:

a) Always fetching strange broken urls from the premirror for "noclone"
   git repositories
b) Not creating or rewriting .md5 stamp files inconsistently
c) Always fetching git source mirror tarballs from the premirror even
   if they already exist but the checkout directory does now
d) Passing "None" values to os.access() and os.path.extsts() checks under
   certain circumstances
e) Not using fetched git mirror tarballs if the preexist and always
   try and fetch them.

This patch rewrites the sections of code in question to be simpler and
more readable, fixing the above problems and most likely other odd
corner cases.

Signed-off-by: Richard Purdie <rpurdie@linux.intel.com>
bitbake/lib/bb/fetch/__init__.py
bitbake/lib/bb/fetch/git.py