We got rid of the funny proxy removal

Merge pull request #479 from mhaskel/update_anchors

(MODULES-1156, MODULES-769) Update anchors

(MODULES-1156, MODULES-769) Remove unnecessary anchors

`apt::ppa` and `apt::setting` don't actually include `apt::update` so
anchors are unnecessary. Move `apt` to use contain instead of anchors,
since it wasn't anchoring properly anyways. Update the tests to make
sure it can have settings and ppas depending on each other without
cycles.

README: update['frequency'] is not a boolean

README: Add highlighting for code blocks.

Fix typo

Merge pull request #477 from mhaskel/doc_updates

Doc updates

README updates for 2.x rewrite

All classes, defines, parameters, and examples should be up to date.

Merge pull request #472 from mhaskel/acceptance_test_fixes

Acceptance test fixes

Merge pull request #473 from mhaskel/freq_cleanup

Remove `update['always'] = true` support

Merge pull request #474 from mhaskel/containers_ftw

Linux open-source builds are regularly maxed out.

Linux open-source builds are regularly maxed out.

Container-based are not. Let's switch!

Remove `update['always'] = true` support

We don't really need both of `update['always'] = true` and
`update['frequency'] = 'always'`.

Make the acceptance tests work

Actually make it possible to use apt_key

The provider wasn't updated for a parameter rename.

Merge pull request #461 from mhaskel/boo_inheritance

Inheritance of apt::params means it can't be private

Inheritance of apt::params means it can't be private

Otherwise, if another module has `class { 'apt': }` in it everything
fails, as `$caller_module_name` will be the other module name.

Merge pull request #453 from puppetlabs/daenney/location-required

apt::source: Make location required.

apt::source: Make location mostly required.

In what universe does it make sense to create a `sources.list.d` entry
for a repository **without** specifying where this repository is?
:confounded:

:disappointed: :weary: :anguished: :scream:

Only when removing the resource should a location not be required.

Merge pull request #452 from puppetlabs/daenney/rename-trusted-source

apt::source: Rename `trusted_source`.

apt::source: Rename `trusted_source`.

It is weird that `trusted_source` would default to `false` as that would
imply that we normally don't trust our sources. This is opposite to the
truth, by default we trust them but only if the Releases file can be
verified (meaning it is signed by a GPG key known to apt).

What we were telling apt is that it should trust this source even if the
Releases file and the repository is unsigned. This is better captured
with `allow_unsigned` and better highlights the danger of what you're
doing, installing packages from a source we cannot authenticate.

Merge pull request #451 from puppetlabs/daenney/consolidate-include

apt::source: Merge `include_*` options into hash.

apt::source: Merge `include_*` options into hash.

This makes its behaviour similar to the `update`, `proxy` and `purge`
hashes on the main classes bringing its API more in line with the rest
of the module.

Merge pull request #450 from puppetlabs/daenney/guard-lsb-facts

apt::params: Complete $xfacts.

apt::params: Complete $xfacts.

`$xfacts` now contains and guards all `lsb*` facts. Looking up any
`lsb*`-related fact should now always be done through `$::apt::xfacts`
to ensure that the values are always set to either the value of the fact
or undef. This avoids all sorts of kerfuffles with strict variables.

Merge pull request #449 from puppetlabs/daenney/fix-strict-vars

apt: Fix all strict variable cases.

apt: Fix all strict variable cases.

A few of these fixes are absolutely horrendous but we have no choice as
we need to stay current- and future-parser compatible for now.

Once we can go Puppet 4 only we can use the `$facts` hash lookup instead
which will return undef/nil for things that aren't set instead of them
not being defined at all.

Merge pull request #448 from puppetlabs/daenney/kill-file-perms

apt::setting: Remove file_perms.

apt::setting: Remove file_perms.

This was a great idea but is pretty pointless. It's also not being used
by anything and not exposed as a switch on the main class so it would
almost never affect any behaviour.

Merge pull request #447 from puppetlabs/daenney/hulk-smash-2

It's been a long time - how have you been

apt::key: Be explicit about the keyserver.

The behaviour of passing down undef through multiple layers gets fuzzy
so for now be explicit about the keyserver.

Once Puppet 4 is out and this behaviour has been crystallised and tested
we can revisit it.

apt::key: Rename $key to $id to match apt_key.

apt::source: Allow passing in a complex key.

Turn `$key` into something that accepts a string or a hash of four keys
representing the different options that can be passed on to `apt::key`.

apt(::key|_key) Remove the `key.*_` prefixes.

Remove a lot of the redundant `key_` prefixes on `apt::key` and the
`keyserver_` prefix on `apt_key`.

apt: Add settings, keys and ppas.

* Allow any configuration of apt to be done through data bindings by
  passing in hashes representing the resources.
