search

pgte / fugue

Unicorn for node.js
MIT License
396 stars 14 forks source link

Update to daemon.cc to include node.h instead of ev.h This allows building on node 0.6.x and fixes #13 #14

Closed Sembiance closed 12 years ago

Sembiance commented 12 years ago

The tests fail with this:

pedroteixeira:fugue pedroteixeira$ make test node-waf configure build Checking for program g++ or c++ : /usr/bin/g++ Checking for program cpp : /usr/bin/cpp Checking for program ar : /usr/bin/ar Checking for program ranlib : /usr/bin/ranlib Checking for g++ : ok

Checking for node path : ok /usr/local/lib/node Checking for node prefix : ok /usr/local 'configure' finished successfully (0.441s) Waf: Entering directory /Users/pedroteixeira/projects/fugue/build' [1/2] cxx: deps/daemon.cc -> build/Release/deps/daemon_1.o [2/2] cxx_link: build/Release/deps/daemon_1.o -> build/Release/daemon.node Waf: Leaving directory/Users/pedroteixeira/projects/fugue/build' 'build' finished successfully (0.582s) node tools/test.js test_zero_workers test_one_worker test_if_i_can_reach_many_workers test_app_reload test_working_dir test_leaks test_worker_dies_after_master_sudden_death test_unix_socket test_pipe_leaks running test test_zero_workers... AssertionErrorErrorNo such moduleError: No such module at Object.start (/Users/pedroteixeira/projects/fugue/lib/fugue.js:190:28)

Sembiance commented 12 years ago

I think it best to close this pull request. Now that I'm aware of alternatives to fugue (LearnBoost's cluster), I can look into using those for node 0.6.x compatibility.