respect $manage_repos, do not include ::apt if set to false

[fraenki]

PR #250 introduced code that unconditionally includes ::apt. This should be changed to respect wether $manage_repos is actually set to true or not.

Otherwise I'm unable to use my own repo management, because the include leads to a duplicate declaration:

Error: Could not retrieve catalog from remote server: Error 400 on SERVER: Evaluation Error: Error while evaluating a Resource Statement, Duplicate declaration: Class[Apt] is already declared

[dhoppe]

@fraenki Thank you very much for your contribution.

[joshuaspence]

Sorry, I commented on the commit without seeing the corresponding pull request. I'll fix a PR.

[roll4life]

@joshuaspence tested your fix on ubuntu 16.04, and it does resolve the issue with php7.0-cli, but if you have dev => true or pear => true it throws Error: Could not find dependency Class[Apt::Update] for Package[php7.0-xml] this is important because pear is required for pecl, which on ubuntu is necessary for php-redis, php-memcached etc . . . since those packages do not adhere to the php7.0 prefix

[joshuaspence]

When you say that you tested my fix, did you mean #323?

[roll4life]

Yes, that is correct. Sorry I should have specified.

[joshuaspence]

Right, okay. I think that problem isn't caused by my pull request though. It seems like an existing issue.

[roll4life]

correct. That is an issue of include ::apt missing from manifests/dev.pp and manifests/pear.pp
at least in puppet 3. I can create a separate issue if needed.