https://travis-ci.org/isaacs/node-touch/jobs/248986144

• Create two classes, Touch and TouchSync. Override just the fs-calling bits to swap out the sync-ness. (Pattern established in minipass, minizlib, and tar.)
• Async method returns a promise, attaches the cb to it if provided.
• handle directories (or at least, error properly)
• Don't use nopt in the bin
• bin takes a timestamp like touch(1)
• tap --100 test coverage

Another dependency installed node-touch in my node modules so now I can't call the unix version from my package.json scripts and am stuck with it. The only real difference is that node-touch error logs the returned fs.stat object. Can this not be the default behavior?

This would make touch play nicely with other standard js libraries.

$ touch node_modules/
bad touch!
/Users/mstade/dev/websdk/nap/node_modules/touch/bin/touch.js:28
      throw er
      ^

Error: EISDIR: illegal operation on a directory, open 'node_modules/'
    at Error (native)

Quite possibly out of scope for the module, but figured I'd report it either way.

I don't use this module myself, but ran into a problem with builds suddenly failing because of it. The reason is during the build, the $PATH variable is prepended with node_modules/.bin, so local executables can be conveniently used in the Makefile. Because it's prepended, touch takes precedence over the system touch, and because this module doesn't work with directories the build failed.

The builds started suddenly failing, because Travis CI started including Node 5.0 in their environment setup. This includes npm 3, which has the new flat module layout. This brought up the this module from the node_modules depths, and voila – failing builds.

Interesting little detour. :o)

It was easy enough to fix. Just changed it to append node_modules/.bin to $PATH, as opposed to prepending it:

PATH := $(PATH):node_modules/.bin

Someone else might run into the same issue, so hopefully searching will lead them here.

It would be useful if directories were also supported.

Passed in a path, e.g. /var/www/myapp/.restart, to touch.sync(path), but touch tries to do fs.futimesSync with fd having the value 18. All of this happens here: https://github.com/isaacs/node-touch/blob/master/touch.js#L127 and I get an EPERM error, since 18 doesn't exist.

In onopen:

  onopen (er, fd) {
    if (er) {
      if (er.code === 'EISDIR')
        this.onopen(null, null)
      else if (er.code === 'ENOENT' && this.nocreate)
        this.emit('done')
      else
        this.emit('error', er)
    } else {
      this.fd = fd
      if (this.ref)
        this.statref()
      else if (!this.atime || !this.mtime)
        this.fstat()
      else
        this.futimes()
    }
  }

In onfstat:

  onfstat (st) {
    if (typeof this.atime !== 'number')
      this.atime = parseInt(st.atime.getTime()/1000, 10)

    if (typeof this.mtime !== 'number')
      this.mtime = parseInt(st.mtime.getTime()/1000, 10)

    this.futimes()
  }

If atime or mtime is missing stat is called:

      else if (!this.atime || !this.mtime)
        this.fstat()

In the stat callback onstat if atime is not number it is overwritten with the value returned by stat:

    if (typeof this.atime !== 'number')
      this.atime = parseInt(st.atime.getTime()/1000, 10)

The doc says atime should be a date not a number

atime like touch -a Can be either a Boolean, or a Date.

The doc is wrong but this causes a bug that is hard to debug.

Any idea how to implement such option in nodejs ?

Hi,

It would be useful to get the list of changes and bugfixes between versions, with migrations notes or recommendations if any. For now we have to check in the code if any change in the API was made, and it can take some time when you have a lot of dependencies to update.

Please provide a CHANGELOG.md file or GitHub releases.

Thank you.

import * as touch from 'node-touch';

C:\projects\theben3\node_modules\node-inserted\index.js:5
var elem = document.createElement('style');
^

ReferenceError: document is not defined
at module.exports (C:\projects\theben3\node_modules\node-inserted\index.js:5:12)
at Object. (C:\projects\theben3\node_modules\node-touch\index.js:1:25)
at Module._compile (module.js:635:30)
at Module._extensions..js (module.js:646:10)
at Object.require.extensions.(anonymous function) [as .js] (C:\projects\theben3\node_modules\babel-register\lib\node.js:152:7)
at Module.load (module.js:554:32)
at tryModuleLoad (module.js:497:12)
at Function.Module._load (module.js:489:3)
at Module.require (module.js:579:17)
at require (internal/module.js:11:18)

Hi,
It looks like the code using nopt was removed in 3e06ea9

However, its still listed as a dependency, https://www.npmjs.com/package/touch?activeTab=dependencies, and nopt uses abbrev, so two extra dependencies are installed

I guess nopt can be removed from https://github.com/isaacs/node-touch/blob/master/package.json ?

I can't find it on npm install node-touch

This error runs when .sync option is used (file just created but no content is written)

When I was trying to use this module I got the following error:

ReferenceError: document is not defined
 at module.exports (C:\Develop\trunk\node_modules\node-touch\node_modules\node-inserted\index.js:5:12)
 at Object.<anonymous> (C:\Develop\trunk\node_modules\node-touch\index.js:1:87)

I am using node 0.10.32 on Windows 7 64bit.

Is this project still alive? Asking, since there are outdated dependencies.

Edit: Just seen that fs.utimesSync will do as a good substitute.

Trying to use touch to make sure a file exists before it's used as a stream, and am getting this error:

c:\users\mike\documents\Git projects\released\flickrmirror>node app

fs.js:427
  return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
                 ^
Error: ENOENT, no such file or directory 'c:\users\mike\documents\Git projects\released\flickrmirror\logs\2013-11-23 13:14:24.log'
    at Object.fs.openSync (fs.js:427:18)
    at openThenF (c:\users\mike\documents\Git projects\released\flickrmirror\node_modules\touch\touch.js:76:40)
    at touch_ (c:\users\mike\documents\Git projects\released\flickrmirror\node_modules\touch\touch.js:69:10)
    at optionsRef (c:\users\mike\documents\Git projects\released\flickrmirror\node_modules\touch\touch.js:41:28)
    at touch (c:\users\mike\documents\Git projects\released\flickrmirror\node_modules\touch\touch.js:65:10)
    at getFileStream (c:\users\mike\documents\Git projects\released\flickrmirror\app.js:84:11)
    at c:\users\mike\documents\Git projects\released\flickrmirror\app.js:87:19
    at Function.<anonymous> (c:\users\mike\documents\Git projects\released\flickrmirror\app.js:100:4)
    at Function.app.configure (c:\users\mike\documents\Git projects\released\flickrmirror\node_modules\express\lib\application.js:393:61)
    at Object.<anonymous> (c:\users\mike\documents\Git projects\released\flickrmirror\app.js:75:5)

Relevant code in the actual app.js:

var dateFormat = require("dateformat"),
    touch = require("touch"),
    getFileStream = function() {
      var now = new Date(),
          suffix = dateFormat(now, "yyyy-mm-dd HH:MM:ss"),
          logfile = "./logs/" + suffix + ".log";
      touch(logfile);
      return fs.createWriteStream(logfile, { flags: 'a+' });
    },
    logfile = getFileStream();

This is on windows 7 pro, 64 bit version.

This would make asynchronously creating several files much easier.

Hey @isaacs - I was wondering if you'd want to publish this as nodetouch for easier use with npx?

Difference between the commands would be:
npx -p touch nodetouch gulpfile.js
vs
npx nodetouch gulpfile.js