Fixed '.' and '-' not being accepted by a commands module.

Fixes issue #2384.
Fix restarting of standard_services with invalid class characters
Closes https://cfengine.com/dev/issues/2404
Merge pull request #490 from nickanderson/bugfix/masterfiles/libraries/cfengine_stdlib/canonify_standard_service_restart_context_restriction

cfengine_stdlib: Fix restarting of standard_services with invalid class characters
Merge pull request #488 from kacfengine/issue-2384

Fixed '.' and '-' not being accepted by a commands module.
Fixed issue with package management and warn.

Previously action_policy => warn would have no effect, and CFEngine
would go ahead and fix the problem anyway.

Fixes issue #1831.
Merge pull request #487 from kacfengine/issue-1831

Fixed issue with package management and warn.
Automatically handle holes in CopyRegularFileDisk

This will allow to remove makeholes from Promise struct
Get rid of Promise.makeholes, file copying may do it automatically
Add acceptance test for dry-run option in packages promises (Redmine #1831)
Allow empty bundles (Redmine #2411)
Added test case to issue #2384.
Error out if unknown bundle type is detected in policy (Redmine #2410)
De-clutter output in case cf-promises has found errors (Redmine #1507)
Fix whitespace
Fix broken StringFormat/StringVFormat
Beautify syntax error reporting (Redmine #1444)
Add unit test for StringFormat/StringVFormat
Merge pull request #491 from kacfengine/issue-2384-tc

Added test case to issue #2384.
Fix double free on current_line. RM #2418
Merge pull request #492 from baptr/issue-2418

Fix double free on current_line. RM #2418