Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(74)

Issue 271853003: Add support for building Ubuntu/Trusty sysroot images. (Closed)

Created:
6 years, 7 months ago by Sam Clegg
Modified:
6 years, 7 months ago
Reviewers:
Lei Zhang
CC:
chromium-reviews, grt+watch_chromium.org
Visibility:
Public.

Description

Add support for building Ubuntu/Trusty sysroot images. The main reason for this is to support cross building for ARM on Trusty machines (where the sysroot should match the cross compiler version). Rename the main sysroot creator script and source it in the trust and wheezy scripts that are actually runnable. This should make it easy to create a script for Debian/Jesse when it is released without duplicating code. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=270253

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Total comments: 8

Patch Set 4 : #

Total comments: 8

Patch Set 5 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+616 lines, -1430 lines) Patch
D chrome/installer/linux/sysroot_scripts/packagelist.debian.wheezy.amd64 View 1 chunk +0 lines, -159 lines 0 comments Download
D chrome/installer/linux/sysroot_scripts/packagelist.debian.wheezy.arm View 1 chunk +0 lines, -158 lines 0 comments Download
D chrome/installer/linux/sysroot_scripts/packagelist.debian.wheezy.i386 View 1 chunk +0 lines, -159 lines 0 comments Download
A chrome/installer/linux/sysroot_scripts/packagelist.trusty.arm View 1 1 chunk +161 lines, -0 lines 0 comments Download
A + chrome/installer/linux/sysroot_scripts/packagelist.wheezy.amd64 View 0 chunks +-1 lines, --1 lines 0 comments Download
A + chrome/installer/linux/sysroot_scripts/packagelist.wheezy.arm View 4 0 chunks +-1 lines, --1 lines 0 comments Download
A + chrome/installer/linux/sysroot_scripts/packagelist.wheezy.i386 View 0 chunks +-1 lines, --1 lines 0 comments Download
A + chrome/installer/linux/sysroot_scripts/sysroot-creator.sh View 1 2 3 4 17 chunks +73 lines, -228 lines 0 comments Download
D chrome/installer/linux/sysroot_scripts/sysroot-creator-debian.wheezy.sh View 1 chunk +0 lines, -729 lines 0 comments Download
A chrome/installer/linux/sysroot_scripts/sysroot-creator-test.sh View 1 2 3 4 1 chunk +23 lines, -0 lines 0 comments Download
A chrome/installer/linux/sysroot_scripts/sysroot-creator-trusty.sh View 1 2 3 4 1 chunk +184 lines, -0 lines 0 comments Download
A chrome/installer/linux/sysroot_scripts/sysroot-creator-wheezy.sh View 1 2 3 4 1 chunk +178 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
Sam Clegg
6 years, 7 months ago (2014-05-13 00:21:25 UTC) #1
Lei Zhang
https://codereview.chromium.org/271853003/diff/40001/chrome/installer/linux/sysroot_scripts/sysroot-creator.sh File chrome/installer/linux/sysroot_scripts/sysroot-creator.sh (right): https://codereview.chromium.org/271853003/diff/40001/chrome/installer/linux/sysroot_scripts/sysroot-creator.sh#newcode5 chrome/installer/linux/sysroot_scripts/sysroot-creator.sh:5: #@ This script builds a Debian sysroot impages for ...
6 years, 7 months ago (2014-05-13 02:16:05 UTC) #2
Sam Clegg
Also added a basic test. https://codereview.chromium.org/271853003/diff/40001/chrome/installer/linux/sysroot_scripts/sysroot-creator.sh File chrome/installer/linux/sysroot_scripts/sysroot-creator.sh (right): https://codereview.chromium.org/271853003/diff/40001/chrome/installer/linux/sysroot_scripts/sysroot-creator.sh#newcode5 chrome/installer/linux/sysroot_scripts/sysroot-creator.sh:5: #@ This script builds ...
6 years, 7 months ago (2014-05-13 18:43:49 UTC) #3
Lei Zhang
https://codereview.chromium.org/271853003/diff/50001/chrome/installer/linux/sysroot_scripts/packagelist.wheezy.arm File chrome/installer/linux/sysroot_scripts/packagelist.wheezy.arm (right): https://codereview.chromium.org/271853003/diff/50001/chrome/installer/linux/sysroot_scripts/packagelist.wheezy.arm#newcode1 chrome/installer/linux/sysroot_scripts/packagelist.wheezy.arm:1: main/e/e2fsprogs/comerr-dev_2.1-1.42.5-1.1_armhf.deb 743b38a2078021023cc3e31f0e5983780aa8c90dab09e085a58ca06f0559ff5b Did you mean to add the checksums? ...
6 years, 7 months ago (2014-05-13 19:57:53 UTC) #4
Sam Clegg
https://codereview.chromium.org/271853003/diff/50001/chrome/installer/linux/sysroot_scripts/packagelist.wheezy.arm File chrome/installer/linux/sysroot_scripts/packagelist.wheezy.arm (right): https://codereview.chromium.org/271853003/diff/50001/chrome/installer/linux/sysroot_scripts/packagelist.wheezy.arm#newcode1 chrome/installer/linux/sysroot_scripts/packagelist.wheezy.arm:1: main/e/e2fsprogs/comerr-dev_2.1-1.42.5-1.1_armhf.deb 743b38a2078021023cc3e31f0e5983780aa8c90dab09e085a58ca06f0559ff5b On 2014/05/13 19:57:53, Lei Zhang wrote: > ...
6 years, 7 months ago (2014-05-13 20:13:21 UTC) #5
Lei Zhang
lgtm
6 years, 7 months ago (2014-05-13 20:24:47 UTC) #6
Sam Clegg
The CQ bit was checked by sbc@chromium.org
6 years, 7 months ago (2014-05-13 20:51:34 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/271853003/70001
6 years, 7 months ago (2014-05-13 20:53:02 UTC) #8
commit-bot: I haz the power
6 years, 7 months ago (2014-05-13 23:57:51 UTC) #9
Message was sent while issue was closed.
Change committed as 270253

Powered by Google App Engine
This is Rietveld 408576698