Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Remove outdated "file" package dependency. #1

Merged
merged 1 commit into from

2 participants

@justmoon

The "file" package is no longer maintained and throws a warning message on inclusion:

The "sys" module is now called "util". It should have a similar interface.

This patch removes the dependency (the same functionality now exists in Node.js core) and adds an engine dependency instead. This should cause NPM to automagically pick the right version of the package.

Please pull, increment the version number and republish, thanks!

@frodwith frodwith merged commit fbe6db5 into frodwith:master
@frodwith
Owner

Thank you very much :)

@justmoon

You're welcome. Tried six options parsers, this is the best. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Feb 7, 2012
  1. @justmoon
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 3 deletions.
  1. +1 −2  lib/yanop.coffee
  2. +3 −1 package.json
View
3  lib/yanop.coffee
@@ -1,4 +1,3 @@
-file = require 'file'
path = require 'path'
reexport = (module) ->
@@ -19,7 +18,7 @@ exports.parse = (spec, argv) ->
exports.zero = () ->
abs = path.normalize process.argv[1]
- rel = file.path.relativePath process.cwd(), abs
+ rel = path.relative process.cwd(), abs
script = if rel.length < abs.length then rel else abs
"#{ process.argv[0] } #{ script }"
View
4 package.json
@@ -10,11 +10,13 @@
"test" : "./test"
},
"dependencies" : {
- "file" : ">=0.1.1"
},
"devDependencies" : {
"nodeunit" : ">=0.5.1"
},
+ "engines" : {
+ "node" : ">=0.6.0"
+ },
"scripts" : {
"test": "nodeunit test/*.js"
},
Something went wrong with that request. Please try again.