node-flowdock icon indicating copy to clipboard operation
node-flowdock copied to clipboard

allow switch out of buffertools

Open bakesteve opened this issue 10 years ago • 4 comments

this package relies on buffertools, which uses node-gyp on windows, that kinda sucks, as there is no dist/executable shipped in the buffertools npm, it does a rebuild from source this basically means that flowdock has a dependency on python2.7 and visual studio thats compounded when you look at hubot-flowdock So, for the functionality this seems to be giving, thats quite a heavy barrier is there any way you could either:

  • remove buffer tools for something that works out of the box across platforms
  • make it pluggable, so I can at least include a local copy

bakesteve avatar Feb 13 '15 16:02 bakesteve

Replacing the JSON streaming with eventsource could work. Windows users seem to be a quite small minority, so this isn't a very high priority for us. That said, we welcome any pull requests to tackle this.

lautis avatar Feb 13 '15 22:02 lautis

Err.. Ok. We might look at trying to plug that for you.. Or possibly just go back to slack given that: A) their stuff just worked B) they didn't just write off a bunch of folks for 1 line of copying a stream...

On Fri, Feb 13, 2015 at 10:35 pm, Ville Lautanala <[email protected]mailto:[email protected]> wrote:

Replacing the JSON streaming with eventsourcehttps://github.com/aslakhellesoy/eventsource-node could work. Windows users seem to be a quite small minority, so this isn't a very high priority for us. That said, we welcome any pull requests to tackle this.

Reply to this email directly or view it on GitHubhttps://github.com/flowdock/node-flowdock/issues/20#issuecomment-74339396.

bakesteve avatar Feb 14 '15 01:02 bakesteve

Sorry – I didn't mean to write off Windows support. We should get this fixed eventually, but I don't expect anyone to immediately jump to work on this. In the meantime, pull requests either by replacing our current uses of buffertools with pure javascript or some other methods would be appreciated especially since we don't have Windows development machines and our knowledge about Node on Windows is pretty much based on issues like this.

lautis avatar Feb 14 '15 09:02 lautis

Ok – will push something up if one of us gets a chance. For now we are getting round it by including a built version of buffertools and changing our gitignore so that gets pulled with the rest of our hubot repo

bakesteve avatar Feb 14 '15 14:02 bakesteve