mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-09-30 11:25:47 +00:00
spec: Don't mix commands with macro definitions
%build section should first define all required macros and then run commands. Interleaving them makes it harder to spot what commands are run.
This commit is contained in:
parent
d5cd5bf031
commit
462863a3ee
@ -1343,10 +1343,6 @@ of recent versions of Linux (and other OSes).
|
|||||||
%define init_scripts --with-init_script=redhat
|
%define init_scripts --with-init_script=redhat
|
||||||
%endif
|
%endif
|
||||||
|
|
||||||
%if 0%{?enable_autotools}
|
|
||||||
autoreconf -if
|
|
||||||
%endif
|
|
||||||
|
|
||||||
%if %{with_selinux}
|
%if %{with_selinux}
|
||||||
%if 0%{?fedora} >= 17 || 0%{?rhel} >= 7
|
%if 0%{?fedora} >= 17 || 0%{?rhel} >= 7
|
||||||
%define with_selinux_mount --with-selinux-mount="/sys/fs/selinux"
|
%define with_selinux_mount --with-selinux-mount="/sys/fs/selinux"
|
||||||
@ -1355,6 +1351,12 @@ of recent versions of Linux (and other OSes).
|
|||||||
%endif
|
%endif
|
||||||
%endif
|
%endif
|
||||||
|
|
||||||
|
# place macros above and build commands below this comment
|
||||||
|
|
||||||
|
%if 0%{?enable_autotools}
|
||||||
|
autoreconf -if
|
||||||
|
%endif
|
||||||
|
|
||||||
%configure %{?_without_xen} \
|
%configure %{?_without_xen} \
|
||||||
%{?_without_qemu} \
|
%{?_without_qemu} \
|
||||||
%{?_without_openvz} \
|
%{?_without_openvz} \
|
||||||
|
Loading…
Reference in New Issue
Block a user