make the local options cloneable

Author: dsherret

rs_lib/src/file_fetcher/mod.rs

@@ -443,7 +443,7 @@ pub struct FetchNoFollowOptions<'a> {
   pub maybe_cache_setting: Option<&'a CacheSetting>,
 }
 
-#[derive(Debug, Default)]
+#[derive(Debug, Clone, Default)]
 pub struct FetchLocalOptions {
   pub include_mtime: bool,
 }
@@ -917,7 +917,7 @@ impl<TBlobStore: BlobStore, TSys: FileFetcherSys, THttpClient: HttpClient>
     path: &Path,
     options: &FetchLocalOptions,
   ) -> std::io::Result<File> {
-    let mut file = self.sys.fs_open(&path, &OpenOptions::new_read())?;
+    let mut file = self.sys.fs_open(path, &OpenOptions::new_read())?;
     let mtime = if options.include_mtime {
       file.fs_file_metadata().and_then(|m| m.modified()).ok()
     } else {

rs_lib/src/local.rs

@@ -471,7 +471,7 @@ impl<TSys: LocalHttpCacheSys> HttpCache for LocalHttpCache<TSys> {
                   self.global_cache.get(&global_key, maybe_checksum)?;
                 if let Some(file) = maybe_global_cache_file {
                   let content = self
-                    .transform_content_on_copy_to_local(&key.url, file.content);
+                    .transform_content_on_copy_to_local(key.url, file.content);
                   atomic_write_file_with_retries(
                     self.env(),
                     &local_file_path,