URLFetchStrategy.archive does a copy vs. a move now
This commit is contained in:
		| @@ -268,7 +268,7 @@ def archive(self, destination): | |||||||
|         if not extension(destination) == extension(self.archive_file): |         if not extension(destination) == extension(self.archive_file): | ||||||
|             raise ValueError("Cannot archive without matching extensions.") |             raise ValueError("Cannot archive without matching extensions.") | ||||||
|  |  | ||||||
|         shutil.move(self.archive_file, destination) |         shutil.copy(self.archive_file, destination) | ||||||
|  |  | ||||||
|     @_needs_stage |     @_needs_stage | ||||||
|     def check(self): |     def check(self): | ||||||
|   | |||||||
| @@ -323,11 +323,8 @@ def check(self): | |||||||
|  |  | ||||||
|     def cache_local(self): |     def cache_local(self): | ||||||
|         archiveDst = join_path(os.path.abspath(spack.cache_path), self.mirror_path) |         archiveDst = join_path(os.path.abspath(spack.cache_path), self.mirror_path) | ||||||
|         if not os.path.exists(archiveDst): #tmp conditional |         mkdirp(os.path.dirname(archiveDst)) | ||||||
|             mkdirp(os.path.dirname(archiveDst)) |         self.fetcher.archive(archiveDst) | ||||||
|             # TODO: this moves the archive for URLFetchStrategy vs. a copy -  |  | ||||||
|             # edit to do a move? |  | ||||||
|             self.fetcher.archive(archiveDst) |  | ||||||
|  |  | ||||||
|  |  | ||||||
|     def expand_archive(self): |     def expand_archive(self): | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Peter Scheibel
					Peter Scheibel