[Drbd-dev] Re: [DRBD-cvs] svn commit by lars - r2551 - in branches/drbd-0.7: . drbd drbd/linux scripts - fix some odities in the paranoia checks of our build sy

[Lars Marowsky-Bree <lmb@suse.de>]

[-- Attachment #1: Type: text/plain, Size: 1055 bytes --]

On 2006-10-19T16:56:00, drbd-cvs@lists.linbit.com wrote:

> Author: lars
> Date: 2006-10-19 16:55:58 +0200 (Thu, 19 Oct 2006)
> New Revision: 2551
> 
> Added:
>    branches/drbd-0.7/scripts/get_uts_release.sh
> Modified:
>    branches/drbd-0.7/Makefile
>    branches/drbd-0.7/drbd.spec.in
>    branches/drbd-0.7/drbd/Makefile
>    branches/drbd-0.7/drbd/drbd_worker.c
>    branches/drbd-0.7/drbd/linux/drbd_config.h
>    branches/drbd-0.7/scripts/adjust_drbd_config_h.sh
> Log:
> fix some odities in the paranoia checks of our build system,
> should build agains 2.6.18 now out of the box.

That actually introduced a bug: building the tgz now requires the
kernel source to be installed, which is in fact only required when the
kernel modules are built (and which could happen on another system).

How about the rather simple patch attached instead?



-- 
High Availability & Clustering
SUSE Labs, Research and Development
SUSE LINUX Products GmbH - A Novell Business	 -- Charles Darwin
"Ignorance more frequently begets confidence than does knowledge"


[-- Attachment #2: drbd.diff --]
[-- Type: text/x-patch, Size: 641 bytes --]

Index: Makefile
===================================================================
--- Makefile	(revision 2564)
+++ Makefile	(working copy)
@@ -160,9 +160,6 @@
 
 KDIR := $(shell echo /lib/modules/`uname -r`/build)
 KVER := $(shell KDIR=$(KDIR) O=$(O) scripts/get_uts_release.sh)
-ifeq ($(KVER),)
-$(error "could not determine uts_release")
-endif
 
 kernel-patch: drbd/drbd_buildtag.c
 	set -o errexit; \
@@ -173,6 +170,10 @@
 
 # maybe even dist/RPMS/$(ARCH) ?
 rpm: tgz
+	@if [ -z "$(KVER)" ]; then \
+		echo "Could not determine uts_release" ; \
+		false ; \
+	fi
 	mkdir -p dist/BUILD \
 	         dist/RPMS  \
 	         dist/SPECS \