fsevents 1.2.3 → 1.2.10

package/.travis.yml

@@ -5,14 +5,13 @@ env:
     - secure: "gve1nkeKkwFEG1VAT3i+JwYyAdF0gKXwKx0uxbkBTsmm2M+0MDusohQdFLoEIkSIFktXBIDefoa7iGpLKRfG2VsZLpwJgnvnD0HqbnuR+k+W+bu7BHt4CAaR6GTllsDCjyq9zNyhUThzSnf2WNIpOEF5kHspZlbGfawURuUJH/U="
     - secure: "jqVpmWxxBVXu2X8+XJMpKH0cooc2EKz9xKL2znBfYdNafJORSXcFAVbjCX5mZmVDcgIMwDtm2+gIG4P73hzJ2e3S+y2Z9ROJGyXHa3AxUTvXHQsxqzH8coHHqB8vTvfr0t2O5aKfpvpICpSea39r0hzNoMv6Ie5SwBdqj1YY9K0="
   matrix:
+    - NODE_VERSION="v12"
+    - NODE_VERSION="v11"
     - NODE_VERSION="v10"
     - NODE_VERSION="v9"
     - NODE_VERSION="v8"
     - NODE_VERSION="v7"
     - NODE_VERSION="v6"
-    - NODE_VERSION="v5"
-    - NODE_VERSION="v4"
-    - NODE_VERSION="v0.10"
 
 before_install:
 

package/fsevents.cc

@@ -6,7 +6,6 @@
 #include "nan.h"
 #include "uv.h"
 #include "v8.h"
-#include "pthread.h"
 #include "CoreFoundation/CoreFoundation.h"
 #include "CoreServices/CoreServices.h"
 #include <iostream>
