Merge branch 'master', remote branch 'upstream/master'
Do not display settings when target is failtarget
[os_execd] fix compiler warnings
[os_execd] fix CID 28574
[os_execd] add nonnull qualifiers
[os_execd] fix windows build
[os_execd] fix windows build #2
[os_execd] fix windows warnings
Merge pull request #359 from cgzones/os_execd

os_execd
Do not display completion message when 'failtarget'

Per the suggestion of cgzones in #357, the if statement has been
extended so that the completion message is not displayed when doing a
build for 'failtarget'.
Merge pull request #357 from awiddersheim/fix_build_settings

Fix build settings
Fix compile warnings printing size_t

Original fix in 8b9566bd was enough to suppress the warning when
building the 32bit version of the Windows agent but the 64bit builds
still complain. An extra layer of encapsulation was required.
Merge pull request #360 from awiddersheim/fix_win32_warning

Fix compile warnings printing size_t
[logcollector] fix compiler warnings
[logcollector] fix clang analyzer warning: Null pointer passed as an argument to a 'nonnull' parameter
[logcollector] remove duplicated statement
[logcollector] fix CID 28448
[logcollector] fix CID 28447
[logcollector] fix CID 28519
remove strange code, relating to logcollector