search

jfromaniello / winser

Run a node.js application as a window service using nssm.
270 stars 47 forks source link

Add 64bit support and improve admin detection #10

Closed dignifiedquire closed 11 years ago

dignifiedquire commented 11 years ago

I've added the ability to detect the architecture and switch thes nssm.exe based on that. (As disscussed in #9) Also the admin detection was not working so I've changed that based on this question on SO: http://stackoverflow.com/questions/4051883/batch-script-how-to-check-for-admin-rights

dignifiedquire commented 11 years ago

Fixed it.

HeyHugo commented 11 years ago

I think administrator detection is not needed since the command line will output this feedback anyway when needed, see issue #11

jfromaniello commented 11 years ago

@HeyHugo are you sure? I remember adding this check because back then nssm.exe didn't show anything and silently failed saying that the service was installed or something alike. I will double check

HeyHugo commented 11 years ago

At least for me the latest nssm 2.16 (which is used in winser) fails with output: "Administrator access is needed to install a service." if I don't run it as administrator.