@@ -19,13 +18,7 @@ namespace fse {
     explicit FSEvents(const char *path);
     ~FSEvents();
 
-    // locking.cc
-    bool lockStarted;
-    pthread_mutex_t lockmutex;
-    void lockingStart();
-    void lock();
-    void unlock();
-    void lockingStop();
+    uv_mutex_t mutex;
 
     // async.cc
     uv_async_t async;
@@ -34,10 +27,10 @@ namespace fse {
     void asyncStop();
 
     // thread.cc
-    pthread_t thread;
+    uv_thread_t thread;
     CFRunLoopRef threadloop;
     void threadStart();
-    static void *threadRun(void *ctx);
+    static void threadRun(void *ctx);
     void threadStop();
 
     // methods.cc - internal
@@ -47,7 +40,7 @@ namespace fse {
     // Common
     CFArrayRef paths;
     std::vector<fse_event*> events;
-    static void Initialize(v8::Handle<v8::Object> exports);
+    static void Initialize(v8::Local<v8::Object> exports);
 
     // methods.cc - exposed
     static NAN_METHOD(New);
@@ -60,41 +53,36 @@ namespace fse {
 using namespace fse;
 
 FSEvents::FSEvents(const char *path)
-   : async_resource("fsevents:FSEvents"), lockStarted(false) {
+   : async_resource("fsevents:FSEvents") {
   CFStringRef dirs[] = { CFStringCreateWithCString(NULL, path, kCFStringEncodingUTF8) };
   paths = CFArrayCreate(NULL, (const void **)&dirs, 1, NULL);
   threadloop = NULL;
-  lockingStart();
+  if (uv_mutex_init(&mutex)) abort();
 }
 FSEvents::~FSEvents() {
-  lockingStop();
-
   CFRelease(paths);
+  uv_mutex_destroy(&mutex);
 }
 
 #ifndef kFSEventStreamEventFlagItemCreated
 #define kFSEventStreamEventFlagItemCreated 0x00000010
 #endif
 
-#include "src/locking.cc"
 #include "src/async.cc"
 #include "src/thread.cc"
 #include "src/constants.cc"
 #include "src/methods.cc"
 
-void FSEvents::Initialize(v8::Handle<v8::Object> exports) {
+void FSEvents::Initialize(v8::Local<v8::Object> exports) {
+  v8::Isolate* isolate = exports->GetIsolate();
+  v8::Local<v8::Context> context = isolate->GetCurrentContext();
   v8::Local<v8::FunctionTemplate> tpl = Nan::New<v8::FunctionTemplate>(FSEvents::New);
   tpl->SetClassName(Nan::New<v8::String>("FSEvents").ToLocalChecked());
   tpl->InstanceTemplate()->SetInternalFieldCount(1);
-  tpl->PrototypeTemplate()->Set(
-           Nan::New<v8::String>("start").ToLocalChecked(),
-           Nan::New<v8::FunctionTemplate>(FSEvents::Start));
-  tpl->PrototypeTemplate()->Set(
-           Nan::New<v8::String>("stop").ToLocalChecked(),
-           Nan::New<v8::FunctionTemplate>(FSEvents::Stop));
-  exports->Set(Nan::New<v8::String>("Constants").ToLocalChecked(), Constants());
-  exports->Set(Nan::New<v8::String>("FSEvents").ToLocalChecked(),
-               tpl->GetFunction());
+  Nan::SetPrototypeTemplate(tpl, "start", Nan::New<v8::FunctionTemplate>(FSEvents::Start));
+  Nan::SetPrototypeTemplate(tpl, "stop", Nan::New<v8::FunctionTemplate>(FSEvents::Stop));
+  Nan::Set(exports, Nan::New<v8::String>("Constants").ToLocalChecked(), Constants());
+  Nan::Set(exports, Nan::New<v8::String>("FSEvents").ToLocalChecked(), tpl->GetFunction(context).ToLocalChecked());
 }
 
 NODE_MODULE(fse, FSEvents::Initialize)

package/node_modules/abbrev/package.json

@@ -1,30 +1,27 @@
 {
-  "_args": [
-    [
-      "abbrev@1.1.1",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "abbrev@1.1.1",
+  "_from": "abbrev@1",
   "_id": "abbrev@1.1.1",
   "_inBundle": false,
   "_integrity": "sha512-nne9/IiQ/hzIhY6pdDnbBtz7DjPTKrY00P/zvPSm5pOFkl6xuGrGnXn/VtTNNfNtAfZ9/1RtehkszU9qcTii0Q==",
   "_location": "/abbrev",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "abbrev@1.1.1",
+    "raw": "abbrev@1",
     "name": "abbrev",
     "escapedName": "abbrev",
-    "rawSpec": "1.1.1",
+    "rawSpec": "1",
     "saveSpec": null,
-    "fetchSpec": "1.1.1"
+    "fetchSpec": "1"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/nopt"
+  ],
   "_resolved": "https://registry.npmjs.org/abbrev/-/abbrev-1.1.1.tgz",
-  "_spec": "1.1.1",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "f8f2c887ad10bf67f634f005b6987fed3179aac8",
+  "_spec": "abbrev@1",
+  "_where": "/Users/pipobscure/fsevents/node_modules/nopt",
   "author": {
     "name": "Isaac Z. Schlueter",
     "email": "i@izs.me"
@@ -32,6 +29,8 @@
   "bugs": {
     "url": "https://github.com/isaacs/abbrev-js/issues"
   },
+  "bundleDependencies": false,
+  "deprecated": false,
   "description": "Like ruby's abbrev module, but in js",
   "devDependencies": {
     "tap": "^10.1"

package/node_modules/ansi-regex/package.json

@@ -1,30 +1,27 @@
 {
-  "_args": [
-    [
-      "ansi-regex@2.1.1",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "ansi-regex@2.1.1",
+  "_from": "ansi-regex@^2.0.0",
   "_id": "ansi-regex@2.1.1",
   "_inBundle": false,
   "_integrity": "sha1-w7M6te42DYbg5ijwRorn7yfWVN8=",
   "_location": "/ansi-regex",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "ansi-regex@2.1.1",
+    "raw": "ansi-regex@^2.0.0",
     "name": "ansi-regex",
     "escapedName": "ansi-regex",
-    "rawSpec": "2.1.1",
+    "rawSpec": "^2.0.0",
     "saveSpec": null,
-    "fetchSpec": "2.1.1"
+    "fetchSpec": "^2.0.0"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/strip-ansi"
+  ],
   "_resolved": "https://registry.npmjs.org/ansi-regex/-/ansi-regex-2.1.1.tgz",
-  "_spec": "2.1.1",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "c3b33ab5ee360d86e0e628f0468ae7ef27d654df",
+  "_spec": "ansi-regex@^2.0.0",
+  "_where": "/Users/pipobscure/fsevents/node_modules/strip-ansi",
   "author": {
     "name": "Sindre Sorhus",
     "email": "sindresorhus@gmail.com",
@@ -33,6 +30,8 @@
   "bugs": {
     "url": "https://github.com/chalk/ansi-regex/issues"
   },
+  "bundleDependencies": false,
+  "deprecated": false,
   "description": "Regular expression for matching ANSI escape codes",
   "devDependencies": {
     "ava": "0.17.0",

package/node_modules/aproba/package.json

@@ -1,30 +1,27 @@
 {
-  "_args": [
-    [
-      "aproba@1.2.0",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "aproba@1.2.0",
+  "_from": "aproba@^1.0.3",
   "_id": "aproba@1.2.0",
   "_inBundle": false,
   "_integrity": "sha512-Y9J6ZjXtoYh8RnXVCMOU/ttDmk1aBjunq9vO0ta5x85WDQiQfUF9sIPBITdbiiIVcBo03Hi3jMxigBtsddlXRw==",
   "_location": "/aproba",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "aproba@1.2.0",
+    "raw": "aproba@^1.0.3",
     "name": "aproba",
     "escapedName": "aproba",
-    "rawSpec": "1.2.0",
+    "rawSpec": "^1.0.3",
     "saveSpec": null,
-    "fetchSpec": "1.2.0"
+    "fetchSpec": "^1.0.3"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/gauge"
+  ],
   "_resolved": "https://registry.npmjs.org/aproba/-/aproba-1.2.0.tgz",
-  "_spec": "1.2.0",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "6802e6264efd18c790a1b0d517f0f2627bf2c94a",
+  "_spec": "aproba@^1.0.3",
+  "_where": "/Users/pipobscure/fsevents/node_modules/gauge",
   "author": {
     "name": "Rebecca Turner",
     "email": "me@re-becca.org"
@@ -32,7 +29,9 @@
   "bugs": {
     "url": "https://github.com/iarna/aproba/issues"
   },
+  "bundleDependencies": false,
   "dependencies": {},
+  "deprecated": false,
   "description": "A ridiculously light-weight argument validator (now browser friendly)",
   "devDependencies": {
     "standard": "^10.0.3",

package/node_modules/are-we-there-yet/CHANGES.md

@@ -1,5 +1,11 @@
 Hi, figured we could actually use a changelog now:
 
+## 1.1.5 2018-05-24
+
+* [#92](https://github.com/iarna/are-we-there-yet/pull/92) Fix bug where
+  `finish` would throw errors when including `TrackerStream` objects in
+  `TrackerGroup` collections.  (@brianloveswords)
+
 ## 1.1.4 2017-04-21
 
 * Fix typo in package.json

package/node_modules/are-we-there-yet/package.json

@@ -1,30 +1,27 @@
 {
-  "_args": [
-    [
-      "are-we-there-yet@1.1.4",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "are-we-there-yet@1.1.4",
-  "_id": "are-we-there-yet@1.1.4",
+  "_from": "are-we-there-yet@~1.1.2",
+  "_id": "are-we-there-yet@1.1.5",
   "_inBundle": false,
-  "_integrity": "sha1-u13KOCu5TwXhUZQ3PRb9O6HKEQ0=",
+  "_integrity": "sha512-5hYdAkZlcG8tOLujVDTgCT+uPX0VnpAH28gWsLfzpXYm7wP6mp5Q/gYyR7YQ0cKVJcXJnl3j2kpBan13PtQf6w==",
   "_location": "/are-we-there-yet",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "are-we-there-yet@1.1.4",
+    "raw": "are-we-there-yet@~1.1.2",
     "name": "are-we-there-yet",
     "escapedName": "are-we-there-yet",
-    "rawSpec": "1.1.4",
+    "rawSpec": "~1.1.2",
     "saveSpec": null,
-    "fetchSpec": "1.1.4"
+    "fetchSpec": "~1.1.2"
   },
-  "_requiredBy": [],
-  "_resolved": "https://registry.npmjs.org/are-we-there-yet/-/are-we-there-yet-1.1.4.tgz",
-  "_spec": "1.1.4",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_requiredBy": [
+    "/npmlog"
+  ],
+  "_resolved": "https://registry.npmjs.org/are-we-there-yet/-/are-we-there-yet-1.1.5.tgz",
+  "_shasum": "4b35c2944f062a8bfcda66410760350fe9ddfc21",
+  "_spec": "are-we-there-yet@~1.1.2",
+  "_where": "/Users/pipobscure/fsevents/node_modules/npmlog",
   "author": {
     "name": "Rebecca Turner",
     "url": "http://re-becca.org"
@@ -32,14 +29,16 @@
   "bugs": {
     "url": "https://github.com/iarna/are-we-there-yet/issues"
   },
+  "bundleDependencies": false,
   "dependencies": {
     "delegates": "^1.0.0",
     "readable-stream": "^2.0.6"
   },
+  "deprecated": false,
   "description": "Keep track of the overall completion of many disparate processes",
   "devDependencies": {
-    "standard": "^6.0.8",
-    "tap": "^5.7.0"
+    "standard": "^11.0.1",
+    "tap": "^12.0.1"
   },
   "files": [
     "index.js",
@@ -60,5 +59,5 @@
   "scripts": {
     "test": "standard && tap test/*.js"
   },
-  "version": "1.1.4"
+  "version": "1.1.5"
 }

package/node_modules/are-we-there-yet/tracker-stream.js

@@ -33,3 +33,4 @@ TrackerStream.prototype._flush = function (cb) {
 delegate(TrackerStream.prototype, 'tracker')
   .method('completed')
   .method('addWork')
+  .method('finish')

package/node_modules/balanced-match/package.json

@@ -1,30 +1,27 @@
 {
-  "_args": [
-    [
-      "balanced-match@1.0.0",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "balanced-match@1.0.0",
+  "_from": "balanced-match@^1.0.0",
   "_id": "balanced-match@1.0.0",
   "_inBundle": false,
   "_integrity": "sha1-ibTRmasr7kneFk6gK4nORi1xt2c=",
   "_location": "/balanced-match",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "balanced-match@1.0.0",
+    "raw": "balanced-match@^1.0.0",
     "name": "balanced-match",
     "escapedName": "balanced-match",
-    "rawSpec": "1.0.0",
+    "rawSpec": "^1.0.0",
     "saveSpec": null,
-    "fetchSpec": "1.0.0"
+    "fetchSpec": "^1.0.0"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/brace-expansion"
+  ],
   "_resolved": "https://registry.npmjs.org/balanced-match/-/balanced-match-1.0.0.tgz",
-  "_spec": "1.0.0",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "89b4d199ab2bee49de164ea02b89ce462d71b767",
+  "_spec": "balanced-match@^1.0.0",
+  "_where": "/Users/pipobscure/fsevents/node_modules/brace-expansion",
   "author": {
     "name": "Julian Gruber",
     "email": "mail@juliangruber.com",
@@ -33,7 +30,9 @@
   "bugs": {
     "url": "https://github.com/juliangruber/balanced-match/issues"
   },
+  "bundleDependencies": false,
   "dependencies": {},
+  "deprecated": false,
   "description": "Match balanced character pairs, like \"{\" and \"}\"",
   "devDependencies": {
     "matcha": "^0.7.0",

package/node_modules/brace-expansion/package.json

@@ -1,30 +1,27 @@
 {
-  "_args": [
-    [
-      "brace-expansion@1.1.11",
-      "/Users/eshanker/Code/fsevents"
-    ]
-  ],
-  "_from": "brace-expansion@1.1.11",
+  "_from": "brace-expansion@^1.1.7",
   "_id": "brace-expansion@1.1.11",
   "_inBundle": false,
   "_integrity": "sha512-iCuPHDFgrHX7H2vEI/5xpz07zSHB00TpugqhmYtVmMO6518mCuRMoOYFldEBl0g187ufozdaHgWKcYFb61qGiA==",
   "_location": "/brace-expansion",
   "_phantomChildren": {},
   "_requested": {
-    "type": "version",
+    "type": "range",
     "registry": true,
-    "raw": "brace-expansion@1.1.11",
+    "raw": "brace-expansion@^1.1.7",
     "name": "brace-expansion",
     "escapedName": "brace-expansion",
-    "rawSpec": "1.1.11",
+    "rawSpec": "^1.1.7",
     "saveSpec": null,
-    "fetchSpec": "1.1.11"
+    "fetchSpec": "^1.1.7"
   },
-  "_requiredBy": [],
+  "_requiredBy": [
+    "/minimatch"
+  ],
   "_resolved": "https://registry.npmjs.org/brace-expansion/-/brace-expansion-1.1.11.tgz",
-  "_spec": "1.1.11",
-  "_where": "/Users/eshanker/Code/fsevents",
+  "_shasum": "3c7fcbf529d87226f3d2f52b966ff5271eb441dd",
+  "_spec": "brace-expansion@^1.1.7",
+  "_where": "/Users/pipobscure/fsevents/node_modules/minimatch",
   "author": {
     "name": "Julian Gruber",
     "email": "mail@juliangruber.com",
@@ -33,10 +30,12 @@
   "bugs": {
     "url": "https://github.com/juliangruber/brace-expansion/issues"
   },
+  "bundleDependencies": false,
   "dependencies": {
     "balanced-match": "^1.0.0",
     "concat-map": "0.0.1"
   },
+  "deprecated": false,
   "description": "Brace expansion as known from sh/bash",
   "devDependencies": {
     "matcha": "^0.7.0",

package/node_modules/chownr/chownr.js

@@ -1,52 +1,125 @@
-module.exports = chownr
-chownr.sync = chownrSync
+'use strict'
+const fs = require('fs')
+const path = require('path')
+
+/* istanbul ignore next */
+const LCHOWN = fs.lchown ? 'lchown' : 'chown'
+/* istanbul ignore next */
+const LCHOWNSYNC = fs.lchownSync ? 'lchownSync' : 'chownSync'
+
+const needEISDIRHandled = fs.lchown &&
+  !process.version.match(/v1[1-9]+\./) &&
+  !process.version.match(/v10\.[6-9]/)
+
+/* istanbul ignore next */
+const handleEISDIR =
+  needEISDIRHandled ? (path, uid, gid, cb) => er => {
+    // Node prior to v10 had a very questionable implementation of
+    // fs.lchown, which would always try to call fs.open on a directory
+    // Fall back to fs.chown in those cases.
+    if (!er || er.code !== 'EISDIR')
+      cb(er)
+    else
+      fs.chown(path, uid, gid, cb)
+  }
+  : (_, __, ___, cb) => cb
+
+/* istanbul ignore next */
+const handleEISDirSync =
+  needEISDIRHandled ? (path, uid, gid) => {
+    try {
+      return fs[LCHOWNSYNC](path, uid, gid)
+    } catch (er) {
+      if (er.code !== 'EISDIR')
+        throw er
+      fs.chownSync(path, uid, gid)
+    }
+  }
+  : (path, uid, gid) => fs[LCHOWNSYNC](path, uid, gid)
 
-var fs = require("fs")
-, path = require("path")
-
-function chownr (p, uid, gid, cb) {
-  fs.readdir(p, function (er, children) {
-    // any error other than ENOTDIR means it's not readable, or
-    // doesn't exist.  give up.
-    if (er && er.code !== "ENOTDIR") return cb(er)
-    if (er || !children.length) return fs.chown(p, uid, gid, cb)
-
-    var len = children.length
-    , errState = null
-    children.forEach(function (child) {
-      var pathChild = path.resolve(p, child);
-      fs.lstat(pathChild, function(er, stats) {
-        if (er)
-          return cb(er)
-        if (!stats.isSymbolicLink())
-          chownr(pathChild, uid, gid, then)
-        else
-          then()
-        })
+// fs.readdir could only accept an options object as of node v6
+const nodeVersion = process.version
+let readdir = (path, options, cb) => fs.readdir(path, options, cb)
+let readdirSync = (path, options) => fs.readdirSync(path, options)
+/* istanbul ignore next */
+if (/^v4\./.test(nodeVersion))
+  readdir = (path, options, cb) => fs.readdir(path, cb)
+
+const chownrKid = (p, child, uid, gid, cb) => {
+  if (typeof child === 'string')
+    return fs.lstat(path.resolve(p, child), (er, stats) => {
+      if (er)
+        return cb(er)
+      stats.name = child
+      chownrKid(p, stats, uid, gid, cb)
     })
-    function then (er) {
-      if (errState) return
-      if (er) return cb(errState = er)
-      if (-- len === 0) return fs.chown(p, uid, gid, cb)
+
+  if (child.isDirectory()) {
+    chownr(path.resolve(p, child.name), uid, gid, er => {
+      if (er)
+        return cb(er)
+      const cpath = path.resolve(p, child.name)
+      fs[LCHOWN](cpath, uid, gid, handleEISDIR(cpath, uid, gid, cb))
+    })
+  } else {
+    const cpath = path.resolve(p, child.name)
+    fs[LCHOWN](cpath, uid, gid, handleEISDIR(cpath, uid, gid, cb))
+  }
+}
+
+
+const chownr = (p, uid, gid, cb) => {
+  readdir(p, { withFileTypes: true }, (er, children) => {
+    // any error other than ENOTDIR or ENOTSUP means it's not readable,
+    // or doesn't exist.  give up.
+    if (er && er.code !== 'ENOTDIR' && er.code !== 'ENOTSUP')
+      return cb(er)
+    if (er || !children.length)
+      return fs[LCHOWN](p, uid, gid, handleEISDIR(p, uid, gid, cb))
+
+    let len = children.length
+    let errState = null
+    const then = er => {
+      if (errState)
+        return
+      if (er)
+        return cb(errState = er)
+      if (-- len === 0)
+        return fs[LCHOWN](p, uid, gid, handleEISDIR(p, uid, gid, cb))
     }
+
+    children.forEach(child => chownrKid(p, child, uid, gid, then))
   })
 }
 
-function chownrSync (p, uid, gid) {
-  var children
+const chownrKidSync = (p, child, uid, gid) => {
+  if (typeof child === 'string') {
+    const stats = fs.lstatSync(path.resolve(p, child))
+    stats.name = child
+    child = stats
+  }
+
+  if (child.isDirectory())
+    chownrSync(path.resolve(p, child.name), uid, gid)
+
+  handleEISDirSync(path.resolve(p, child.name), uid, gid)
+}
+
+const chownrSync = (p, uid, gid) => {
+  let children
   try {
-    children = fs.readdirSync(p)
+    children = readdirSync(p, { withFileTypes: true })
   } catch (er) {
-    if (er && er.code === "ENOTDIR") return fs.chownSync(p, uid, gid)
+    if (er && er.code === 'ENOTDIR' && er.code !== 'ENOTSUP')
+      return handleEISDirSync(p, uid, gid)
     throw er
   }
-  if (!children.length) return fs.chownSync(p, uid, gid)
 
-  children.forEach(function (child) {
-    var pathChild = path.resolve(p, child)
-    var stats = fs.lstatSync(pathChild)
-    if (!stats.isSymbolicLink())
-      chownrSync(pathChild, uid, gid)
-  })
-  return fs.chownSync(p, uid, gid)
+  if (children.length)
+    children.forEach(child => chownrKidSync(p, child, uid, gid))
+
+  return handleEISDirSync(p, uid, gid)
 }
+
+module.exports = chownr
+chownr.sync = chownrSync