Skip to content

Commit

Permalink
Use a wrapper NPM package for double-ended-queue.
Browse files Browse the repository at this point in the history
Most immediately, this techique serves as a workaround for
npm/read-installed#40, since the latest version
of [double-ended-queue](https://www.npmjs.com/package/double-ended-queue)
is still a prerelease version. This issue is going to rear its head the
next time we rebuild node for the dev bundle, and it will be very
difficult to diagnose the problem later if we don't fix it now.

Longer-term, we could move things like `Meteor._SynchronousQueue` into the
meteor-deque package, but that's a discussion for a different commit.
  • Loading branch information
benjamn committed Jun 23, 2015
1 parent e2616e8 commit 5de06ad
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
9 changes: 7 additions & 2 deletions packages/meteor/.npm/package/npm-shrinkwrap.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/meteor/fiber_helpers.js
Expand Up @@ -14,7 +14,7 @@ Meteor._noYieldsAllowed = function (f) {
}
};

Meteor._DoubleEndedQueue = Npm.require('double-ended-queue');
Meteor._DoubleEndedQueue = Npm.require('meteor-deque');

// Meteor._SynchronousQueue is a queue which runs task functions serially.
// Tasks are assumed to be synchronous: ie, it's assumed that they are
Expand Down
2 changes: 1 addition & 1 deletion packages/meteor/package.js
Expand Up @@ -11,7 +11,7 @@ Package.registerBuildPlugin({
});

Npm.depends({
"double-ended-queue": "2.1.0-0"
"meteor-deque": "2.1.0"
});

Package.onUse(function (api) {
Expand Down

0 comments on commit 5de06ad

Please sign in to comment.