Akio Takebe wrote:
> Hi,
>
> I updated the patch of the following thread.
> http://thread.gmane.org/gmane.comp.emulators.kvm.devel/8899
> Changelog:
> - don't change Makefile
> - put back some options (BuildRequires and so on).
>
>
Thanks, it's much improved.
However, kvm.spec has already changed, so the patch won't apply.
Please pull from kvm.git to get the latest sources. Also note the
comments below.
> If you try my patch with kvm-49, you can do the below.
> 1. rm SOURCES
> 2. mkdir SOURCES
> 3. ./configure
> 4. make rpm
>
> I tested this patch on RHEL5.
>
> Signed-off-by: Akio Takebe <[EMAIL PROTECTED]>
>
> Best Regards,
>
> Akio Takebe
>
> ---
>
> --- kvm-49.orig/kvm.spec 2007-10-29 02:14:57.000000000 +0900
> +++ kvm-49/kvm.spec 2007-10-31 23:24:05.000000000 +0900
> @@ -1,3 +1,12 @@
> +%define Distribution %(rpm -q -qf /etc/redhat-release --qf '%{name}' | cut
> -d -
> -f 1)
> +%define os_release %(rpm -q --qf '%%{version}' /etc/redhat-release)
> +%define bindir /usr/bin
> +%define bin %{bindir}/kvm
> +%define initdir /etc/init.d
> +%define confdir /etc/kvm
> +%define utilsdir /etc/kvm/utils
> +%define _prebuilt %{?prebuilt:1}%{!?prebuilt:0}
> +
>
Why are you moving this around? It makes the patch harder to review
because you can't immediately see if the code has changed.
> Name: kvm
> Version: 0.0
> Release: 0
> @@ -11,20 +20,8 @@ BuildRoot: %{_tmppath}/%{name}-%{ve
> ExclusiveArch: i386 x86_64
>
> Requires: kvm-kmod bridge-utils
> -
> -%define Distribution %(cat /etc/redhat-release | awk '{ print $1}' | tr
> [A-Z]
> [a-z])
> -%define os_release %(rpm -q --qf '%%{version}' %{Distribution}-release)
> -
> -%if %{Distribution} == "fedora" && %{os_release} == 5 || %{Distribution} ==
> "centos" && %{os_release} == 4
> -BuildRequires: compat-gcc-32
> -%else
> -BuildRequires: compat-gcc-34
> -%endif
> -
> BuildRequires: SDL-devel zlib-devel alsa-lib-devel
>
> -%define _prebuilt %{?prebuilt:1}%{!?prebuilt:0}
> -
> %if !%{_prebuilt}
> Source0: kvm.tar.gz
> Source1: user.tar.gz
> @@ -32,15 +29,32 @@ Source2: kernel.tar.gz
> Source3: scripts.tar.gz
> Source4: Makefile
> Source5: configure
> -Source5: kvm_stat
> +Source6: kvm_stat
> %endif
>
> %description
> The Kernel Virtual Machine provides a virtualization enviroment for
> processors
> with hardware support for virtualization: Intel's VT and AMD's AMD-V.
>
> -%prep
> +%if %([ x"%{Distribution}" = x"fedora" -a x"%{os_release}" = x"5" ] && echo
> 1 |
> | echo 0)
> +%define fedora5 1
> +%endif
>
Why not use regular rpm conditionals instead of calling the shell?
Can you add an explicit test for RHEL 5?
--
Any sufficiently difficult bug is indistinguishable from a feature.
-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel
