mscdex

Results 571 comments of mscdex

Aside from the inline comments, there are a couple of other issues: - The `err` argument passed to the callback in `_send()` should be checked (in case REST is not...

Are you mixing major node versions (e.g. v0.10, v0.12, v4.x, v5.x, v6.x, etc.) or trying to use the same compiled addon across different platforms (e.g. Linux and OS X)?

You have to make sure major node versions match, not just npm versions. If both dev and production are running within the same release branch (e.g. node v4.0.0 and v4.4.5)...

@RachelScodes How are you installing this addon?

@RachelScodes Can you post the console output you get when you `npm --loglevel=silly install mariasql` or `node-gyp rebuild` (latter preferred)?

@danialm `mariasql` v0.1.x is old and some versions had some issues, you will need to upgrade.

If there are outstanding requests for an sftp session when it's cleaned up, errors are passed to the requests' callbacks, which should in turn emit errors on the stream.

You're probably better off using `mysql2` instead.

@wzrdtales No I mean the `mysql2` module on npm. You can use whichever server you want, they both speak the same protocol.