mirror of
https://github.com/facebook/react-native.git
synced 2025-11-01 09:14:26 +00:00
Adam Miskiewicz
98e5e2b427
Summary:
As spicyj mentioned in commit 6a838a4, the ideal state of affairs when it comes to consuming `react` and `fbjs` from NPM is for the packager not to have knowledge of either package. This PR addresses the `fbjs` part of that, and relies on https://github.com/facebook/fbjs/pull/95. **DO NOT MERGE** until #95 (or a variation) is in `fbjs` and is released to npm.
This PR does several things:
1. Adds stub modules within RN that expose `fbjs` modules to be required using Haste. After discussing a few ideas with spicyj, this seemed like a good option to keep internal FB devs happy (and not make them change the way they write JS), but allow for removing packager complexity and fit in better with the NPM ecosystem. Note -- it skips stubbing `fetch`, `ExecutionEnvironment`, and `ErrorUtils`, due to the fact that these need to have Native specific implementations, and there's no reason for those implementations to exist in `fbjs`.
2. Removes the modules that were previously being used in lieu of their `fbjs` eq
Closes https://github.com/facebook/react-native/pull/5084
Reviewed By: bestander
Differential Revision: D2803288
Pulled By: davidaurelio
fb-gh-sync-id: fd257958ee2f8696eebe9048c1e7628c168bf4a2
shipit-source-id: fd257958ee2f8696eebe9048c1e7628c168bf4a2
86 lines
2.5 KiB
JavaScript
86 lines
2.5 KiB
JavaScript
/**
|
|
* Copyright (c) 2015-present, Facebook, Inc.
|
|
* All rights reserved.
|
|
*
|
|
* This source code is licensed under the BSD-style license found in the
|
|
* LICENSE file in the root directory of this source tree. An additional grant
|
|
* of patent rights can be found in the PATENTS file in the same directory.
|
|
*/
|
|
'use strict';
|
|
|
|
var path = require('path');
|
|
|
|
// Don't forget to everything listed here to `package.json`
|
|
// modulePathIgnorePatterns.
|
|
var sharedBlacklist = [
|
|
/node_modules[/\\]react[/\\]dist[/\\].*/,
|
|
'node_modules/react/lib/React.js',
|
|
'node_modules/react/lib/ReactDOM.js',
|
|
|
|
// Those conflicts with the ones in fbjs/. We need to blacklist the
|
|
// internal version otherwise they won't work in open source.
|
|
'downstream/core/CSSCore.js',
|
|
'downstream/core/TouchEventUtils.js',
|
|
'downstream/core/camelize.js',
|
|
'downstream/core/createArrayFromMixed.js',
|
|
'downstream/core/createNodesFromMarkup.js',
|
|
'downstream/core/dom/containsNode.js',
|
|
'downstream/core/dom/focusNode.js',
|
|
'downstream/core/dom/getActiveElement.js',
|
|
'downstream/core/dom/getUnboundedScrollPosition.js',
|
|
'downstream/core/dom/isNode.js',
|
|
'downstream/core/dom/isTextNode.js',
|
|
'downstream/core/emptyFunction.js',
|
|
'downstream/core/emptyObject.js',
|
|
'downstream/core/getMarkupWrap.js',
|
|
'downstream/core/hyphenate.js',
|
|
'downstream/core/hyphenateStyleName.js',
|
|
'downstream/core/invariant.js',
|
|
'downstream/core/nativeRequestAnimationFrame.js',
|
|
'downstream/core/toArray.js',
|
|
|
|
/website\/node_modules\/.*/,
|
|
|
|
// TODO(jkassens, #9876132): Remove this rule when it's no longer needed.
|
|
'Libraries/Relay/relay/tools/relayUnstableBatchedUpdates.js',
|
|
];
|
|
|
|
var platformBlacklists = {
|
|
web: [
|
|
'.ios.js',
|
|
'.android.js',
|
|
],
|
|
ios: [
|
|
'.web.js',
|
|
'.android.js',
|
|
],
|
|
android: [
|
|
'.web.js',
|
|
'.ios.js',
|
|
],
|
|
};
|
|
|
|
function escapeRegExp(pattern) {
|
|
if (Object.prototype.toString.call(pattern) === '[object RegExp]') {
|
|
return pattern.source.replace(/\//g, path.sep);
|
|
} else if (typeof pattern === 'string') {
|
|
var escaped = pattern.replace(/[\-\[\]\{\}\(\)\*\+\?\.\\\^\$\|]/g, '\\$&');
|
|
// convert the '/' into an escaped local file separator
|
|
return escaped.replace(/\//g,'\\' + path.sep);
|
|
} else {
|
|
throw new Error('Unexpected packager blacklist pattern: ' + pattern);
|
|
}
|
|
}
|
|
|
|
function blacklist(platform, additionalBlacklist) {
|
|
return new RegExp('(' +
|
|
(additionalBlacklist || []).concat(sharedBlacklist)
|
|
.concat(platformBlacklists[platform] || [])
|
|
.map(escapeRegExp)
|
|
.join('|') +
|
|
')$'
|
|
);
|
|
}
|
|
|
|
module.exports = blacklist;
|