Pass in the platform options when loading dependencies

Reviewed By: @jingc

Differential Revision: D2488262
This commit is contained in:
Amjad Masad 2015-09-28 22:40:56 -07:00 committed by facebook-github-bot-4
parent c74f1e9bc9
commit 2cf80d481b
2 changed files with 26 additions and 4 deletions

View File

@ -47,9 +47,9 @@ exports.buildBundleFromUrl = function(options, reqUrl) {
});
};
exports.getDependencies = function(options, main) {
exports.getDependencies = function(options, bundleOptions) {
var server = createServer(options);
return server.getDependencies(main)
return server.getDependencies(bundleOptions)
.then(function(r) {
server.end();
return r.dependencies;

View File

@ -96,6 +96,21 @@ const bundleOpts = declareOpts({
}
});
const dependencyOpts = declareOpts({
platform: {
type: 'string',
required: true,
},
dev: {
type: 'boolean',
default: true,
},
entryFile: {
type: 'string',
required: true,
},
});
class Server {
constructor(options) {
const opts = validateOpts(options);
@ -174,8 +189,15 @@ class Server {
return this.buildBundle(options);
}
getDependencies(main) {
return this._bundler.getDependencies(main);
getDependencies(options) {
return Promise.resolve().then(() => {
const opts = dependencyOpts(options);
return this._bundler.getDependencies(
opts.entryFile,
opts.dev,
opts.platform,
);
});
}
_onFileChange(type, filepath, root) {