Merge lp://qastaging/~deeptik/linaro-ci/enable_custom_gcc_usage into lp://qastaging/linaro-ci

Proposed by Deepti B. Kalakeri
Status: Merged
Merged at revision: 69
Proposed branch: lp://qastaging/~deeptik/linaro-ci/enable_custom_gcc_usage
Merge into: lp://qastaging/linaro-ci
Diff against target: 73 lines (+23/-3) (has conflicts)
2 files modified
jenkins_common_lib (+10/-3)
jenkins_kernel_build_inst (+13/-0)
Text conflict in jenkins_common_lib
To merge this branch: bzr merge lp://qastaging/~deeptik/linaro-ci/enable_custom_gcc_usage
Reviewer Review Type Date Requested Status
Fathi Boudra Approve
Linaro Infrastructure Pending
Review via email: mp+107337@code.qastaging.launchpad.net

Description of the change

Enable using customized toolchain to be used to build the kernel

To post a comment you must log in.
Revision history for this message
Fathi Boudra (fboudra) wrote :

On 25 May 2012 10:39, Deepti B. Kalakeri wrote:
> === modified file 'jenkins_kernel_build_inst'
> --- jenkins_kernel_build_inst   2012-05-15 11:32:29 +0000
> +++ jenkins_kernel_build_inst   2012-05-25 07:38:17 +0000
> @@ -30,6 +30,19 @@
>  if test -z "$lava_test_plan"; then
>   lava_test_plan="ltp, pwrmgmt"
>  fi
> +
> +if test -z "$toolchain_url"; then
> +  gcc_compiler=`which arm-linux-gnueabi-gcc`
> +  gcc_cc_version=`dpkg -s gcc-arm-linux-gnueabi | grep Version | cut -d ' ' -f2`
> +else
> +  wget -cq $toolchain_url
> +  toolchain_tar_filename=`basename $toolchain_url`
> +  mkdir "toolchain"
> +  tar -C "toolchain"  --strip-components 1 -xf $toolchain_tar_filename
> +  gcc_compiler=`find $PWD/toolchain/bin/arm-linux-gnueabi-gcc |head -n1`
> +  gcc_cc_version=`$gcc_compiler --version | head -n 1 | awk -F ')' '{print  $2")"}' | sed 's/[\(\)]/ /g'`

1. version parsing is generic. gcc_cc_version could be common to both
the packaged toolchain and the binary toolchain.

2. the kernel provides scripts/gcc-version.sh, you can use it directly.
e.g. /path/to/kernel/sources/scripts/gcc-version.sh -p $gcc_compiler

Revision history for this message
Fathi Boudra (fboudra) :
review: Needs Fixing
69. By Deepti B. Kalakeri

Using the kernel gcc-version.sh script to find the CC version

Revision history for this message
Fathi Boudra (fboudra) wrote :

Approved.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches