Merge pull request #273 from hunner/fix_trustcert

Fix :false to be default value

Fix :false to be default value

false is not a valid value; only :false is.

Merge pull request #262 from webfactory/autorequire-hg

Autorequire Package['mercurial']

Merge pull request #272 from tphoney/dos2unix_readme

dos2unix the readme

dos2unix the readme

Merge pull request #269 from monai/develop

Add feature 'depth' and parameter 'trust_server_cert' to svn

Add feature depth and param trust_server_cert to svn

added param trust server cert

updated depth feature

updated README.markdown

trust_server_cert is not feature

trust_server_cert default value should be false

add test for depth and trust_server_cert

Merge pull request #257 from alexmv/master

Explitly unset SSH_AUTH_SOCK; it is preferred to the -i flag

Merge pull request #266 from puppet-by-examples/example-shallow-clone-with-just-one-commit

Example how to create a shallow clone with just one commit

Example how to create a shallow clone with just one commit

Merge pull request #265 from mhaskel/MODULES-1232

MODULES-1232 Make sure HOME is set correctly

MODULES-1232 Make sure HOME is set correctly

Remove 2.7 from travis matrix

I believe 2.7 in the matrix is an error, this module has never claimed
support for 2.7 and 2.7 currently doesn't work (though fails in places
that are not unit tested)

Merge pull request #264 from hunner/fix_hang

Fix acceptance hang

Fix acceptance hang
The ssh-keygen is waiting for a 'y' to confirm overwrite, and the test hangs.

Merge pull request #263 from mhaskel/MODULES-2326

MODULES-2326 - Run Regexp.escape on the source URL

MODULES-2326 - Run Regexp.escape on the source URL

The URL can have special characters, making the regex match fail.

Autorequire Package['mercurial']

Along the lines of 2b190756260346931b8f9a0dda8afc0c815710d6, if the
Mercurial package is being managed, it stands to reason that the Mercurial
package should be installed before trying to potentially manage Mercurial
repositories using vcsrepo resources.

This commit adds an autorequire to the vcsrepo type that reflects the
above premise.

Merge pull request #261 from mhaskel/MODULES-2125

MODULES-2125 - Allow revision to be passed without source

MODULES-2125 - Allow revision to be passed without source

Will also work with empty repositories.

Merge pull request #260 from mhaskel/MODULES-1800

MODULES-1800 - fix case where ensure => latest and no revision specified

MODULES-1800 - fix case where ensure => latest and no revision specified

This would explode when revision was unspecified when you were on a
branch. Use the branch you're currently on when updating.

Merge pull request #259 from mhaskel/merge_1.3.x_to_master

Merge 1.3.x to master

Merge pull request #258 from mhaskel/1.3.1-prep

1.3.1 prep

1.3.1 prep

Explitly unset SSH_AUTH_SOCK; it is preferred to the -i flag

Even if the -i flag is explicitly passed via the command line, openssh
ignores the identity file if an socket exists to an SSH authentication
agent.  In cases where puppet has been configured to use an explicit
identity file, altering behavior based on the calling environment of
puppet violates the principle of least surprise, and can lead to
inconsistent deployments.

Work around this odd corner case of ssh by explicitly unsetting
SSH_AUTH_SOCK inside the ssh wrapper.

Merge pull request #256 from keeleysam/master

fix for detached HEAD on git 2.4+

fix for detached HEAD on git 2.4+

Merge pull request #242 from BillWeiss/bump-ssl-expiry

Make sure the embedded SSL cert doesn't expire

Merge pull request #255 from jonnytpuppet/depth_fix

MODULES-2131 Git provider now takes account of revision property when using depth property.

MODULES-2131 Git provider now takes account of revision property when using depth property.

Merge pull request #252 from DavidS/add-https-clone-test

acceptance: Add a test verifying anonymous https cloning

Merge pull request #254 from hunner/add_helper

Add helper to install puppet/pe/puppet-agent

Add helper to install puppet/pe/puppet-agent

Merge pull request #253 from justinstoller/maint/master/set-puppet-version

(maint) allow setting PUPPET_VERSION in acceptance

(maint) allow setting PUPPET_VERSION in acceptance

acceptance: Add a test verifying anonymous https cloning

In MODULES-891 the question arose whether https sources are supported.
This test shows that it works.

Merge pull request #232 from netors/master

Consider the submodules flag on specific revision cloning

Add ability to unittest puppet 4

Merge pull request #251 from mhaskel/merge_1.3.x_to_master

Merge 1.3.x to master

Merge branch 'psoloway-readme' into 1.3.x

Last-minute README corrections

* Corrected the list of supported platforms under Limitations
* Spelled “revision spec” instead of “revisionspec”.

Merge pull request #249 from hunner/release_1.3.0

Release 1.3.0

Release 1.3.0

Merge pull request #246 from psoloway/readme

Update README per DOC-1501

Merge pull request #248 from hunner/fix_tmp

(MODULES-821) Don't use /tmp

(MODULES-821) Don't use /tmp

Very often /tmp is mounted noexec for security reasons related to it
being writable by all users. This stopped vcsrepo's git provider from
working.

This pull request uses puppet's statedir as it is not writable by all
users and is a good candidate for transient state like the git
provider's identity script for the $GIT_SSH command.

An alternative in the future that wouldn't require a temporary file is
to set $GIT_SSH_COMMAND instead of $GIT_SSH, except this was added in
git 2.3 and is too new to depend on.

Merge pull request #247 from puppetlabs/modulesync_updates

Modulesync updates

beaker gemfile fixes

sync via modulesync

sync via modulesync

