[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#484272: partman-auto-lvm: Should support for explicit logical volume names in recipes



Package: partman-auto-lvm
Version: 27
Severity: wishlist
Tags: patch

The attached patch adds support for explicitely preseeding a logical
volume name (instead of computing one) in recipes.

This is done by using "lvname{ <name> }", which fits nicely with the
"vgname" option proposed in Grégory Oestreicher's patch in #462396.
(No test was done with both patches applied, though.)

Cheers,
-- 
Jérémy Bobbio                        .''`. 
lunar@debian.org                    : :Ⓐ  :  # apt-get install anarchism
                                    `. `'` 
                                      `-   
commit 8f27d9d47f34903e8ba5fa613f6c4c525194e22c
Author: Jérémy Bobbio <lunar@debian.org>
Date:   Tue Jun 3 11:09:23 2008 +0000

    Add support for preseeding logical volume names directly
    
    partman-auto-lvm now support the direct preseeding of a logical volume
    names (instead of computing one) using "lvname{ <name> }" in the recipe.

diff --git a/packages/partman/partman-auto-lvm/debian/changelog b/packages/partman/partman-auto-lvm/debian/changelog
index 7200752..72a8a76 100644
--- a/packages/partman/partman-auto-lvm/debian/changelog
+++ b/packages/partman/partman-auto-lvm/debian/changelog
@@ -1,3 +1,11 @@
+partman-auto-lvm (28) UNRELEASED; urgency=low
+
+  [ Jérémy Bobbio ]
+  * Add support for preseeding logical volume names directly using
+    lvname{ <name> } in the recipe.
+
+ -- Jérémy Bobbio <lunar@debian.org>  Tue, 03 Jun 2008 11:06:17 +0000
+
 partman-auto-lvm (27) unstable; urgency=low
 
   [ Updated translations ]
diff --git a/packages/partman/partman-auto-lvm/perform_recipe_by_lvm b/packages/partman/partman-auto-lvm/perform_recipe_by_lvm
index c530104..c466b52 100755
--- a/packages/partman/partman-auto-lvm/perform_recipe_by_lvm
+++ b/packages/partman/partman-auto-lvm/perform_recipe_by_lvm
@@ -48,7 +48,11 @@ db_progress STEP 1
 
 name_number=1
 foreach_partition '
-	if echo $* | grep -q "mountpoint{"; then
+	if echo $* | grep -q "lvname{"; then
+		lvname=$(echo $* | sed \
+			-e "s/.*lvname{  *\([^ }]*\) *}.*/\1/g"
+		)
+	elif echo $* | grep -q "mountpoint{"; then
 		lvname=$(echo $* | sed \
 			-e "s/.*mountpoint{  *\([^ }]*\) *}.*/\1/g" \
 			-e "s!//!/!g" \

Attachment: signature.asc
Description: Digital signature


Reply to: