Auto Restart package for QNAP users

[simoncn]

(10-10-2013 16:12)KoS Wrote:  Thanks for your fast reply. I agree with you that it would be nicer and more efficient and elegant to have it directly in minimserver, but a solution for the meantime would be nice. And as we already have a simple way and script to restart minim based on inotify chanages, it would be great to have this supported. There are no big changes in your QNAP package to add the additional named pipe to it. e.g. there are three lines were minimserver is started and the named pipe needs to be connected. plus two additional lines to create the pipe. so i would greatly appreciate if you could add that to the QNAP package. or i can add it by myself and send you a patch file?

The cost of adding a feature isn't just a matter of counting the number of lines of code involved. When this was added to the Synology version of MinimServer, there were a number of discussions over many weeks and multiple iterations of the design to find something that worked well enough on the Synology firmware. This process isn't yet complete, with further changes expected in the next release of MinimServer for Synology.

Another consideration is that adding any feature sets an implied precedent for user expectations. In this case, supporting this on Synology has given you the expectation that it should be supported on QNAP as well. If I were to add support for QNAP, there are likely to be further requests for adding this support on Windows, Mac, Linux or ReadyNAS.

I don't have the bandwidth at the moment to do any work on this, including design discussions. This is because I am currently focusing my efforts on completing and delivering the next release of MinimServer.

You are of course welcome to make changes to the QNAP minimserver.sh file to support the named pipes feature for testing and for your own personal use. If you would like to make an "Auto Restart" package for QNAP available to other users, you could include code in this "Auto Restart" package to patch the minimserver.sh file with the changes needed to make the "Auto Restart" package work. This patching approach is needed because the MinimServer license terms don't allow redistributing a modified version of the minimserver.sh file as part of some other package.