Look at NI instead of PRI in ps output on Linux

The PRI field is the Linux kernel scheduling priority which is
different from the niceness of a process (which is what a system
adminitrator and policy-writer is likely to want to select upon). This
value is almost always larger than 20, so it isn't selectable if the
defined range per the documentation is -20,+20 anyway.
Out-of-tree builds broke again, fix.
Fix: display file mode in octal, not decimal
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