dynamicdan / sn-filesync

2-way sync ServiceNow field values to local files
MIT License
66 stars 37 forks source link

Migrate from Restify to Restler #22

Closed Echo3ToEcho7 closed 8 years ago

Echo3ToEcho7 commented 8 years ago

In order to remove the need for a compiler, I have started a fork that uses restler instead of restify.

Thoughts?

dynamicdan commented 8 years ago

What compiler is actually needed? NPM still does all the installing?

Echo3ToEcho7 commented 8 years ago

Restify uses dtrace-provider which is a native module and needs a C++ compiler installed. NPM will still install it, but will fail doing the install if there is no C++ compiler (like gcc or llvm) installed. I always have Xcode installed on my Mac so it isn't an issue for me, but not all of the SNow developers that I work with have a compiler on their computer.

dynamicdan commented 8 years ago

experimental branch now contains your fork. Initial testing seems fine. 1 part needs re-testing for dodgy https/SSL setups. I will test this branch a bit more before merging it in to master. Thanks for the awesome upgrade!