]> xenbits.xensource.com Git - people/dariof/libvirt.git/commitdiff
spec: Don't mix commands with macro definitions
authorJiri Denemark <jdenemar@redhat.com>
Tue, 30 Jul 2013 14:04:48 +0000 (16:04 +0200)
committerJiri Denemark <jdenemar@redhat.com>
Tue, 30 Jul 2013 15:13:40 +0000 (17:13 +0200)
%build section should first define all required macros and then run
commands. Interleaving them makes it harder to spot what commands are
run.

libvirt.spec.in

index 156146296e79c14b1e4061134d13fae4052319ea..fce7f9126dd8bb1af520069d2569da5df24096ae 100644 (file)
@@ -1343,10 +1343,6 @@ of recent versions of Linux (and other OSes).
     %define init_scripts --with-init_script=redhat
 %endif
 
-%if 0%{?enable_autotools}
- autoreconf -if
-%endif
-
 %if %{with_selinux}
     %if 0%{?fedora} >= 17 || 0%{?rhel} >= 7
         %define with_selinux_mount --with-selinux-mount="/sys/fs/selinux"
@@ -1355,6 +1351,12 @@ of recent versions of Linux (and other OSes).
     %endif
 %endif
 
+# place macros above and build commands below this comment
+
+%if 0%{?enable_autotools}
+ autoreconf -if
+%endif
+
 %configure %{?_without_xen} \
            %{?_without_qemu} \
            %{?_without_openvz} \