Minor changes from code review

This commit is contained in:
Avital Oliver
2013-01-16 16:00:37 -08:00
committed by David Glasser
parent 5352db933a
commit dceef1461c
3 changed files with 3 additions and 5 deletions

View File

@@ -130,7 +130,7 @@ var files = module.exports = {
// for a packge that exists in localPackageDirs, find the directory
// in which it exists
localPackageDir: function(name) {
directoryForLocalPackage: function(name) {
var ret;
_.find(this.localPackageDirs(), function(packageDir) {
var dir = path.join(packageDir, name);

View File

@@ -97,7 +97,7 @@ _.extend(Package.prototype, {
// - $METEOR/packages
// @returns {Boolean} was the package found in any local package sets?
initFromLocalPackages: function (name) {
var packageDir = files.localPackageDir(name);
var packageDir = files.directoryForLocalPackage(name);
if (packageDir) {
this._initFromPackageDir(name, packageDir);
return true;
@@ -234,8 +234,6 @@ _.extend(Package.prototype, {
}
});
// (OLD COMMENT?) in the future, this could be an on-disk cache that
// tracks mtimes.
var compiledPackages = {};
var packages = module.exports = {

View File

@@ -339,7 +339,7 @@ var DependencyWatcher = function (deps, app_dir, relativeFiles, on_change) {
self.specific_files = {};
for (var pkg in (deps.packages || {})) {
_.each(deps.packages[pkg], function (file) {
self.specific_files[path.join(files.localPackageDir(pkg), file)]
self.specific_files[path.join(files.directoryForLocalPackage(pkg), file)]
= true;
});
};