[libvirt] [PATCH 2/2] spec: indent %if to make it easier to see conditions

[Viktor Mihajlovski]

On 01/10/2013 12:45 AM, Eric Blake wrote:
> * libvirt.spec.in: Add some indentation to make it easier to follow
> various conditionals.
> ---
>   libvirt.spec.in | 858 ++++++++++++++++++++++++++++----------------------------
>   1 file changed, 428 insertions(+), 430 deletions(-)
>
Painful experience makes me agree that the unindented nested %ifs 
decrease the readability of the spec file and thus would be in favor of 
this patch.
However, indentation seems to be uncustomary. On the other hand, it
doesn't seem to be explicitly forbidden, looking at e.g.
http://fedoraproject.org/wiki/Packaging:Guidelines
Any packager here that wants to throw in 2 cents?

>   # Fedora 18 / RHEL-7 are first where firewalld support is enabled
>   %if 0%{?fedora} >= 17 || 0%{?rhel} >= 7
None of your doing, but the contradiction between comment (F18) and
directive (F17) should be resolved.

-- 

Mit freundlichen Grüßen/Kind Regards
    Viktor Mihajlovski

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martina Köderitz
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294