Merge lp:~milo/linaro-ci/bug1034288 into lp:linaro-ci

Proposed by Milo Casagrande
Status: Merged
Merged at revision: 98
Proposed branch: lp:~milo/linaro-ci/bug1034288
Merge into: lp:linaro-ci
Diff against target: 48 lines (+15/-1)
3 files modified
node/setup-oneiric-node (+1/-0)
node/setup-precise-node (+1/-0)
node/setup_lib (+13/-1)
To merge this branch: bzr merge lp:~milo/linaro-ci/bug1034288
Reviewer Review Type Date Requested Status
Fathi Boudra Approve
Deepti B. Kalakeri Pending
Paul Sokolovsky Pending
Linaro Infrastructure Pending
Review via email: mp+126676@code.launchpad.net

Description of the change

The MP adds a new function in the setup_lib bash file, in order to install more recent version of Linaro Image Tools.
Updated also the node scripts.

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

Some minor fixes:
1) Add the function call where we need it (Oneiric/Precise node). Skip Natty node for now.
2) Rename install_lit to lit_install for consistency with live_build_install.
3) Call the function between live_build_install and git_config, to keep the packages installation block together.
4) VERSION should contains the debian version i.e. 2012.09.1-0ubuntu1~linaro1
5) use a single wget i.e. wget "${DOWNLOAD_URL}${DEB_FILE} ${DOWNLOAD_URL}${PYTHON_LIT}"

Thanks!

review: Needs Fixing
lp:~milo/linaro-ci/bug1034288 updated
100. By Milo Casagrande

Removed installation of l-i-t from natty node.

101. By Milo Casagrande

Fixed version variable, wget line, and wrong variable use.

102. By Milo Casagrande

Renamed function, moved function calls in the nodes.

Revision history for this message
Milo Casagrande (milo) wrote :

Hi Fathi,

thanks for the review.

> Some minor fixes:
> 1) Add the function call where we need it (Oneiric/Precise node). Skip Natty
> node for now.
> 2) Rename install_lit to lit_install for consistency with live_build_install.
> 3) Call the function between live_build_install and git_config, to keep the
> packages installation block together.
> 4) VERSION should contains the debian version i.e. 2012.09.1-0ubuntu1~linaro1
> 5) use a single wget i.e. wget "${DOWNLOAD_URL}${DEB_FILE}
> ${DOWNLOAD_URL}${PYTHON_LIT}"

this is all done.
Just a note on the wget call: it needs to have two separate "strings" (wget "URL1" "URL2").

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

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'node/setup-oneiric-node'
--- node/setup-oneiric-node 2012-08-07 14:37:21 +0000
+++ node/setup-oneiric-node 2012-09-28 07:06:18 +0000
@@ -28,6 +28,7 @@
28 uboot-mkimage28 uboot-mkimage
2929
30live_build_install30live_build_install
31lit_install
31git_config32git_config
3233
33if test -e /tmp/fifo3; then34if test -e /tmp/fifo3; then
3435
=== modified file 'node/setup-precise-node'
--- node/setup-precise-node 2012-08-16 04:56:33 +0000
+++ node/setup-precise-node 2012-09-28 07:06:18 +0000
@@ -55,6 +55,7 @@
55 zsync55 zsync
5656
57live_build_install57live_build_install
58lit_install
58git_config59git_config
5960
60if test -e /tmp/fifo3; then61if test -e /tmp/fifo3; then
6162
=== modified file 'node/setup_lib'
--- node/setup_lib 2012-06-19 09:50:30 +0000
+++ node/setup_lib 2012-09-28 07:06:18 +0000
@@ -45,7 +45,19 @@
45 git config --system http.proxy http://ci.linaro.org:312845 git config --system http.proxy http://ci.linaro.org:3128
46}46}
4747
4848# Installs the correct linaro-image-tools and python-linaro-image-tools
49# versions.
50# The deb files are taken from the ubuntu-build-production archive:
51# https://launchpad.net/~linaro-infrastructure/+archive/ubuntu-build-production
52lit_install()
53{
54 VERSION='2012.09.1-0ubuntu1~linaro1'
55 DOWNLOAD_URL="https://launchpad.net/~linaro-infrastructure/+archive/ubuntu-build-production/+files/"
56 LIT_DEB="linaro-image-tools_${VERSION}_all.deb"
57 PYTHON_LIT="python-linaro-image-tools_${VERSION}_all.deb"
58 wget "${DOWNLOAD_URL}${LIT_DEB}" "${DOWNLOAD_URL}${PYTHON_LIT}"
59 dpkg -i ${LIT_DEB} ${PYTHON_LIT}
60}
4961
50fifo_cleanup()62fifo_cleanup()
51{63{

Subscribers

People subscribed via source and target branches