From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15949 invoked by alias); 17 Nov 2006 02:45:53 -0000 Received: (qmail 15931 invoked by uid 9447); 17 Nov 2006 02:45:52 -0000 Date: Fri, 17 Nov 2006 02:45:00 -0000 Message-ID: <20061117024552.15928.qmail@sourceware.org> From: agk@sourceware.org To: lvm-devel@redhat.com, lvm2-cvs@sourceware.org Subject: LVM2 ./WHATS_NEW tools/pvremove.c Mailing-List: contact lvm2-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Post: List-Help: , Sender: lvm2-cvs-owner@sourceware.org X-SW-Source: 2006-11/txt/msg00015.txt.bz2 List-Id: CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: agk@sourceware.org 2006-11-17 02:45:52 Modified files: . : WHATS_NEW tools : pvremove.c Log message: Fix pvremove error path for case when PV is in use. Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.499&r2=1.500 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/tools/pvremove.c.diff?cvsroot=lvm2&r1=1.10&r2=1.11 --- LVM2/WHATS_NEW 2006/11/16 17:36:00 1.499 +++ LVM2/WHATS_NEW 2006/11/17 02:45:51 1.500 @@ -1,5 +1,6 @@ Version 2.02.15 - ==================================== + Fix pvremove error path for case when PV is in use. Warn if certain duplicate config file entries are seen. Enhance lvm_dump.sh for sysreport integration and add man page. Fix --autobackup argument which could never disable backups. --- LVM2/tools/pvremove.c 2006/05/09 21:23:51 1.10 +++ LVM2/tools/pvremove.c 2006/11/17 02:45:51 1.11 @@ -92,9 +92,9 @@ } if (!dev_test_excl(dev)) { - log_error("Can't open %s exclusively. Mounted filesystem?", - dev_name(dev)); - return 0; + log_error("Can't open %s exclusively - not removing. " + "Mounted filesystem?", dev_name(dev)); + goto error; } /* Wipe existing label(s) */