Commit graph

39 commits

Author SHA1 Message Date
R.I.Pienaar
12f38d86ee add files needed for the module forge 2012-04-30 10:45:08 +01:00
R.I.Pienaar
e9c97b8820 Merge pull request #18 from dalen/master
A patch to make puppet-concat better comply with community style guidelines.
2012-04-24 08:01:43 -07:00
Erik Dalén
800bec11ae Style guideline fixes
Comply better with puppet community style guidelines.

Change-Id: Ie1782d266539b1a8f9890b2ade77280011aa22aa
2012-04-24 16:53:20 +02:00
R.I.Pienaar
b4a80d4655 Merge pull request #17 from pdxcat/refactor_remove_gnuflag
Remove the gnu parameter from concat
2012-04-17 12:08:05 -07:00
Reid Vandewiele
81d5ee80f3 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.
2012-04-17 12:02:38 -07:00
Sharif Nassar
30c96286c9 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.
2011-12-14 13:45:46 -08:00
R.I.Pienaar
bd21154fa0 Merge pull request #12 from mrwacky42/master
Unset silly Nexenta environment variable.
2011-12-13 22:56:07 -08:00
Sharif Nassar
9dac53afc8 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
2011-12-13 20:24:10 -08:00
R.I.Pienaar
61c8d5c75b Merge pull request #11 from cwarden/fully-qualify
Fix use of unqualified variable names
2011-12-06 09:11:59 -08:00
Christian G. Warden
7722415753 Fix use of unqualified variable names
Fully qualify $::id, $::puppetversion, and $::concat_basedir.
2011-12-05 16:19:24 -08:00
R.I.Pienaar
0cd6bacfc9 Merge pull request #8 from duritong/master
use client's vardir - make module usable for unprivileged users
2011-08-03 06:44:57 -07:00
Peter Meier
185adddae8 Merge remote-tracking branch 'ripienaar/master' 2011-08-03 14:27:37 +02:00
R.I.Pienaar
4a08cd40b5 Merge pull request #10 from mediatemple/master
Fix for /bin/sh on Solaris.
2011-07-12 01:35:11 -07:00
Sharif Nassar
4dff5634b3 On Solaris, 'test -a' does not work with /bin/sh
-a file                 True if file exists. (Not  available
                             in sh.)
2011-07-11 16:54:20 -07:00
R.I.Pienaar
103504b310 Merge pull request #5 from TimBaldoni/master
updated to better support dash
2011-06-23 15:20:24 -07:00
Peter Meier
b9e33c4e6b 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.
2011-06-23 13:57:09 +02:00
Peter Meier
45dfa6984c 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.
2011-06-23 11:05:43 +02:00
Peter Meier
cfda8dd6ac Merge remote-tracking branch 'tim/master' 2011-06-23 10:17:32 +02:00
R.I.Pienaar
41a8ea2799 Merge pull request #7 from mrwacky42/master
As discussed .. Make $concatdir only readable by root.
2011-06-21 14:44:34 -07:00
Sharif Nassar
96936d9efb Make sure interesting data in the concatdir is only available to root 2011-06-20 13:31:25 -07:00
Tim
0c5dcc82c4 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 2011-04-20 14:54:27 -07:00
Pierre-Yves Ritschard
031bf26128 - make the root group configurable in the concat::setup class for portability
- use sh and not bash
2011-02-03 18:01:36 +00:00
R.I.Pienaar
3c46b82d09 Fix doco typo 2011-01-19 17:36:20 +00:00
Jonathan Boyett
03ac60902b add "g" option to getopts 2010-12-14 11:06:27 -08:00
R.I.Pienaar
131a157272 Update changelog 2010-12-03 20:55:34 +00:00
Jonathan Boyett
fa80fc2b31 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
2010-12-03 12:35:23 -08:00
R.I.Pienaar
40245d9fcc Update cangelog 2010-10-04 22:48:47 +01:00
martin f. krafft
af63a0b3eb 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>
2010-10-04 22:46:18 +01:00
Chris Jones
2a3abe58a6 Fix the Usage example so it parses and orders properly 2010-08-04 07:18:27 +08:00
R.I.Pienaar
18a7897992 Make the filebucket behavior of files configurable 2010-07-14 13:01:45 +01:00
R.I.Pienaar
1fb38b2961 improve documentation 2010-05-22 11:14:28 +01:00
R.I.Pienaar
86f1837c70 improve documentation 2010-05-22 11:09:47 +01:00
R.I.Pienaar
7590f36dbd Markdown really is pretty rubbish and inconsistent. 2010-05-14 15:44:20 +01:00
R.I.Pienaar
6532f7cf9d Markdown tweaks 2010-05-14 15:40:49 +01:00
R.I.Pienaar
d21209f7c9 Add markdown readme and clear up old info from README 2010-05-13 19:48:30 +01:00
R.I.Pienaar
166891e281 update README with a sample 2010-05-12 16:35:46 +01:00
R.I.Pienaar
d09acb4a42 Improve error message and documentation 2010-05-07 19:15:21 +01:00
R.I.Pienaar
ad89c2d075 fix spacing/tab mixup 2010-05-06 22:56:26 +01:00
R.I.Pienaar
ee1fe7a023 Pull in work from private Subversion repository 2010-05-06 22:55:02 +01:00