diff mbox

[2/2] kvm: configure: --with-patched-kernel doesn't need kernelversion

Message ID 1232966602-28492-2-git-send-email-markmc@redhat.com (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Mark McLoughlin Jan. 26, 2009, 10:43 a.m. UTC
Signed-off-by: Mark McLoughlin <markmc@redhat.com>
---
 configure |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Glauber Costa Jan. 26, 2009, 10:57 a.m. UTC | #1
On Mon, Jan 26, 2009 at 8:43 AM, Mark McLoughlin <markmc@redhat.com> wrote:
> Signed-off-by: Mark McLoughlin <markmc@redhat.com>

Why not?

I use it regularly to unequivocally define what kernel tree I'm using.
The default
is often not the right choice.
Mark McLoughlin Jan. 26, 2009, 11:10 a.m. UTC | #2
On Mon, 2009-01-26 at 08:57 -0200, Glauber Costa wrote:
> On Mon, Jan 26, 2009 at 8:43 AM, Mark McLoughlin <markmc@redhat.com> wrote:
> > Signed-off-by: Mark McLoughlin <markmc@redhat.com>
> 
> Why not?
> 
> I use it regularly to unequivocally define what kernel tree I'm using.
> The default
> is often not the right choice.

It is only used by depmod when installing kvm.ko ... which we don't do
with --with-patched-kernel

Cheers,
Mark.

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/configure b/configure
index 1fd133c..c3910cd 100755
--- a/configure
+++ b/configure
@@ -119,7 +119,7 @@  if [ -d "$kerneldir/include2" ]; then
     kernelsourcedir=${kerneldir%/*}/source
 fi
 
-if [ -n "$no_uname" ]; then
+if [ -n "$no_uname" -a "$want_module" ]; then
     if [ -e "$kerneldir/.kernelrelease" ]; then
         depmod_version=`cat "$kerneldir/.kernelrelease"`