biojs / slush-biojs

DEPRECATED - please use https://github.com/biojs/generator-biojs-webcomponents
https://github.com/biojs/generator-biojs-webcomponents
7 stars 3 forks source link

Dependency of glup failed #7

Closed Endle closed 10 years ago

Endle commented 10 years ago

$ slush biojs Then the terminal complained:

npm WARN deprecated gulp-rimraf@0.1.1: Use npmjs.org/del instead, see https://github.com/gulpjs/gulp/blob/master/docs/recipes/delete-files-folder.md
npm WARN optional dep failed, continuing fsevents@0.2.1

https://github.com/gulpjs/gulp/blob/master/docs/recipes/delete-files-folder.md

wilzbach commented 10 years ago

thanks for reporting :)

npm WARN optional dep failed, continuing fsevents@0.2.1

comes from watchify which uses --> chokidar. Chokidar has an optional dependency to fsevents.

fsevents is a native library for file access on Mac OS and chokidar uses it to receive file events faster on OS X. So it is okay if this optional dep fails on Linux.

For more info see their package.json and the implementation

Endle commented 10 years ago

@greenify commented on 2014年10月5日 下午8:18 GMT+0800:

thanks for reporting :)

npm WARN optional dep failed, continuing fsevents@0.2.1

comes from watchify which uses --> chokidar. Chokidar has an optional dependency to fsevents.

fsevents is a native library for file access on Mac OS and chokidar uses it to receive file events faster on OS X. So it is okay if this optional dep fails on Linux.

For more info see their package.json

and the implementation

Thanks. Do we need to modify our code to make the installer quiet?

wilzbach commented 10 years ago

Do we need to modify our code to make the installer quiet?

Do you mind about this line WARN optional dep failed showing up? Honestly I don't.

Anyway the idea of slush is to make it very easy to generate a super-awesome new package with many development helpers included. So of course you are allowed to modify everything and if you want to remove sth. (e.g. watchify) in your package, feel free ;-)

Furthermore if you really don't want to see log messages, you could use

npm_config_loglevel=error npm install

or include that in a .npmrc in your project

loglevel=error

NPM also supports user and system config files.

I don't recommend to silence log messages

Endle commented 10 years ago

@greenify commented on 2014年10月5日 下午9:39 GMT+0800:

Do we need to modify our code to make the installer quiet?

Do you mind about this line WARN optional dep failed showing up?

Honestly I don't.

Anyway the idea of slush is to make it very easy to generate a super-awesome new package with many development helpers included.

So of course you are allowed to modify everything and if you want to remove sth. (e.g. watchify) in your package, feel free ;-)

Furthermore if you really don't want to see log messages, you could use

npm_config_loglevel=error npm install `

or include that in a .npmrc in your project

`loglevel=error

NPM also supports user and system [config files](https://www.npmjs.org/doc/misc/npm-config.html).

**I don't recommend to silence log messages**

I don't, neither. I worry that if we face a real bug in future, such warning may produce noise for debugging.

How about writing them down in README or other documents?

wilzbach commented 10 years ago

I worry that if we face a real bug in future, such warning may produce noise for debugging. How about writing them down in README or other documents?

+1. I created a new article in our wiki.

https://github.com/biojs/biojs/wiki/Common-error-messages