Dash (default /bin/sh on Debian and Ubuntu) does not support 'echo -e'.  That invocation of echo is not POSIX compliant: https://bugs.launchpad.net/ubuntu/+source/dash/+bug/72167
Make sure interesting data in the concatdir is only available to root
Merge pull request #7 from mrwacky42/master

As discussed .. Make $concatdir only readable by root.
Merge remote-tracking branch 'tim/master'
don't use a hardcoded concatdir

Get the concat base directory from a fact -> makes it useable if
Puppet's :vardir is not /var/lib/puppet/. This fixes problems with
PE and makes the module also useable for puppet runs as
unpriviledged user.
finish work for unpriviledged user

To be able to use the module as an unprivileged user we need to
adjust certain things:

* only enforce the run user and group if we are root and can
  actually change the user
* set owner/group of our files and directories to our own user/group
* place the concat script in a location we can write -> use the
  concat dir for that.
Merge pull request #5 from TimBaldoni/master

updated to better support dash
On Solaris, 'test -a' does not work with /bin/sh

     -a file                 True if file exists. (Not  available
                             in sh.)
Merge pull request #10 from mediatemple/master

Fix for /bin/sh on Solaris.
Merge remote-tracking branch 'ripienaar/master'
Merge pull request #8 from duritong/master

use client's vardir - make module usable for unprivileged users
Fix use of unqualified variable names

Fully qualify $::id, $::puppetversion, and $::concat_basedir.
Merge pull request #11 from cwarden/fully-qualify

Fix use of unqualified variable names
Remove silly environment variable that corrupts reality.

Not sure how this is set in our puppet environment, but it is.
http://nexenta.org/projects/site/wiki/Personalities
Merge pull request #12 from mrwacky42/master

Unset silly Nexenta environment variable.
Fix comments to match actual behavior, clean up old script.

* Zap the old copy of /usr/local/bin/concatfragments.sh.
* Fix comments to point at new location of script.
* Delete trailing whitespace.
Remove the gnu parameter from concat

Previously, the concatfragments.sh script was would default to using
GNU-specific flags for find, sort, and xargs. This necessitated
explicit passing of a "gnu = false" parameter to the concat define in
order to successfully run the script without GNU-specific flags when
working with systems that do not by default include GNU versions of the
utilities (solaris, for example).

This commit modifies the concatfragments script such that GNU versions
of the utilities are not needed at all. It does this while preserving
the original use case for the GNU flags, which was to allow special
characters (like spaces) in the filenames, which it accomplished by
using GNU flags to separate fields using null characters instead of
newlines.

In order to preserve backwards-compatibility with existing puppet
installations that make use of the "gnu = false" parameter, the
parameter list for the concat define has not been changed. Rather, the
gnu parameter is now deprecated and ignored.
Merge pull request #17 from pdxcat/refactor_remove_gnuflag

Remove the gnu parameter from concat
Style guideline fixes

Comply better with puppet community style guidelines.

Change-Id: Ie1782d266539b1a8f9890b2ade77280011aa22aa
Merge pull request #18 from dalen/master

A patch to make puppet-concat better comply with community style guidelines.