diff options
author | Florian Dold <florian.dold@gmail.com> | 2017-04-20 03:09:25 +0200 |
---|---|---|
committer | Florian Dold <florian.dold@gmail.com> | 2017-04-24 16:14:29 +0200 |
commit | 82f2b76e25a4a67e01ec67e5ebe39d14ad771ea8 (patch) | |
tree | 965f6eb89b84d65a62b49008fd972c004832ccd1 /node_modules/isexe | |
parent | e6e0cbc387c2a77b48e4065c229daa65bf1aa0fa (diff) | |
download | wallet-core-82f2b76e25a4a67e01ec67e5ebe39d14ad771ea8.tar.xz |
Reorganize module loading.
We now use webpack instead of SystemJS, effectively bundling modules
into one file (plus commons chunks) for every entry point. This results
in a much smaller extension size (almost half). Furthermore we use
yarn/npm even for extension run-time dependencies. This relieves us
from manually vendoring and building dependencies. It's also easier to
understand for new developers familiar with node.
Diffstat (limited to 'node_modules/isexe')
-rw-r--r-- | node_modules/isexe/README.md | 14 | ||||
-rw-r--r-- | node_modules/isexe/access.js | 15 | ||||
-rw-r--r-- | node_modules/isexe/index.js | 2 | ||||
-rw-r--r-- | node_modules/isexe/mode.js | 10 | ||||
-rw-r--r-- | node_modules/isexe/package.json | 9 | ||||
-rw-r--r-- | node_modules/isexe/test/basic.js | 10 | ||||
-rw-r--r-- | node_modules/isexe/windows.js | 14 |
7 files changed, 40 insertions, 34 deletions
diff --git a/node_modules/isexe/README.md b/node_modules/isexe/README.md index 30995ad7d..35769e844 100644 --- a/node_modules/isexe/README.md +++ b/node_modules/isexe/README.md @@ -1,9 +1,9 @@ # isexe -Minimal module to check if a file is executable. +Minimal module to check if a file is executable, and a normal file. -Uses `fs.access` if available, and tests against the `PATHEXT` -environment variable on Windows. +Uses `fs.stat` and tests against the `PATHEXT` environment variable on +Windows. ## USAGE @@ -34,8 +34,8 @@ var isExe = isexe.sync('maybe-missing-file', { ignoreErrors: true }) Check if the path is executable. If no callback provided, and a global `Promise` object is available, then a Promise will be returned. -Will raise whatever errors may be raised by `fs.access` or `fs.stat`, -unless `options.ignoreErrors` is set to true. +Will raise whatever errors may be raised by `fs.stat`, unless +`options.ignoreErrors` is set to true. ### `isexe.sync(path, [options])` @@ -45,7 +45,7 @@ Same as `isexe` but returns the value and throws any errors raised. * `ignoreErrors` Treat all errors as "no, this is not executable", but don't raise them. -* `uid` Number to use as the user id when using the `mode` approach. -* `gid` Number to use as the group id when using the `mode` approach. +* `uid` Number to use as the user id +* `gid` Number to use as the group id * `pathExt` List of path extensions to use instead of `PATHEXT` environment variable on Windows. diff --git a/node_modules/isexe/access.js b/node_modules/isexe/access.js deleted file mode 100644 index e67b28bd6..000000000 --- a/node_modules/isexe/access.js +++ /dev/null @@ -1,15 +0,0 @@ -module.exports = isexe -isexe.sync = sync - -var fs = require('fs') - -function isexe (path, _, cb) { - fs.access(path, fs.X_OK, function (er) { - cb(er, !er) - }) -} - -function sync (path, _) { - fs.accessSync(path, fs.X_OK) - return true -} diff --git a/node_modules/isexe/index.js b/node_modules/isexe/index.js index ff8ef113b..553fb32b1 100644 --- a/node_modules/isexe/index.js +++ b/node_modules/isexe/index.js @@ -2,8 +2,6 @@ var fs = require('fs') var core if (process.platform === 'win32' || global.TESTING_WINDOWS) { core = require('./windows.js') -} else if (typeof fs.access === 'function') { - core = require('./access.js') } else { core = require('./mode.js') } diff --git a/node_modules/isexe/mode.js b/node_modules/isexe/mode.js index 204428072..1995ea4a0 100644 --- a/node_modules/isexe/mode.js +++ b/node_modules/isexe/mode.js @@ -4,13 +4,17 @@ isexe.sync = sync var fs = require('fs') function isexe (path, options, cb) { - fs.stat(path, function (er, st) { - cb(er, er ? false : checkMode(st, options)) + fs.stat(path, function (er, stat) { + cb(er, er ? false : checkStat(stat, options)) }) } function sync (path, options) { - return checkMode(fs.statSync(path), options) + return checkStat(fs.statSync(path), options) +} + +function checkStat (stat, options) { + return stat.isFile() && checkMode(stat, options) } function checkMode (stat, options) { diff --git a/node_modules/isexe/package.json b/node_modules/isexe/package.json index c86fe7493..e45268944 100644 --- a/node_modules/isexe/package.json +++ b/node_modules/isexe/package.json @@ -1,6 +1,6 @@ { "name": "isexe", - "version": "1.1.2", + "version": "2.0.0", "description": "Minimal module to check if a file is executable.", "main": "index.js", "directories": { @@ -9,10 +9,13 @@ "devDependencies": { "mkdirp": "^0.5.1", "rimraf": "^2.5.0", - "tap": "^5.1.2" + "tap": "^10.3.0" }, "scripts": { - "test": "tap test/*.js --branches=100 --statements=100 --functions=100 --lines=100" + "test": "tap test/*.js --100", + "preversion": "npm test", + "postversion": "npm publish", + "postpublish": "git push origin --all; git push origin --tags" }, "author": "Isaac Z. Schlueter <i@izs.me> (http://blog.izs.me/)", "license": "ISC", diff --git a/node_modules/isexe/test/basic.js b/node_modules/isexe/test/basic.js index 969fc9a0a..d926df64b 100644 --- a/node_modules/isexe/test/basic.js +++ b/node_modules/isexe/test/basic.js @@ -207,5 +207,15 @@ function runTest (t, options) { }) }) + t.test('directory is not executable', function (t) { + isexe(__dirname, options, function (er, is) { + if (er) { + throw er + } + t.notOk(is) + t.end() + }) + }) + t.end() } diff --git a/node_modules/isexe/windows.js b/node_modules/isexe/windows.js index aba8561f3..34996734d 100644 --- a/node_modules/isexe/windows.js +++ b/node_modules/isexe/windows.js @@ -24,13 +24,19 @@ function checkPathExt (path, options) { return false } +function checkStat (stat, path, options) { + if (!stat.isSymbolicLink() && !stat.isFile()) { + return false + } + return checkPathExt(path, options) +} + function isexe (path, options, cb) { - fs.stat(path, function (er, st) { - cb(er, er ? false : checkPathExt(path, options)) + fs.stat(path, function (er, stat) { + cb(er, er ? false : checkStat(stat, path, options)) }) } function sync (path, options) { - fs.statSync(path) - return checkPathExt(path, options) + return checkStat(fs.statSync(path), path, options) } |