jhurliman / node-echoprint-server

A node.js implementation of the Echoprint music identification server
74 stars 37 forks source link

Imposible to add new track on Windows 7 #2

Closed devops85 closed 11 years ago

devops85 commented 11 years ago

Hi, Your project is amazing. I am trying to install it on Windows. Everything looks fine, but I am faccing an issue when I try to add a new record. This message appears:

debug: No matched tracks debug: Track does not exist in the database yet, status NO_RESULTS info: Creating track TR3CLO6IH55Y48CW ("Tainted") error: Failed to ingest track: Error: ENOENT, open 'C:\tmp\echoprint-TR3CLO6IH55 Y48CW'

Could you please shed some light here?

Thank you very much.

jhurliman commented 11 years ago

Ah, there is a lazy assumption in there assuming /tmp/ exists. I can fix that to use the correct temp dir for the platform. As a quick hack try creating C:\tmp\

On Jul 27, 2012, at 5:20 PM, devops85reply@reply.github.com wrote:

Hi, Your project is amazing. I am trying to install it on Windows. Everything looks fine, but I am faccing an issue when I try to add a new record. This message appears:

debug: No matched tracks debug: Track does not exist in the database yet, status NO_RESULTS info: Creating track TR3CLO6IH55Y48CW ("Tainted") error: Failed to ingest track: Error: ENOENT, open 'C:\tmp\echoprint-TR3CLO6IH55 Y48CW'

Could you please shed some light here?

Thank you very much.


Reply to this email directly or view it on GitHub: https://github.com/jhurliman/node-echoprint-server/issues/2

devops85 commented 11 years ago

Thanks man, This hack works!! :) Do you know which line in the code we have to change in order to set a new temp dir?. Thanks

jhurliman commented 11 years ago

I just committed 6ea272bf7c05740939e2e9e338e3a4fa917b896f which should address this issue. It uses a platform-agnostic library called node-temp to find the temporary directory on your machine instead of hardcoding /tmp/. Let me know if you have any issues with this and I'll reopen the ticket. Thanks for the report!