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 #437 from fraenki/fix_example

fix hiera example in documentation

fix hiera example in documentation

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 #425 from nibalizer/master

Remove travis badge

Remove travis badge

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

Merge pull request #416 from mkrakowitzer/add_param_documentation

(doc) Add documentation for apt::key parameters and apt::pin parameters

Merge pull request #418 from cmurphy/rspec

Pin rspec gems

Pin rspec gems

(doc) Add documentation for apt::key parameters and apt::pin parameters

Add documentation for apt::key parameters and apt::pin parameters.
Fix minor spelling error

Merge pull request #414 from phaf/fix_doc_unattended_upgrades

Documentation: Fix typo and syntax for example of apt::unattended_upgrad...

Documentation: Fix typo and syntax for example of apt::unattended_upgrades

Merge pull request #398 from irconan/backports-pin

Separate apt::pin for apt::backports to allow pin by release instead of ...

Separate apt::pin for apt::backports to allow pin by release instead of origin

Updated tests for backports

Merge pull request #395 from udienz/master

Add Ubuntu vivid (15.04) release

Merge pull request #411 from mhaskel/trusted_fix

Trusted will be a reserved word in Puppet 4

Trusted will be a reserved word in Puppet 4

Rename the new `trusted` parameter to `trusted_source`.

Merge pull request #407 from mkrakowitzer/f3792625/trusted_param

Add support for parameter trusted MODULES-1658

Merge branch 'master' of https://github.com/puppetlabs/puppetlabs-apt into f3792625/trusted_param

Merge pull request #403 from WolverineFan/fix_apt_updates_facts

Fix apt_has_updates fact not parsing apt-check output correctly

Fix apt_has_updates fact not parsing apt-check output correctly

The /usr/lib/update-notifier/apt-check script returns its output
to STDERR but a recent change to the script redirects STDERR to
/dev/null.  This will cause the array to always be empty.

Combined with that problem, while we were checking for the result
being nil, we never checked for an invalid array.  As a result,
the apt_has_updates was always true and the apt_updates and
apt_security_updates facts were trying to read from an empty array
and failing.

Merge pull request #409 from cyberious/MOD1661

MODULES-1661 Fix to do delete with short key not long

MODULES-1661 Fix to do delete with short key not long

Merge pull request #408 from cyberious/MODULES-1661

MODULES-1661 Fix issue with apt_key destroy, also added mutliple deletes

MODULES-1661 Fix issue with apt_key destroy, also added mutliple deletes

Add support for parameter trusted

* Add support for paramater trusted, valid options are 'true' and false.
defaults to false. True sets the value to trusted=yes.

trusted=yes can be set to indicate that packages from this source are
always authenticated even if the Release file is not signed or the
signature can't be checked.

* Update documentation