Merge remote-tracking branch 'shared/master'

Merge pull request #38 from jpoppe/master

added (file) $replace parameter to concat

added description of replace in the comment block

added (file) $replace parameter to concat

Merge pull request #35 from Zugschlus/master

please consider pulling

gracefully handle \n in fragment resource name

refer to manifest files for option docs

fix markup

Prepare for version 0.2.0

Merge pull request #32 from UnifiedPost/develop

Fixes various puppet-lint warnings.

Fixes various puppet-lint warnings.

* ./manifests/init.pp - WARNING: ensure found on line but it's not the first attribute on line 222
* ./manifests/setup.pp - WARNING: selector inside resource block on line 34
* ./manifests/fragment.pp - WARNING: case statement without a default case on line 28

Merge pull request #31 from UnifiedPost/develop

Allow using a custom name and provide a path to the file that needs to be created using concat.

Document the path parameter and provide a small use case.

Add a second spec test for the name/path combination

Allow overriding the target path of a (concat) file and use a custom name.

Merge pull request #29 from x-way/master

Add puppet-lint and rspec-puppet gems to make Travis CI work again

Add puppet-lint and rspec-puppet gem for Travis CI

Merge pull request #28 from x-way/master

Remove trailing whitespace to make puppet-lint happy

Remove trailing whitespace in comment

puppet-lint does not like trailing whitespaces

Merge pull request #27 from djmitche/auto-setup

Include concat::setup from concat, so users don't need to

Include concat::setup from concat, so users don't need to

Merge pull request #26 from djmitche/patch-1

Remove spurious 'e' character.

Remove spurious 'e' character.

Merge pull request #25 from branan/module_package_name

Fix module name to make the PMT happier

Fix module name to make the PMT happier

Merge pull request #24 from pabelanger/improvements

Improvements

Reformatting for puppet-lint
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>

Add Travis CI support
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>

Lets use puppetlabs_spec_helper
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>

update README.markdown to clearly indicate how to start using the concat module

Merge pull request #23 from sathlan/feature/add-basic-tests

Feature/add basic tests

Cleanup.

Adding basic tests.

Successfully tested inside rvm:
  - ruby 1.8.7-p302;
  - puppet (2.7.13)
  - facter (1.6.9)
  - bundler (1.1.3)
  - diff-lcs (1.1.3)
  - metaclass (0.0.1)
  - mocha (0.11.4)
  - puppet-lint (0.1.13)
  - rake (0.9.2.2)
  - rspec (2.8.0)
  - rspec-core (2.8.0)
  - rspec-expectations (2.8.0)
  - rspec-mocks (2.8.0)
  - rspec-puppet (0.1.3)
  - rubygems-bundler (1.0.2)
  - rvm (1.11.3.3)

Merge branch 'master' of github.com:ripienaar/puppet-concat

Merge pull request #21 from mrwacky42/basedir_warning

Fail with helpful advice if $::concat_basedir is not yet set

Fail with helpful advice if $::concat_basedir is not yet set

Building on the new docs in https://github.com/ripienaar/puppet-concat/pull/20

Merge pull request #20 from endemics/master

add information about concat_basedir and the need for pluginsync = true at least once

add information about concat_basedir and the need for pluginsync = true at least once

add files needed for the module forge

Merge pull request #18 from dalen/master

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

Style guideline fixes

Comply better with puppet community style guidelines.

Change-Id: Ie1782d266539b1a8f9890b2ade77280011aa22aa

Merge pull request #17 from pdxcat/refactor_remove_gnuflag

Remove the gnu parameter from concat

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.

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.

Merge pull request #12 from mrwacky42/master

Unset silly Nexenta environment variable.

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 #11 from cwarden/fully-qualify

Fix use of unqualified variable names

Fix use of unqualified variable names

Fully qualify $::id, $::puppetversion, and $::concat_basedir.

Merge pull request #8 from duritong/master

use client's vardir - make module usable for unprivileged users

Merge remote-tracking branch 'ripienaar/master'

Merge pull request #10 from mediatemple/master

Fix for /bin/sh on Solaris.

On Solaris, 'test -a' does not work with /bin/sh

     -a file                 True if file exists. (Not  available
                             in sh.)

Merge pull request #5 from TimBaldoni/master

updated to better support dash

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.

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.

Merge remote-tracking branch 'tim/master'

Merge pull request #7 from mrwacky42/master

As discussed .. Make $concatdir only readable by root.

Make sure interesting data in the concatdir is only available to root

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 the root group configurable in the concat::setup class for portability
- use sh and not bash

Fix doco typo

add "g" option to getopts

Update changelog

add a -g flag to concatfragments.sh to disable GNU extensions to find, sort and xargs; makes concat work on Solaris.

Add a $gnu parameter to make use of the aforementioned -g flag, and a $order parameter to make use of the -n flag in concatfragments.sh

Update cangelog

Parametrize warning message

The concat warn parameter can now be any string (and needs to include
the appropriate commenting character) and will then override the default
(which still gets printed when warn=>true).

Signed-off-by: martin f. krafft <madduck@madduck.net>

Fix the Usage example so it parses and orders properly

Make the filebucket behavior of files configurable

improve documentation

improve documentation

Markdown really is pretty rubbish and inconsistent.

Markdown tweaks

Add markdown readme and clear up old info from README

update README with a sample

Improve error message and documentation

fix spacing/tab mixup

Pull in work from private Subversion repository