ref: Clean up code

It is a bit weird to have this block defining two variables inside the definition of another variable.

I think the following is a bit more readable. It also keeps the code DRY.

Author: szokeasaurusrex

src/commands/mobile_app/upload.rs

@@ -276,21 +276,13 @@ fn normalize_directory(path: &Path) -> Result<TempFile> {
         // Need to set the last modified time to a fixed value to ensure consistent checksums
         // This is important as an optimization to avoid re-uploading the same chunks if they're already on the server
         // but the last modified time being different will cause checksums to be different.
-        #[cfg(not(windows))]
-        let options = {
-            let metadata = fs::metadata(&entry_path)?;
-            let mode = metadata.permissions().mode();
-            SimpleFileOptions::default()
-                .compression_method(zip::CompressionMethod::Stored)
-                .last_modified_time(DateTime::default())
-                .unix_permissions(mode)
-        };
-
-        #[cfg(windows)]
         let options = SimpleFileOptions::default()
             .compression_method(zip::CompressionMethod::Stored)
             .last_modified_time(DateTime::default());
 
+        #[cfg(not(windows))]
+        let options = options.unix_permissions(fs::metadata(&entry_path)?.permissions().mode());
+
         zip.start_file(relative_path.to_string_lossy(), options)?;
         let file_byteview = ByteView::open(&entry_path)?;
         zip.write_all(file_byteview.as_slice())?;