Merge pull request #241 from tykeal/master

Clean up puppet lint warnings

Merge pull request #237 from cmurphy/rspec-puppet-2

Testing updates

Merge upstream changes into readme branch

Update README per DOC-1501

1. Remove any "what this affects" sections, except where particularly warranted.
2. Make sure that for each parameter, where applicable, there is a data type and a default value.
3. Make sure that for each parameter that's applicable, there is a note if the parameter is optional.
4. Make sure the links in the README work and are accurate.
5. Update the link in the Contributing section to point here: https://docs.puppetlabs.com/forge/contributing.html
6. General copyediting.

Merge pull request #224 from puppet-by-examples/single-branch

Shallow clone with one commit that corresponds to arbitrary tag from arbitrary branch

Merge pull request #243 from rnelson0/quotes

Enforce the style guide's recommendation of single quotes as the default.

Enforce the style guide's recommendation of single quotes as the default.

Just bumping the expiration date...

Clean up puppet lint warnings

There were puppet lint warnings due to indentation in several examples.
This commit cleans up the warnings.

Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>

Merge pull request #220 from ddisisto/1.2.x_add_svn_accept

Add support for 'conflict' parameter to populate svn --accept arg

Merge pull request #238 from anodelman/master

(BKR-147) add Gemfile setting for BEAKER_VERSION for puppet...

Merge pull request #240 from cmurphy/hash_ordering

Fix remote hash ordering for unit tests

Fix remote hash ordering for unit tests

Without this commit, the unit tests for the git provider changing
multiple remotes mocks the remotes in a particular order. While in
practice it doesn't matter which remote the update_remotes method
updates first, the unit tests must be able to mock them in the correct
order. For ruby 1.8.7, a Hash will not necessarily produce key value
pairs in the same order on each run, which causes intermittent failures
in the unit tests. This change sorts the :source property values before
trying to update them, and updates the unit tests to expect the values
in alphabetical order.

(BKR-147) add Gemfile setting for BEAKER_VERSION for puppet...

puppetdb, etc

- support for BEAKER_VERSION and BEAKER_RSPEC_VERSION in gemfile

Update .travis.yml

This commit makes the following changes to the test matrix:

- Runs tests on ruby 2.1.5 instead of 2.0.0
- Runs tests on ruby 1.8.7 with puppet 3.x
- Adds an environment to run on an intermediate 3.x puppet version
  instead of latest (specifically 3.4.x)
- Adds an environment to run with the future parser on latest puppet.
  This would affect the test runs for the validate and spec checks.
- Runs the tests on docker

Keep testing on puppet 2.7

The modulesync config repo is dropping universal support for puppet
2.7, but individual repos should still keep support until the next
naturally-occuring major release.

Unpin rspec-puppet and remove unneeded deps

puppetlabs_spec_helper takes care of these for us.

(MODULES-1551) Add support for SVN conflict handling

Merge pull request #236 from dummey/master

Updating 'excludes' in readme to be more concise.

Update README.markdown

Clarified what the parameter 'excludes' is intended to do.

Merge pull request #235 from cmurphy/rspec

Pin rspec gems

Pin rspec gems

Bugfix: removed duplicated condition

Added submodules true condition before update_submodules on revision method

Use branch parameter

Merge pull request #230 from jonnytpuppet/multiple_remotes

Implemented multiple remotes feature for git provider.

Implemented multiple remotes feature for git provider.

Merge pull request #218 from dduvnjak/master

Add submodules feature to git provider

Add submodules feature to git provider

Merge pull request #229 from hunner/update_before

Update let to use instance variable instead.

Remove let use as variable.

Since lets can't be used in before :all any more.

Merge pull request #228 from tphoney/removing_private_tests

removing private tests

removing private tests

removing private tests, due to rspec3 not handling private methods

Merge pull request #226 from cmurphy/master

Add IntelliJ files to the ignore list

Add IntelliJ files to the ignore list

Merge pull request #225 from mhaskel/MODULES-1596

MODULES-1596 - Repository repeatedly destroyed/created with force

MODULES-1596 - Repository repeatedly destroyed/created with force

The `retrieve` method was calling `create` and `destroy` on every run
with `force => true`. Retrieve should not be making any changes to the
system, so removed that code, and updated `working_copy_exists` to make
sure that the directory not only contains a `.git` directory, but also
if `source` is specified it also matches `#{path}/.git/config` so that
it will overwrite a git repo with a different source.

Updated tests to not check for the old broken behavior. Added a regression test.

Merge pull request #223 from infopro-digital/MODULES-1597

Fix for MODULES-1597: "format" is a file not a directory

Fix for MODULES-1597: "format" is a file not a directory

The "format" entry in a SVN directory is a file, not a directory.

Merge pull request #222 from cmurphy/master

Update .travis.yml, Gemfile, Rakefile, and CONTRIBUTING.md

Add json gem

The puppet_facts gem implicitly depends on the json gem.
On Ruby 1.8.7, json is not built in. On Puppet 2.7, it is
not explicitly pulled in by the hiera gem. So we add it
here explicitly.

Update .travis.yml, Gemfile, Rakefile, and CONTRIBUTING.md

Merge pull request #219 from laurenrother/readme

Updates README per MODULES-1425

Updates README per MODULES-1425

Adds note to Setup section stating that the module will not create parent directories or install VCS software.

Merge pull request #217 from mhaskel/test

Merge 1.2.x into master

Merge pull request #216 from mhaskel/missing_doc

Add missing doc update.

Add missing doc update.

Merge pull request #215 from mhaskel/1.2.0-prep

1.2.0 prep

1.2.0 prep