make stringprep usage optional

[trentm]

"npm install" fails on the node-stringprep dependency on Mac OS X. I played with trying to get node-stringprep to build, but couldn't, either with the icu installs with "brew" or with MacPorts.

From what I understand stringprep is only being used for case-normalization (and possibly other types of normalization?) of international domain names. As a result I've tried to make the usage of stringprep optional here.

Would you be interested in merging this patch? I'd be happy to adjust it if it doesn't work for you as is.

Cheers,
Trent

[astro]

Thanks!

v0.2.3 is out with some minor modifications to this feature.

[donquixote]

Hi,
node-stringprep is now optional, but..

• still a warning showing up with each command, that can be confusing. esp, if you have other errors in your custom app, and think they must be caused by the stringprep thing (which they are not)
• documentation (README) should explain installation of stringprep and icu. E.g. i had no idea what "icu" is and that i can install it with apt-get in debian.

One solution to point 1:
var xmpp = require('node-xmpp').noStringPrep()

-> that would be a conscious decision by the developer that he currently cannot install stringprep, or does not want it. so the message can be removed.

Thanks!