2 fixes for ansible.spec

[dermute]

Fix 1: SLES build:

• python_sitelib just like with RHEL
• BuildRequires: python-devel instead of python2-devel
• Requires: python-PyYAML instead of PyYAML

Fix 2: remove refs to absent man pages (man3):

• cp in %install
• %doc in %files

[dirtyharrycallahan]

There is a previous pull request to "fix" the SuSE rpm build.

[mpdehaan]

Might be useful to link the ticket number :)

[dirtyharrycallahan]

The earlier pull request is #4689
I quoted "fix" because not all of the RPMs are available from SuSE for all versions or at least I could not find them in the repos my other administrators mirrors and transfers across our airgap. I ended up pulling the tar balls then using setup.py to build the rpms. Since I did not rename the rpms created by setup.py I override them.

[jctanner]

@dermute Please split the second fix out into a new pull request.

[l1k]

Just my 2 cents on this patch:

As to "#2: remove refs to absent man pages": @mpdehaan originally uploaded an incomplete tarball to PyPI for release 1.3.4 (see issue #4804) which didn't contain the man3 manpages, @dermute probably based this patch on that incomplete tarball. That portion of the patch is therefore no longer necessary.

As to "#1: fix for SLES build": The clause "BuildArch: noarch" only works for openSUSE versions later than 11.1 and must therefore be encapuslated in an appropriate %if clause ("%if %{?suse_version: %{suse_version} > 1110} %{!?suse_version:1}"). This patch will therefore not work on openSUSE 11.1 and earlier. You may want to consider my pull request #5042 which doesn't have that deficiency. That pull request also includes the appropriate %fdupes stuff which is required for the package to be accepted by rpmlint on build.opensuse.org.

Also, please consider cherry-picking commit 2f25330 from pull request #5042 which contains fixes to ansible.spec that are unrelated to SUSE and thus of general use. Thanks.

[jctanner]

@l1k 2f25330 has been partially committed in cdbce4a

[jctanner]

@dermute as of 4422219 a lot your fixes are merged. Could you validate against your suse systems and report and bugs or make a new PR with fixes please?

[l1k]

@jctanner I've reviewed and tested the changes you've made and issued pull request #5281 to correct a few remaining errors. Thanks.