I've put a pull request to ckolivas to fix 'quit'
I guess it will be there once he accepts the pull (or if anyone for some unknown reason needs it urgently ... it's in my git or look at the pull request)
It seems it hasn't worked for a while
Edit: and it's now in ckolivas' git.
And an explanation for anyone who uses BAMT and wants 'quit' to work:
The next version (or current git) will be API version 1.2
With the new version you must include "--api-listen --api-allow W:127.0.0.1" when you start cgminer if you want the local machine to be able to send cgminer the 'quit' command
(The 'W:' means allow 127.0.0.1 to have write/priviledged access to cgminer)
The next verison will still allow all read access to cgminer as any current version does without changing the options you start cgminer with.
As mentioned above in my quote, the 'quit' command will also be fixed in the next release.
Also for lodcrappo:And what is an unavoidable bug in probably every piece of API usage software, that you can fix with 1.2:
At the moment you probably look for "MHS 5s" to find the 5s average.
That will actually fail if anyone starts cgminer with the --log option with some value other than 5.
You can find the correct value for '5' with the 'config' command in the new 1.2 "Log Interval" - which of course will be '5' unless someone runs cgminer with the --log option.
(The --log option has been in cgminer for a long time)
So if "Log Interval" is "10" then you should look for "MHS 10s" not "MHS 5s"
Edit: and in case you didn't spot the obvious:
...
quit without the | did not work in the version I used in creating the BAMT controller for cgminer. I will retest this the next time we update.
...
"quit\n" didn't (and never will) work.
"quit" always has been accepted by the API to mean quit cgminer - but stopped working due to a problem with the api.c doquit() function - however, the command "quit" was being accepted and attempting to do what was requested.
You should not be adding extra characters on the end - as your normal reporting interface shouldn't do that either (if it does)