Merge pull request #72 from mckern/master

(maint) Update CentOS 7 mock stubs to use internal release repos
(RE-2368) Update for CentOS 7

We can now remove the bandaids because CentOS 7 has been officially
released. This consists of removing the bandaid files themselves and
updating the mocks to remove lingering patch code. We also have updated
the pe-el-mock-config template to pull from osmirror instead of yo.
Merge pull request #73 from clantant/RE-2368/master/rel7

(RE-2368) Update for CentOS 7
(maint) Untab mock templates, use install for chroot setup on el7

Adding tabs to the mock template makes it easier to read, but breaks
python's parsing of the file. Additionally, mixing groups and packages
means that we need to use install instead of groupinstall for the el7
case.
Merge pull request #74 from haus/maint/master/untab_mock_use_install

(maint) Untab mock templates, use install for chroot setup on el7
(RE-2465) Remove unnecessary proxy declaration from PE mock configs

Because we have internal mirrors of el and sles, we shouldn't need to
use a proxy for anything at all. This commit removes the proxy
declarations from the el and sles mock configs.
(RE-2465) Remove mirrorlist declarations from PE mock configs

Because all of the PE EL oses are mirrored internally, we shouldn't need
to configure a mirrorlist for them. This combined with setting a proxy,
causes bad things to happen if the repodata between the two doesn't
agree (for say relative pathing). This commit removes those declarations
from the el and sles templates.
(RE-2466) Remove legacy config, roll into EL mock config

Now that we have internal mirrors of el4, the mock config doesn't need
to substantially deviate from the current EL mock config. Conditionals
are added here to add the group definitions for el4 and to not define
the epel repo for el4, but it is otherwise identical.
(maint) Enable build-tools repo on all EL dists for PE

Previously the build-tools repo was only enabled on el 5 and 6. With the
addition of the legacy mock to the template, el4 is also required. And
el7 has a build-tools repo available as well. This commit updates that
logic to just check for el and add the repo if on el.
(RE-2466) Remove legacy mock defined types

Previous commits removed the need for a legacy mock in templates by
adding a few conditional stanzas. This commit makes a similar change to
the puppet code, removing the pe_legacy_mock and pe_legacy_mockset
defined types. It also updates the pe_mocks and pe_mockset to include
el4 mocks for PE.
(maint) Make chroot_setup_cmd consistent in pe el mock template
Merge pull request #75 from haus/ticket/master/re_2465_re_2466

Fix broken el7 mocks, remove legacy mock config
(re-1897) Add sles 10 mocks to rpmbuilder

This adds SLES 10 to the PE mocksets. Since we will typically
build against SLES sp1, but we need to build against sp4 for SLES 10
(as it is the only supported version currently) this adds a parameter for
the SLES mocks that defaults to 'sp1' and is overridden for the SLES 10
mocks.
Update sles mocks to use the "osmirror" alias
(re-1897) Add conditional logic for dep repo for sles11 and fix arch for sles10
Merge pull request #76 from shrug/maint/master/sles10

(re-1897) Add sles 10 mocks to rpmbuilder
(maint) Update release comparisons in pe-mock template to strings

Previously the @release variable was compared to the integers 4 and 7,
which always returns false because 4 and 7 will be "4" and "7", which
are strings. This commit updates the pe-el mock template to do string
comparisons instead of integer comparisons. It also updates the erb
conditionals to not generate extra whitespace for easier to read mock
configs.
Merge pull request #77 from haus/maint/master/strings_not_integers

(maint) Update release comparisons in pe-mock template to strings
(re-1897) Update sles mocks with local repo location and change release comparisons to be string compares
(bugfix) stop comparing strings and integers

It looks like this conditional is never being satisfied because we are
comparing a string and an int. Rather than saying `@release.to_i`, I
think it's cleaner just to say `"7"`. This will get the correct value of
`config_opts['chroot_setup_cmd']` on el7.