BREAKING: metro: rename 'unbundle' to 'ram bundle'

Summary:
@public

Let's get rid of the "unbundle" terminology and instead use "RAM bundle", short for "Random Access Bundle" format. THIS IS A BREAKING CHANGE FOR OSS, as the command becomes `ram-bundle` instead of `unbundle`. It realy shouldn't be a command to start with (only a "format" specifier for the `bundle` command), but I don't want to do that change at this point.

Reviewed By: davidaurelio

Differential Revision: D8894433

fbshipit-source-id: 5565f9ae94c7c2d7f6b25f95ae45b64f27f6aec8
This commit is contained in:
Jean Lauliac 2018-07-20 09:17:21 -07:00 committed by Facebook Github Bot
parent 583cb137b8
commit ebf5aeab28
3 changed files with 45 additions and 13 deletions

View File

@ -0,0 +1,34 @@
/**
* Copyright (c) 2015-present, Facebook, Inc.
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @format
*/
'use strict';
const bundleWithOutput = require('./bundle').withOutput;
const bundleCommandLineArgs = require('./bundleCommandLineArgs');
const outputUnbundle = require('metro/src/shared/output/unbundle');
/**
* Builds the bundle starting to look for dependencies at the given entry path.
*/
function ramBundle(argv, config, args) {
return bundleWithOutput(argv, config, args, outputUnbundle);
}
module.exports = {
name: 'ram-bundle',
description:
'builds javascript as a "Random Access Module" bundle for offline use',
func: ramBundle,
options: bundleCommandLineArgs.concat({
command: '--indexed-ram-bundle',
description:
'Force the "Indexed RAM" bundle file format, even when building for android',
default: false,
}),
};

View File

@ -9,25 +9,22 @@
'use strict';
const bundleWithOutput = require('./bundle').withOutput;
const bundleCommandLineArgs = require('./bundleCommandLineArgs');
const outputUnbundle = require('metro/src/shared/output/unbundle');
/**
* Builds the bundle starting to look for dependencies at the given entry path.
*/
function unbundle(argv, config, args) {
return bundleWithOutput(argv, config, args, outputUnbundle);
}
module.exports = {
name: 'unbundle',
description: 'builds javascript as "unbundle" for offline use',
func: unbundle,
description: 'Deprecated. Renamed to `ram-bundle`.',
func: () => {
throw new Error(
'The `unbundle` command has been renamed `ram-bundle` to better ' +
'represent the actual functionality. `ram` mean "Random Access ' +
'Module", this particular format of bundle. Functionality remained ' +
'unchanged.',
);
},
options: bundleCommandLineArgs.concat({
command: '--indexed-unbundle',
description:
'Force indexed unbundle file format, even when building for android',
description: 'Deprecated. Renamed to `--indexed-ram-bundle`.',
default: false,
}),
};

View File

@ -42,6 +42,7 @@ const documentedCommands = [
require('./library/library'),
require('./bundle/bundle'),
require('./bundle/unbundle'),
require('./bundle/ramBundle'),
require('./eject/eject'),
require('./link/link'),
require('./link/unlink'),