[PATCH] Fix more problems in expandModuleSet, based on a patch from markmc (#480307).

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

 



(1) Remove the extra calls to cat.
(2) Remove an extra paren on the modules.block line.
(3) Use $version instead of $kernel.
---
 scripts/mk-images |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/scripts/mk-images b/scripts/mk-images
index 631434c..951ca8a 100755
--- a/scripts/mk-images
+++ b/scripts/mk-images
@@ -261,15 +261,15 @@ expandModuleSet() {
         if [ $char = '=' ]; then
 	    NAME=$(echo $name | cut -c2-)
 	    if [ "$NAME" = "ata" ]; then
-		SET="$SET $(cat $MBD_DIR/lib/modules/$kernel/modules.block |egrep '(ata|ahci)' |sed -e 's/.ko//')"
+		SET="$SET $(egrep '(ata|ahci)' $MBD_DIR/lib/modules/$version/modules.block |sed -e 's/.ko//')"
 	    elif [ "$NAME" = "scsi" ]; then
-		SET="$SET $(cat $MBD_DIR/lib/modules/$kernel/modules.block) |sed -e 's/.ko//')"
+		SET="$SET $(sed -e 's/.ko//' $MBD_DIR/lib/modules/$version/modules.block)"
 	    elif [ "$NAME" = "net" ]; then
-		SET="$SET $(cat $MBD_DIR/lib/modules/$kernel/modules.networking) |sed -e 's/.ko//')"
+		SET="$SET $(sed -e 's/.ko//' $MBD_DIR/lib/modules/$version/modules.networking)"
 	    else
                 # Ignore if group list does not exist
-                if [ -e $MBD_DIR/lib/modules/$kernel/modules.$NAME ]; then
-		    SET="$SET $(cat $MBD_DIR/lib/modules/$kernel/modules.$NAME |sed -e 's/.ko//')"
+                if [ -e $MBD_DIR/lib/modules/$version/modules.$NAME ]; then
+		    SET="$SET $(sed -e 's/.ko//' $MBD_DIR/lib/modules/$version/modules.$NAME)"
                 fi
 	    fi
 	else
-- 
1.6.0.3

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux