Merge pull request #465 from citibeth/efischer/160229-RemoteRootBugfix

Bug Fix: When Spack create roots around for other versions, it someti…
This commit is contained in:
Todd Gamblin 2016-03-02 10:24:17 -08:00
commit 9f6ebd7c6e

View file

@ -58,24 +58,29 @@ def get_checksums(versions, urls, **kwargs):
tty.msg("Downloading...") tty.msg("Downloading...")
hashes = [] hashes = []
for i, (url, version) in enumerate(zip(urls, versions)): i = 0
for url, version in zip(urls, versions):
stage = Stage(url) stage = Stage(url)
try: try:
stage.fetch() stage.fetch()
if i == 0 and first_stage_function: if i == 0 and first_stage_function:
first_stage_function(stage) first_stage_function(stage)
hashes.append( hashes.append((version,
spack.util.crypto.checksum(hashlib.md5, stage.archive_file)) spack.util.crypto.checksum(hashlib.md5, stage.archive_file)))
except FailedDownloadError, e: except FailedDownloadError, e:
tty.msg("Failed to fetch %s" % url) tty.msg("Failed to fetch %s" % url)
continue continue
except Exception, e:
tty.msg('Something failed on %s, skipping.\n (%s)' % (url, e))
continue
finally: finally:
if not keep_stage: if not keep_stage:
stage.destroy() stage.destroy()
i += 1
return zip(versions, hashes) return hashes