-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Windows or Docker support for telepat-api #9
Comments
@jakobrosenberg could you detail your question a bit? telepat-api is an Express NodeJS application, so it's compatible with both Windows and Docker deployments. We also ship Dockerfiles with each Telepat component, however the images available in the Docker Registry are a little old, so I'd recommend rebuilding from Dockerfile. |
That was swift, so excuse my late reply. I've been trying to I'll try cloning a fresh copy and see what errors I get and report them back here. |
One issue to check here is your NodeJS version. Because we're still waiting on some of our dependencies to become compatible with the latest NodeJS versions, right now we're still doing most of our work on the 0.12 branch. Official tests with the LTS version are scheduled and should be coming in the week or so. |
|
I'll try remove some of the binary dependencies from the global dir and see if that helps. |
The issue here is that our own dependencies are not installing correctly on node 4.2.6. You shouldn't have issues with the 0.12 branch of node. We'll try to prioritize proper support at least for the LTS versions of node. |
Was finally able to run
I then had to change the start script in Then I get the following error:
|
Unfortunately, we didn't do any tests under Windows. telepat-models dynamically loads every implemented loggers under telepat-models/lib/logger and syslogger is there (this should only work for Linux environments). node-syslog is an optional dependency that's why It will be fixed sometime in the future so that platform dependent adapters will not crash telepat if used (but it should be fine if you load the adapter as long as it doesn't try to require the node package that's build upon, node-syslog in the case of the syslog adapter). For a quick workaround, you can try to delete the syslog_logger.js script as you'll never be using it. |
Thanks. I'll give it a try. By the way, you might want to look at your npm start script and bin/www file if you're planning to support Windows. |
node_syslog will be replaced by winston in the next release (which is a few days away) telepat-io/telepat-models@0b3ed95 |
Does either exist?
The text was updated successfully, but these errors were encountered: