geekmaster / eyefiserver2

Automatically exported from code.google.com/p/eyefiserver2
0 stars 0 forks source link

init.d script broken #5

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
I haven't tried to run it but it looks like the current init.d script is 
brokne, it puts the "start" command in the wrong place.

Feel free to fix this, as I may not get a chance to look at this for a few days.

Original issue reported on code.google.com by davidgrant on 22 May 2012 at 3:57

GoogleCodeExporter commented 8 years ago
I apologize, but my fix would be blind, as I don't have start-stop-daemon tool.

Original comment by vilius.s...@gmail.com on 22 May 2012 at 4:02

GoogleCodeExporter commented 8 years ago
No problem, I saw you mentioned that I should test it. I will give it a shot. 
My wife is due to have a baby any day now so I'm busy. On the other hand, I'm 
off work right now so I do have some time.

Original comment by davidgrant on 22 May 2012 at 4:22

GoogleCodeExporter commented 8 years ago
As daemonisation is now implemented, I think wee should rewrite the startup 
script to make it environment independent so it could work on both mine and 
yours.

The startup script should directly call eyefiserver.py with 
start|stop|restart|instance parameters. No pids or daemonisation tools should 
be used.

I will assign it to myself as you wont have any time nearest 18 years starting 
from now. :) Congrats by the way.

Original comment by vilius.s...@gmail.com on 23 May 2012 at 6:41

GoogleCodeExporter commented 8 years ago
Yeah I guess the startup script should just call "stop/start/restart" and the 
"instance" parameter can be used from command-line, for example. Thanks for 
taking.

Original comment by davidgrant on 23 May 2012 at 6:45

GoogleCodeExporter commented 8 years ago
Ok, init.d script is fine now, so I will close this. I will open a new issue to 
make the init.d script work on Linux and QNAP.

Original comment by davidgrant on 24 Jun 2012 at 9:46

GoogleCodeExporter commented 8 years ago

Original comment by davidgrant on 24 Jun 2012 at 9:47

GoogleCodeExporter commented 8 years ago
I already have one that both Linux and QNAP compatible, will commit ASAP.

Original comment by duke...@gmail.com on 25 Jun 2012 at 4:37