(#12917) Add contributors to README

Prior to this commit, this modules README had no list of contributors.
This commit adds such a list and corrects a typo I overlooked on the
last pull request.

Merge pull request #26 from pdxcat/ticket_12823_aptkey_defined_type

(#12823) Add apt::key defined type and modify apt::source to use it

Reviewed and tested by Ryan Coleman (ryan@puppetlabs.com)

(#12823) Add apt::key defined type and modify apt::source to use it

Adding this defined type allows puppet to add keys to the apt keystore without
needing to add a corresponding source; it also adds the "key_source" parameter
for wget'ing keys from arbitrary URLs, and allows for keys to be explicity
removed.

apt::key allows a key to be ensured present multiple times to account for
apt::source resources that all reference the same key. However, this means
that it is possible for a given key to be defined multiple times with
differing source parameters. e.g.

apt::key { "Add key: 4BD6EC30 from Apt::Source bunny":
  key        => "4BD6EC30",
  key_server => "pgp.mit.edu",
}

apt::key { "Add key: 4BD6EC30 from Apt::Source rabbit":
  key        => "4BD6EC30",
  key_server => "keyserver.ubuntu.com",
}

The defined type will accept both definitions and will create multiple exec
resources. This was deemed preferable to the alternative (creating only one
exec resource) in that one broken definition won't hose an entire catalog. If
one definition fails to install the key because of a bad "key_server", the
next apt::key that uses the key will get it done.

Merge pull request #27 from blkperl/ticket_12809_refactor_release

(#12809) $release should use $lsbdistcodename and fall back to manual in...

Reviewed and tested by Ryan Coleman (ryan@puppetlabs.com)

(#12809) $release should use $lsbdistcodename and fall back to manual input

This commit changes $release to default to Facter's $lsbdistcodename
and fall back to a Parse Error if $release is not set and $lsbdistcodename
does not exist. Previously $release was hardcoded to karmic.

This commit also modifies apt::ppa to use $release and sets the
files to be ensured so that they are not purged when purge_sources_list_d
is set to true.

Merge pull request #23 from haus/python_properties

Check if python-software-properties is defined before attempting to define it.

Reviewed and tested by Ryan Coleman (ryan@puppetlabs.com)

Merge pull request #22 from haus/unique_required_packages

Allow duplicate required_packages between sources

Reviewed and tested by Ryan Coleman (ryan@puppetlabs.com)

Merge pull request #24 from haus/fix_source_key_tests

Fix source specs to test all key permutations

Reviewed and tested by Ryan Coleman (ryan@puppetlabs.com)

Merge pull request #25 from rcoleman/ticket_12522_purge_option_take_2

(#12522) Split purge option to spare sources.list

(#12522) Adjust spec test for splitting purge

Now that the purge parameter is split into two, the spec test needs
adjusted. This commit makes that adjustment.

(#12522) Split purge option to spare sources.list

Prior to this commit, when using the purge option, unmanaged entries
in both /etc/apt/sources.list and sources.list.d would be purged.

This commit splits purge into purge_sources_list and purge_sources_list_d
which handle the purging of those items separately. Brief documentation on
each added to class documentation.

Fix source specs to test all key permutations

Previously only one should in each block was being evaluated. This moves each
should to its own block so that all values are tested. It also adds another set
of params so that all useful permutations of key, key_server, and key_content
are generated. It also replaces the previous ternary assignment for param_set
with a hash merge.

Add test for python-software-properties package

This test adds the precondition that the python-software-packages package be
installed before the apt class is synced. If the defined function were not
called around the package resource, this test would fail with a duplicate
package resource error.

Check if python-software-properties is defined before attempting to define it.

Add tests for required_packages change

This updates the tests for the apt::source defined type to use the new name for
the exec resource and the new parameter for the same exec.

Allow duplicate $required_packages

Previously, if more than one apt::source required the same packages to be
installed it would fail with a duplicate exec resource. This adds the name of
the source resource to the exec and gives the exec a name, moving it to a
command parameter for the exec.

Merge branch 'allow-same-key'

(#12430) Add tests for changes to apt module

This update reflects the changes to the apt module to allow duplicate keys. It
mostly involves tests for changes to the resource names to make them unique
between defines.

Test two sources with the same key

Merge pull request #17 from blkperl/ticket_12526_apt_disable_keys

(#12526) Add ability to reverse apt { disable_keys => true }

Reviewed by Ryan Coleman (ryan@puppetlabs.com)

(#12526) Add ability to reverse apt { disable_keys => true }

The setting `disable_keys => true` parameter in the apt module creates
/etc/apt/apt.conf.d/99unauth with the contents
"APT::Get::AllowUnauthenticated 1;". Changing `disable_keys`
does not remove this file. This patch makes it so that
`disable_keys => false` will remove /etc/apt/apt.conf.d/99unauth.

Merge pull request #20 from blkperl/ticket_12522_purge_option

(#12522) Adding purge option to apt class

Reviewed by Ryan Coleman

Merge pull request #1 from rcoleman/ticket_12522_purge_option

Add Modulefile to puppet-apt

Add Modulefile to puppet-apt

Now that puppet-apt depends on puppetlabs-stdlib, an explicit
dependency is required. Modulefile was entirely missing. This commit
adds on.

Allow the use of the same key in multiple sources

Allow the use of the same key in multiple sources by including the name
of the source in the declaration for the exec which adds the key.

(#12522) Adding purge option to apt class

Adds a purge option to the apt class to remove repositories
that are not managed by apt::source

Merge pull request #18 from bodepd/apt_proxy_tests

(#12529) Add unit test for apt proxy settings

(#12529) Add unit test for apt proxy settings

This commit adds unit tests to validate that the
apt proxy is configured as expected when the class
parameter proxy_host is set as a class parameter for
the apt class.

Merge pull request #15 from bodepd/proxysupport

(#12529) Add parameter to support setting a proxy for apt

(#12529) Add parameter to support setting a proxy for apt

This commit adds two class parameter to apt that can be used to
specify a proxy to use with apt.

  - proxy_host
  - proxy_port

Merge pull request #14 from haus/ticket/12094_add_spec_tests

Ticket/12094 add spec tests

(#12094) Replace chained .with_* with a hash

The hash passing to the with method is cleaner and closer to puppet code, so
all of the with_$param have been replaced with with($hash). This also
includes two minor whitspace changes to unstable.pp and source.pp.
This also replaces the ternary switch on param_set with a hash merge,
which is cleaner and will support more use cases.

(#12094) Remove deprecated spec.opts file

Rspec no longer uses the spec.opts file, and this particular file also uses
options for rspec that are no longer honored or available, like loadby. So this
commit removes the file. Users that want to customize rspec behavior should use
a .rspec file in the repository root or in their home directory.

(#12094) Add rspec-puppet tests for apt

This commit adds full coverage for the apt module as it currently exists. It
adds rspec-puppet tests for the defines (apt::builddep, apt::force, apt::pin,
apt::ppa, apt::source) and classes (apt, debian::testing, debian::unstable,
apt::params, apt::release).

(#12094) Replace name with path in file resources

When the name is used with a file resource, it becomes difficult to test the
resource using rspec-puppet, as the name parameter gets aliased to path. So to
maintain consistency between tests and manifests, this replaces all name
parameters in file resources with the equivalent path parameter.

Merge pull request #4 from webfactory/pin_before_source

Suggested fix for #10449

Merge pull request #12 from haus/11953_fix_apt_force_version

(#11953) Apt::force passes $version to aptitude

(#11953) Apt::force passes $version to aptitude

Previously, even if $version were passed to apt::force, aptitude would just
install the default version of the package available. This updates the aptitude
call to use the version string if it exists. If no version is passed, or if it
is false, no version is added to the aptitude call.  This also updates the
rspec-puppet tests for the define, to reflect the changes to the exec.

Merge pull request #11 from bodepd/11413_apt_force_query_fix

11413 apt force query fix

Merge pull request #8 from cwarden/source-example-improvement

include_src parameter in example; Whitespace cleanup

(#11413) Add spec test for apt::force to verify changes to unless

In the Previous commits, the query being done by the
apt::force install command was not strict enough and
could lead to false positives.

These queries represented by the unless parameter have been
resolved in another commit. This commit accompanies that commit
and adds basic unit tests to correspond to the changes.

(#11413) Update dpkg query used by apt::force

This patch fixes the query used by apt::force to
determine rather or not the package is installed.

Previously, the expression was not specific enough and
could not lead to false positives in cases where a package
name is contained within another package name (puppet could be
incorrectly determined as being installed if puppet-common is
installed)

This commit resolves that by improving the query expression.

Merge pull request #9 from bodepd/10451_fix_apt_update_refresh

10451 fix apt update refresh

(#10451) Add test coverage to apt::ppa

This commit adds test coverage for apt::ppa.

This test coverage is suficient to verify the
code changes that resolve the issue with
aptitude update not being called when ppas were
added (#10451).

include_src parameter in example; Whitespace cleanup

In the apt::source example, include the include_src parameter.  Clean-up
whitespace in the example to be consistent with the style guide.

Merge pull request #7 from cwarden/patch-1

fix typos in "repositories"

fix typos in "repositories"

Fix (#10451) - apt::ppa fails to "apt-get update" when new PPA source is added

Set the pin priority before adding the source (Fix #10449)

Merge pull request #3 from rnavarro/master

Fixed the infinite ppa add bug.

Refactored as per mlitteken

* https://github.com/rnavarro/puppet-apt/commit/1af9a13c40c33e1ab8fb56f33cb9aa5e16cead0e#commitcomment-539287

Added some crazy bash madness to check if the ppa is installed already. Otherwise the manifest tries to add it on every run!

(#8720) Replace Apt::Ppa with Apt::Builddep

Probably a copy & paste error

Reviewed-by: Matthaus Litteken <matthaus@puppetlabs.com>
Signed-off-by: Spencer Krum <spencer@puppetlabs.com>

added builddep command.

added the ability to specify the content of a key

Allows us to pass key_content when we specify a source.

Fixes ppa test.

reformatted whitespace to match recommended style
of 2 space indentation.

ignore swap files.

added smoke tests for module.

reformatted apt::ppa according to recommended
style.

removed require apt in favor of marking a
dependency.

converted release define into a class since
the resources are singletons.

Created a params class to hold global data.

- Removes coupling between global data and
  resources from apt class.
- Makes it easier to organize things into stages.

Added two params for apt class

both of these params facilitate options that
ease the management of apt repos in dev
environments

1. disable_keys - allows repos without properly
signed keys
2. always_apt_update - refreshes via apt update
every time that puppet runs.

Update LICENSE.

Support ability to add more than one PPA

Add call to apt-update after add-apt-repository in apt::ppa

Add package definition for python-software-properties

Abs paths for all commands

Merge branch 'master' of github.com:afternoon/puppet-apt

Merge branch 'master' of github.com:evolvingweb/puppet-apt

Merge branch 'master' of github.com:evolvingweb/puppet-apt

Adding LICENSE

Heading fix in README

Typo in README

First commit