* Switch apt::ppa to use `distid` as set in `apt::params. This makes
  `apt::ppa` also work for LinuxMint.

apt: Change how update is managed.

* Instead of having 4 options controlling purging we now have a single
  hash with four possible keys.
* Include `apt::update` only _after_ we've assembled the `$_update`
  hash.

apt: Change how purging is managed.

* Instead of having 4 options controlling purging we now have a single
  hash with four possible keys.
* We purge everything by default.
* `/etc/apt/preferences` is now always managed.
* Add missing `mode` to some of the files.

Merge pull request #446 from puppetlabs/daenney/apt-proxy

apt: Add proxy support on the class.

apt: Add proxy support on the class.

Re-introduce proxy support at the class level. Needing to configure a
proxy is such a common scenario that having it on the class is a
reasonable thing. It also affects `apt::ppa`.

Change `apt::ppa` to no longer have its own `proxy` parameter but use
the proxy as configured on the main `apt` class.

spec/(apt|ppa): Enough with all the strings.

Most options can and should be named through symbols, makes it much
easier to read too with syntax highlighting.

Merge pull request #445 from mhaskel/updates_everywhere

Make apt::setting notify Exec['apt_update'] by default

Make apt::setting notify Exec['apt_update'] by default

Can be disabled by setting `notify_update => false`

Merge pull request #444 from puppetlabs/daenney/parse-setting-from-name

apt::setting: Parse type and name from title.

apt::setting: Parse type and name from title.

Instead of having two additional parameters, `base_name` and
`setting_type` simply parse it from `title`.

We need to prefix most resources with `list-`, `conf-`, or `pref-` any
way to avoid duplicate resources so we might as well leverage that.

Merge pull request #443 from mhaskel/proxy_updates

proxy_* params were removed from class apt

Merge pull request #441 from mhaskel/use_setting

Convert to use apt::setting instead of file resource

Convert to use apt::setting instead of file resources

This adds consistency across the module

proxy_* params were removed from class apt

Add them to PPA since they were being used there, and add a placeholder
example for setting up the proxy files.

Merge pull request #442 from mhaskel/add_base_name

Add base_name parameter to apt::setting

Merge pull request #440 from mhaskel/fix_setting_validation

Allow priorities to be zero-padded

Add base_name parameter to apt::setting

This allows you to work around duplicate resource issues when you have
settings of different types with the same name. When the files are built
it is path/${priority}${base_name}${extension}.

Allow priorities to be zero-padded

Merge pull request #439 from mhaskel/rspec3

Convert specs to RSpec 3.1.7 syntax with Transpec

Merge pull request #438 from puppetlabs/daenney/cant-touch-params-nanananana

apt::params: Make the class private.

Convert specs to RSpec 3.1.7 syntax with Transpec

This conversion is done by Transpec 3.0.8 with the following command:
    transpec spec/classes spec/defines spec/unit

* 87 conversions
    from: it { should ... }
      to: it { is_expected.to ... }

* 14 conversions
    from: obj.should
      to: expect(obj).to

* 7 conversions
    from: == expected
      to: eq(expected)

* 1 conversion
    from: it { should_not ... }
      to: it { is_expected.not_to ... }

For more details: https://github.com/yujinakayama/transpec#supported-conversions

apt::params: Make the class private.

Prevent direct access to apt::params. This will ensure that any other
module cannot blindly access apt::params and get settings that have been
potentially overridden at the apt level.

Our own module still can since any class in apt has a module_name of
'apt' but that's up to us to prevent from happening.

Every setting must now be accessed by a qualified lookup into the apt
namespace.

Merge pull request #433 from mhaskel/ppa

PPA Cleanup, pt 1

PPA Cleanup, pt 1

Make the code much cleaner, but don't make a t&p.

Merge pull request #432 from mhaskel/source_cleanup

Cleanup for `apt::source`

Convert apt::source to use apt::setting

May as well use the define since we added it

Cleanup for `apt::source`

* Update `release` to default to `$::lsbdistcodename`
* Default `include_src` to false
* Validate more things!
* Stop redefining variables from `apt::params`

Merge pull request #435 from mhaskel/setting_fix

Type is a reserved word in puppet 4

Merge pull request #431 from mhaskel/local_variable_declarations

Stop redeclaring variables from params

Stop redeclaring variables from params

It really seems unnecessary.

Type is a reserved word in puppet 4

So replace `type` with `setting_type` in apt::setting

Merge pull request #428 from puppetlabs/daenney/apt-setting

apt: Add apt::setting defined type.

apt: Add apt::setting defined type.

This is a 'base' type. It's a simple wrapper around a file which takes
`type`, `ensure`, `content`, `source` and `file_perms`. It is intended
for usage by `apt::conf`, `apt::source` and an upcoming `apt::pref`.

Merge pull request #430 from mhaskel/fix_typo

Fix typo from #426

Merge pull request #429 from mhaskel/remove_include_update

Remove 'include apt::update'

Fix typo from #426

Too many `$`s

Remove 'include apt::update'

It is included in `class apt`, and there are no promises about anything
working without that.

Merge pull request #426 from puppetlabs/daenney/fix-missing-param-and-naming

apt::params: Add two missing entries, use them.

Merge pull request #427 from mhaskel/required_packages

RFC - Remove required packages

apt::params: Add two missing entries, use them.

Remove required packages

I'm not entirely clear on the history behind this feature, and this
feels sort of hack-y. If you could explain why this is needed that would
be awesome, or if it isn't just merge this :)

Merge pull request #424 from mhaskel/hulk_smash

Hulk smash

Fix lint issue in examples

Fix acceptance tests

Fix broken unit tests

Placeholders for needed examples

Update examples

Get rid of hold

Get rid of release

Get rid of force

get rid of builddep

Merge pull request #1 from puppet-community/daenney/remove_unattended_upgrades_remains

Remove some vars for unattended_upgrades.

Remove some vars for unattended_upgrades.

stop including params everywhere

Remove backports and debian

move tests to examples

Get rid of unattended upgrades and cleanup unused templates

and an acceptance test

Unbreak broken tests

Starting to clean up init.pp

Merge pull request #422 from cmurphy/lint

Ensure linter fails on lint warnings

Merge pull request #421 from cmurphy/future

Update spec tests for future parser

Ensure linter fails on lint warnings

Update spec tests for future parser

Strings must be quoted in the future parser.

Merge pull request #389 from stdietrich/global_header_partial

(MODULES-1200) Fix inconsistent header across files

(MODULES-1200) Fix inconsistent header across files