Fix several small bugs for adt installer
cover Fix several small bugs for adt installer
From: Liping Ke <[email protected]>
Two bugs are found
1) image download file path is not correct. So even if file is downloaded already, it can't be detected.
2) sdk image now is renamed as sato-sdk, we need to change the name accordingly.
Pull URL: git://git.pokylinux.org/poky-contrib.git
Branch: lke/adt-newfix
Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=lke/adt-newfix
Thanks,
Liping Ke <[email protected]>
---
Liping Ke (1):
ADT: Fix several bugs for adt installer
.../installer/adt-installer/adt_installer | 9 +++++---- .../installer/adt-installer/adt_installer.conf | 8 ++++---- .../installer/adt-installer/scripts/data_define | 2 +- 3 files changed, 10 insertions(+), 9 deletions(-)
PATCH 1/1 ADT: Fix several bugs for adt installer
From: Liping Ke <[email protected]>
Two bugs are found
1) image download file path is not correct. So even if file is downloaded already, it can't be detected.
2) several images now are renamed, such as sato-dev, sato-sdk, we need to change the name accordingly.
--- .../installer/adt-installer/adt_installer | 9 +++++---- .../installer/adt-installer/adt_installer.conf | 8 ++++---- .../installer/adt-installer/scripts/data_define | 2 +- 3 files changed, 10 insertions(+), 9 deletions(-)
meta/recipes-devtools/installer/adt-installer/adt_installer +5 -4
@@ -216,17 +216,18 @@ done download_file() { -if [ -f "$LOCAL_DOWNLOAD/$1" ]; then - confirm_download $1 +local filename=`echo ${1##*/}` +if [ -f "$LOCAL_DOWNLOAD/$filename" ]; then + confirm_download $filename result="$?" if [ ! "$result" == "0" ]; then return else echo "Removing old file [$1]" - rm -rf "$LOCAL_DOWNLOAD/$1" + rm -rf "$LOCAL_DOWNLOAD/$filename" fi fi -echo_info "Downloading file: $1..." +echo_info "Downloading file: $filename..." wget "$YOCTOADT_IPKG_REPO/$1" -P $LOCAL_DOWNLOAD --progress=bar:force 2>&1 | tee -a "$YOCTOADT_INSTALL_LOG_FILE" }
meta/recipes-devtools/installer/adt-installer/adt_installer.conf +4 -4
@@ -39,14 +39,14 @@ YOCTOADT_NFS_UTIL="Y" #YOCTOADT_METADATA="Y" #The followings are for setting up specific target architecture -#YOCTOADT_ROOTFS_$arch is for specifying what root filesystem image files you want to download from the repository. The valid values to replace $arch are: arm, x86, x86_64, powerpc, mips. The valid image files are: minimal, sato, sdk and lsb. If you want to download multiple images, the entries are space separated -YOCTOADT_ROOTFS_arm="minimal sato" +#YOCTOADT_ROOTFS_$arch is for specifying what root filesystem image files you want to download from the repository. The valid values to replace $arch are: arm, x86, x86_64, powerpc, mips. The valid image files are: minimal, minimal-dev, sato, sato-dev, sato-sdk,lsb, lsb-dev, lsb-sdk. If you want to download multiple images, the entries are space separated +YOCTOADT_ROOTFS_arm="minimal sato-sdk" #Specify which root filesystem file to use to extract as target sysroot. Please ensure the entry is in the list of downloaded root filesystem files that specified above in YOCTOADT_ROOTFS_$arch YOCTOADT_TARGET_SYSROOT_IMAGE_arm="minimal" #The location where the target sysroot will be setup YOCTOADT_TARGET_SYSROOT_LOC_arm="$HOME/test-yocto/arm" #Here's another example for setting up target arch of x86, by uncommenting it will trigger the installer to download and setup 2 sysroot environment for 2 target arches: arm and x86. If you want to add more target arch support, you can append more entries by following these samples -#YOCTOADT_ROOTFS_x86="sdk" -#YOCTOADT_TARGET_SYSROOT_IMAGE_x86="sdk" +#YOCTOADT_ROOTFS_x86="sato-sdk" +#YOCTOADT_TARGET_SYSROOT_IMAGE_x86="sato-sdk" #YOCTOADT_TARGET_SYSROOT_LOC_x86="$HOME/test-yocto/x86"
meta/recipes-devtools/installer/adt-installer/scripts/data_define +1 -1
@@ -31,7 +31,7 @@ OPKG_LIBDIR="/var/lib" # List all supported root fs types and target types, # it will be used in user configuration validity checking -YOCTOADT_SUPPORTED_ROOTFS="minimal sato sdk lsb" +YOCTOADT_SUPPORTED_ROOTFS="minimal minimal-dev sato sato-dev sato-sdk lsb lsb-dev lsb-sdk" YOCTOADT_SUPPORTED_TARGETS="x86 x86_64 arm powerpc mips" # Different host (32 bit or 64 bit) will have different opkg # configuration files
Signature · 1 line
1.7.0.4
Re: [PATCH 1/1] ADT: Fix several bugs for adt installer
Quoted 9 lines · Liping Ke
From: Liping Ke <[email protected]>
Two bugs are found
1) image download file path is not correct. So even if file is downloaded already, it can't be detected.
2) several images now are renamed, such as sato-dev, sato-sdk, we need to change the name accordingly.Signed-off-by Liping Ke <[email protected]>
Acked by: Jessica Zhang <[email protected]>
Quoted 91 lines
--- .../installer/adt-installer/adt_installer | 9 +++++---- .../installer/adt-installer/adt_installer.conf | 8 ++++---- .../installer/adt-installer/scripts/data_define | 2 +- 3 files changed, 10 insertions(+), 9 deletions(-)diff --git a/meta/recipes-devtools/installer/adt-installer/adt_installer b/meta/recipes-devtools/installer/adt-installer/adt_installer index 1f481de..cca576f 100755
meta/recipes-devtools/installer/adt-installer/adt_installer +5
-4@@ -216,17 +216,18 @@ done download_file() { -if [ -f "$LOCAL_DOWNLOAD/$1" ]; then - confirm_download $1 +local filename=`echo ${1##*/}` +if [ -f "$LOCAL_DOWNLOAD/$filename" ]; then + confirm_download $filename result="$?" if [ ! "$result" == "0" ]; then return else echo "Removing old file [$1]" - rm -rf "$LOCAL_DOWNLOAD/$1" + rm -rf "$LOCAL_DOWNLOAD/$filename" fi fi -echo_info "Downloading file: $1..." +echo_info "Downloading file: $filename..." wget "$YOCTOADT_IPKG_REPO/$1" -P $LOCAL_DOWNLOAD --progress=bar:force 2>&1 | tee -a "$YOCTOADT_INSTALL_LOG_FILE" }diff --git
a/meta/recipes-devtools/installer/adt-installer/adt_installer.confb/meta/recipes-devtools/installer/adt-installer/adt_installer.confindex db602f6..d2af70d 100644meta/recipes-devtools/installer/adt-installer/adt_installer.conf +0
-1@@ -39,14 +39,14 @@ YOCTOADT_NFS_UTIL="Y" #YOCTOADT_METADATA="Y" #The followings are for setting up specific target architecture -#YOCTOADT_ROOTFS_$arch is for specifying what root filesystem imagefiles you want to download from the repository. The valid values to replace $arch are: arm, x86,
x86_64, powerpc, mips. The valid image files are: minimal, sato, sdk and lsb. If you want to download multiple images, the entries are space separatedmeta/recipes-devtools/installer/adt-installer/adt_installer.conf +1
-1-YOCTOADT_ROOTFS_arm="minimal sato" +#YOCTOADT_ROOTFS_$arch is for specifying what root filesystem imagefiles you want to download from the repository. The valid values to
replace $arch are: arm, x86,x86_64, powerpc, mips. The valid image files are: minimal, minimal-dev, sato, sato-dev, sato-sdk,lsb, lsb-dev, lsb-sdk. If you want to download multiple images, the entries are space separated +YOCTOADT_ROOTFS_arm="minimal sato-sdk"
#Specify which root filesystem file to use to extract as target sysroot. Please ensure the entry is in the list of downloaded root filesystem files that specified above in YOCTOADT_ROOTFS_$arch YOCTOADT_TARGET_SYSROOT_IMAGE_arm="minimal" #The location where the target sysroot will be setup YOCTOADT_TARGET_SYSROOT_LOC_arm="$HOME/test-yocto/arm"#Here's another example for setting up target arch of x86, by
uncommenting it will trigger the installer to download and setup 2 sysroot environment for 2 target arches: arm and x86. If you want to add more target arch support, you can append more entries by following these samplesmeta/recipes-devtools/installer/adt-installer/adt_installer.conf +2
-2-#YOCTOADT_ROOTFS_x86="sdk" -#YOCTOADT_TARGET_SYSROOT_IMAGE_x86="sdk" +#YOCTOADT_ROOTFS_x86="sato-sdk" +#YOCTOADT_TARGET_SYSROOT_IMAGE_x86="sato-sdk" #YOCTOADT_TARGET_SYSROOT_LOC_x86="$HOME/test-yocto/x86"diff --git a/meta/recipes-devtools/installer/adt-installer/scripts/data_define b/meta/recipes-devtools/installer/adt-installer/scripts/data_define index 2895c4f..77f8b38 100644
---
a/meta/recipes-devtools/installer/adt-installer/scripts/data_define
+++
b/meta/recipes-devtools/installer/adt-installer/scripts/data_definemeta/recipes-devtools/installer/adt-installer/adt_installer.conf +1
-1@@ -31,7 +31,7 @@ OPKG_LIBDIR="/var/lib" # List all supported root fs types and target types, # it will be used in user configuration validity checking -YOCTOADT_SUPPORTED_ROOTFS="minimal sato sdk lsb" +YOCTOADT_SUPPORTED_ROOTFS="minimal minimal-dev sato sato-dev sato-sdk lsb lsb-dev lsb-sdk" YOCTOADT_SUPPORTED_TARGETS="x86 x86_64 arm powerpc mips" # Different host (32 bit or 64 bit) will have different opkg # configuration files