public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
* Minor install fix
@ 2007-07-14  1:20 John Lenz
       [not found] ` <469824E8.1010809-d0mtIEKrxfl4piUD7e9S/g@public.gmane.org>
  0 siblings, 1 reply; 2+ messages in thread
From: John Lenz @ 2007-07-14  1:20 UTC (permalink / raw)
  To: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f

I noticed kvm 29 added a command into make install
to move the distro installed kvm.ko modules out of
the way.

But for me, I had already removed the drivers/kvm
directory, so make install was getting a file does
not exist error.  So I added a test for the directory.

John


--- kvm-29/kernel/Makefile      2007-07-12 05:20:47.000000000 -0500
+++ kvm-29.mod/kernel/Makefile  2007-07-13 20:13:45.000000000 -0500
@@ -41,7 +41,9 @@
 install:
        mkdir -p $(DESTDIR)/$(INSTALLDIR)
        cp *.ko $(DESTDIR)/$(INSTALLDIR)
-       for i in $(ORIGMODDIR)/drivers/kvm/*.ko; do mv $$i $$i.orig; done
+       if [ -d $(ORIGMODDIR)/drivers/kvm ]; then \
+         for i in $(ORIGMODDIR)/drivers/kvm/*.ko; do mv $$i $$i.orig; done \
+       fi
        /sbin/depmod -a

 tmpspec = .tmp.kvm-kmod.spec

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Minor install fix
       [not found] ` <469824E8.1010809-d0mtIEKrxfl4piUD7e9S/g@public.gmane.org>
@ 2007-07-15  6:34   ` Avi Kivity
  0 siblings, 0 replies; 2+ messages in thread
From: Avi Kivity @ 2007-07-15  6:34 UTC (permalink / raw)
  To: John Lenz; +Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f

John Lenz wrote:
> I noticed kvm 29 added a command into make install
> to move the distro installed kvm.ko modules out of
> the way.
>
> But for me, I had already removed the drivers/kvm
> directory, so make install was getting a file does
> not exist error.  So I added a test for the directory.
>
>   

I fixed this in a slightly different way.  Thanks.

-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-07-15  6:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-07-14  1:20 Minor install fix John Lenz
     [not found] ` <469824E8.1010809-d0mtIEKrxfl4piUD7e9S/g@public.gmane.org>
2007-07-15  6:34   ` Avi Kivity

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox