2012-06-11 21:45:46

by Ozan Çağlayan

[permalink] [raw]
Subject: [PATCH] bin/get-compat-kernels: Do not depend on Ubuntu

Use Ubuntu PPA vanilla kernels on other distributions too.
The .deb files are extracted using 'ar' and 'tar'.

Also check if the target directory exists before downloading
the deb files.

Signed-off-by: Ozan Çağlayan <[email protected]>
---
bin/get-compat-kernels | 67 +++++++++++++++++++++++++++++++-----------------
1 file changed, 43 insertions(+), 24 deletions(-)

diff --git a/bin/get-compat-kernels b/bin/get-compat-kernels
index e6c586d..6ebaa0c 100755
--- a/bin/get-compat-kernels
+++ b/bin/get-compat-kernels
@@ -10,6 +10,9 @@
# to test compile the Linux kernel compatibility module. You can
# then use ckmake to cross compile against all supported kernels.

+KERNELS=""
+KPATH="http://kernel.ubuntu.com/~kernel-ppa/mainline/"
+
function get_ubuntu_kernels() {

ARCH=$(uname -m)
@@ -28,10 +31,6 @@ function get_ubuntu_kernels() {
;;
esac

- KERNELS=""
-
- KPATH="http://kernel.ubuntu.com/~kernel-ppa/mainline/"
-
KERNELS="$KERNELS ${KPATH}/v2.6.24/linux-headers-2.6.24-020624_2.6.24-020624_all.deb"
KERNELS="$KERNELS ${KPATH}/v2.6.24/linux-headers-2.6.24-020624-generic_2.6.24-020624_${TARGET}.deb"
KERNELS="$KERNELS ${KPATH}/v2.6.24/linux-image-2.6.24-020624-generic_2.6.24-020624_${TARGET}.deb"
@@ -125,19 +124,49 @@ function get_ubuntu_kernels() {
continue
fi

- if [[ ! -f $FILE ]]; then
+ # Do not download if installed or deb exists
+ if [[ ! -d /usr/src/$PKG && ! -f $FILE ]]; then
wget -c $i
fi
done

- # Let dpkg figure out dependency magic.
- #
- # XXX: I tried adding some magic to not install a package if
- # if its already presently installed but then had to deal
- # with the dependency mess. I welcome someone else to
- # figure this out. Running this can come in handy once
- # a new public kernel gets released.
- sudo dpkg -i *.deb
+ LSB_RED_ID=$(/usr/bin/lsb_release -i -s)
+ case $LSB_RED_ID in
+ "Ubuntu")
+ # Install with dpkg
+ sudo dpkg -i *.deb
+ ;;
+ *)
+ # For every other distribution around
+
+ # Create a temporary directory first
+ TEMP_DIR=`mktemp -d`
+
+ for deb in $(ls linux-*.deb); do
+ DIR_NAME=$(echo $deb | awk -F"_" '{print $1}')
+ if [[ ! -d /usr/src/$DIR_NAME ]]; then
+ echo "Extracting $deb..."
+ ar p $deb data.tar.gz | sudo tar xz --exclude=usr/share -C $TEMP_DIR
+ fi
+ done
+
+ # Move the extracted folders into the system
+ if [[ -d $TEMP_DIR/lib/modules ]]; then
+ mv $TEMP_DIR/lib/modules/* /lib/modules
+ fi
+ if [[ -d $TEMP_DIR/usr/src ]]; then
+ # Because of a bug in make < 3.82, mixed implicit and normal
+ # rules do not cause harm. Since the bug is fixed in the new make
+ # we have to adjust older kernel's Makefiles to fix the bug.
+ sed -i 's#^/ %/:#%/:#' $TEMP_DIR/usr/src/linux-headers-2.6.2[45678]-0*/Makefile &>/dev/null
+
+ mv $TEMP_DIR/usr/src/* /usr/src
+ fi
+
+ # Remove the temporary directory
+ rm -rf $TEMP_DIR
+ ;;
+ esac
}

function usage() {
@@ -162,14 +191,4 @@ if [[ $# -eq 1 && $1 = "-i" ]]; then
INSTALL_IMAGES="y"
fi

-
-LSB_RED_ID=$(/usr/bin/lsb_release -i -s)
-case $LSB_RED_ID in
-"Ubuntu")
- get_ubuntu_kernels
- ;;
-*)
- echo -e "Unsupported distribution"
- exit
- ;;
-esac
+get_ubuntu_kernels
--
1.7.10.2



2012-06-11 23:10:15

by Luis R. Rodriguez

[permalink] [raw]
Subject: Re: [PATCH] bin/get-compat-kernels: Do not depend on Ubuntu

On Mon, Jun 11, 2012 at 2:45 PM, Ozan Çağlayan <[email protected]> wrote:
> Use Ubuntu PPA vanilla kernels on other distributions too.
> The .deb files are extracted using 'ar' and 'tar'.
>
> Also check if the target directory exists before downloading
> the deb files.

Nice! Some notes below but these things can be addressed as
evolutionary changes. Since we already use sudo the part below is fine
too, but if we can later make ckmake be non-dependent on root that'd
be neat. But merged as-is for now, thanks!

Luis