(bugfix) Insufficient warning level to catch that bug

All - Rework config API to allow default/optional values

- Now returns a boolean success, and takes a pointer in which to write the
  successfully parsed value (unchanged on failure)
- Also cleaned up argument processing to be flatter and slightly less hacky

Server - Ignore SIGPIPE (just let it return a read error)

Fix unbounded door thread creation

Add config to push script

Add BuildAndPush script

Client - Quit when --help/-h is passed

Merge pull request #4 from mtearle/master

Add config_modbus_port option to allow Modbus port to be set (for virtualcoke)

Revert change to items.cfg

gsCoke_ModbusPort should be giCoke_ModbusPort

Add config_modbus_port option to allow Modbus port to be set (for virtualcoke)

Merge pull request #3 from mtearle/master

Add Client Config file to enable local system override of dispense host/port

Client - Document command line flags

Client - Code to load up config file and set dispense server/port

Tidy up initialisation in doregex.c

Use common regex functions in client

Move Regex functions into common code and header

Initial move to common config code directory

Note further package dependencies required for build

Merge pull request #2 from mtearle/master

Add client and overall install target

Add client and overall install target

Fix Makefile to add references to sqlite3 and pthread to build on Ubuntu Trusty 14.04

Document account names in manpage

Fix position of balance output with set

Allow logging to be disabled, rename 'add' account

Server - Separate account for 'add'

Itemdb - Add 'membership' pseudo item type

Client - Fix uninitialised value in 'acct'

Door - Move thread init to after fork()

Server - Add error message if a bad baud rate is chosen

Server - Fixed leaked file handles in both itemdb and modbus

- Also improved logging in coke handler

Client - Cleaned up subcommands

Server - added 'install' make target

Client - Replaced some magic values/strings with #defines

Implemented new relay and threaded timing

Fixed warnings in cokebank

Fixed up some error reporting

Client - Allowed menu to cursor onto expensive items (not selectable)

server/door - cleared door serial buffer on open, just in case

Server - Disabled snack dispensing, handled by vendserv

Implimented pin checks fully

Pin checks and disabled account behavior

Client - Fixed funny colours

Added a manpage makefile

Server - Implimented pins in server code

Noted libmodbus requirement in the README

Client - Added colours and fixed help string

Client - Broke the client out into separate files

- Added help to the ncurses display

Allowed transferring to/from disabled accounts

Coke - Dropped minimum period to 2s (about what the hardware limit is)

Server - Changed logging to note when a test dispense is done

Client - Fixed edge case where -coke is interpreted as a count of dispenses

Server - Deduplicated PIDfile name

Server - Moved to new door control (via 115200 USB serial)

Client - Better scrolling (jk and arrows work)

Documented `dispense donate`

Fixes from when the new coke controller was tested

Fixed test mode defaulting to on when invalid value

Added zero-cost testing hack

Fixes to new coke code, appears to work now

Added basic error reporting to coke code

Slight clean to coke code

Merge branch 'master' of serenade.mutabah.net:opendispense2

Commenting in server.c

Cleaning up warnings in client

Debugging authident, adding new config file

Removed an unneeded error message from client

Adding new config system, template modbus coke, AUTHIDENT in client

Server - Slight fixes to coke handler to cope with the machine being dodge

[PATCH] server - make the finger output historically accurate

Server - Added bac to mark zero priced slots as erroring

Added finger support

Server - Fixed SETEUSER not checking disabled flag

Misc changes - Fixed coke, shut up boot, fixed disabled users dispensing

Applied fixes to AUTHIDENT - works

- Thanks goes to Zanchey [DAA] for AUTHIDENT
- Also disabled coke debug

[PATCH] client - spell amount correctly in help text

[PATCH] server - allow IDENT auth by anyone from a trusted subnet

The door, it is fixed

Applied patches from DAA

- [PATCH 1/2] manpage: spell amount correctly
- [PATCH 2/2] manpage: tidy up trailing commas
- [PATCH 1/2] .gitignore - ignore compiled client binary
- [PATCH 2/2] .gitignore - ignore compiled shared libraries

Server - Add authentication by IDENT support

Fixed client makefile for clean

Fixed door for direct serial

Client - Switched back to cents display, fixed an infinite loop in ReadLine

Fixed price display

Client - Fixed scrolling code

client - Fixed some bugs with long prices

Set baud rate to 1200 for door code

Changed door code to use direct serial instead of llogin

- The DECServer gave up the magic smoke on the weekend

Manual page :)

Client - Fix to possible infinite loop in ReadLine

- If the remote host closes, recv() returns 0, this was not checked
 > Caused 100% CPU on motsugo when un in a backup script

Renamed liability account

Client - Documentation for `enumitems` (commented out)

Server - Root behavior

- Disallowed heathred
- Disallowed root to use `dispense acct`
 > Exception for new accounts (matches "treasurer: new user")
- Fixed username not being set for AUTOAUTH

Fixed client error, added dead snack catch, fixing server io redirection

Extra help info, fixed logging for dispense failures, possible fix for logging errors

Added merlo to trusted list and changed default server to merlo

[TPG] Fixes to bugs exposed during move

- Serial Input/Output flags were left unset
- Coke debug enabled
- Empty line detection added to slot status reads.
- Fixed compile issues in client (OpenSSL is no longer needed)

Fixed bug with item selection (>= instead of > for price check)

Multiple fixes

- Client fixed to pass error messages correctly
- Fiddling with client ReadLine (was infinite looping, should be fixed now)
- Fixed server to start the timer thread in the main process (not the parent)
- Added a hack to detect refund `dispense acct` messages and reject them

Fixing some issues spotted by graham's run of clang's static analyser