bloomberg / blpapi-node

Bloomberg Open API module for node.js

http://www.bloomberglabs.com/api/about/

Other

246 stars 50 forks source link

Update Linux/Windows BLPAPI to v3.8.8.1. #58

Closed apaprocki closed 9 years ago

apaprocki commented 9 years ago

Fixes #56.

apaprocki commented 9 years ago

32 should also be re-tested once the new stable version lands.

apaprocki commented 9 years ago

@ericvw Didn't realize this one was still outstanding.. any issues on your end?

ericvw commented 9 years ago

I believe the DLL is needed for Windows. I'll verify and provide an update.

ericvw commented 9 years ago

@apaprocki, please rebase your changes against the latest master. We dropped support for node < 0.11.0, which is why we are getting the build failures. I'll test these changes on Windows tomorrow.

apaprocki commented 9 years ago

@ericvw rebased, Travis building it now...

ericvw commented 9 years ago

I tested this PR on my Windows VM and it all checks out.