From e69b81313f68268c4425579b8196d771f042f858 Mon Sep 17 00:00:00 2001 From: Jean Lauliac Date: Mon, 8 May 2017 04:28:26 -0700 Subject: [PATCH] packager: ModuleGraph: include all possible asset types Summary: Afaik. the list of asset types is hardcoded for the Buck worker so far, and I'm not sure how that would be customizable. For now, let's include all the default asset extensions. Reviewed By: davidaurelio Differential Revision: D5002607 fbshipit-source-id: 41069e817d2b73156bca684bc2f73077132928a7 --- packager/src/ModuleGraph/worker/transform-module.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/packager/src/ModuleGraph/worker/transform-module.js b/packager/src/ModuleGraph/worker/transform-module.js index 2e7cf2e1968..e15a3993d69 100644 --- a/packager/src/ModuleGraph/worker/transform-module.js +++ b/packager/src/ModuleGraph/worker/transform-module.js @@ -12,8 +12,10 @@ const babel = require('babel-core'); const collectDependencies = require('./collect-dependencies'); +const defaults = require('../../../defaults'); const docblock = require('../../node-haste/DependencyGraph/docblock'); const generate = require('./generate'); +const path = require('path'); const series = require('async/series'); const {basename} = require('path'); @@ -39,12 +41,14 @@ const defaultVariants = {default: {}}; const moduleFactoryParameters = ['global', 'require', 'module', 'exports']; const polyfillFactoryParameters = ['global']; +const ASSET_EXTENSIONS = new Set(defaults.assetExts); + function transformModule( content: Buffer, options: TransformOptions, callback: Callback, ): void { - if (options.filename.endsWith('.png')) { + if (ASSET_EXTENSIONS.has(path.extname(options.filename).substr(1))) { transformAsset(content, options, callback); return; }