[font] Only include supported font files (expo#27002)

Azzadd · Feb 9, 2024 · 07286b1 · 07286b1
1 parent a1906b7
commit 07286b1
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 4 deletions.
diff --git a/packages/expo-font/CHANGELOG.md b/packages/expo-font/CHANGELOG.md
@@ -8,6 +8,8 @@
 
 ### 🐛 Bug fixes
 
+- Only include supported font files when using the plugin. ([#27002](https://github.com/expo/expo/pull/27002) by [@alanjhughes](https://github.com/alanjhughes))
+
 ### 💡 Others
 
 - Remove most of Constants.appOwnership. ([#26313](https://github.com/expo/expo/pull/26313) by [@wschurman](https://github.com/wschurman))

diff --git a/packages/expo-font/plugin/build/utils.js b/packages/expo-font/plugin/build/utils.js
diff --git a/packages/expo-font/plugin/build/withFontsAndroid.js b/packages/expo-font/plugin/build/withFontsAndroid.js
diff --git a/packages/expo-font/plugin/src/utils.ts b/packages/expo-font/plugin/src/utils.ts
@@ -12,5 +12,7 @@ export async function resolveFontPaths(fonts: string[], projectRoot: string) {
     }
     return [resolvedPath];
   });
-  return (await Promise.all(promises)).flat();
+  return (await Promise.all(promises))
+    .flat()
+    .filter((p) => p.endsWith('.ttf') || p.endsWith('.otf'));
 }
diff --git a/packages/expo-font/plugin/src/withFontsAndroid.ts b/packages/expo-font/plugin/src/withFontsAndroid.ts
@@ -17,7 +17,9 @@ export const withFontsAndroid: ConfigPlugin<string[]> = (config, fonts) => {
           );
           await fs.mkdir(fontsDir, { recursive: true });
           const output = path.join(fontsDir, path.basename(asset));
-          await fs.copyFile(asset, output);
+          if (output.endsWith('.ttf') || output.endsWith('.otf')) {
+            await fs.copyFile(asset, output);
+          }
         })
       );
       return config;