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

Issue 5854001: Add package.keywords logic to build version of chrome we specify. (Closed)

Created:
10 years ago by sosa
Modified:
9 years, 6 months ago
Reviewers:
anush
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

Add package.keywords logic to build version of chrome we specify. Change-Id: I32c17e7353b420d7a895acffab4579c9b861095c BUG=chromium-os:8693 TEST=Ran it with chrome_rev=tot, latest_release and stable_release Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=cd2196d

Patch Set 1 #

Patch Set 2 : Nit #

Patch Set 3 : Fix todo #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -4 lines) Patch
M bin/cbuildbot.py View 1 2 5 chunks +10 lines, -4 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
sosa
10 years ago (2010-12-15 01:38:58 UTC) #1
anush
10 years ago (2010-12-15 01:41:26 UTC) #2
LGTM

On Tue, Dec 14, 2010 at 5:38 PM,  <sosa@chromium.org> wrote:
> Reviewers: anush,
>
> Description:
> Add package.keywords logic to build version of chrome we specify.
>
> Change-Id: I32c17e7353b420d7a895acffab4579c9b861095c
>
> BUG=chromium-os:8693
> TEST=Ran it with chrome_rev=tot, latest_release and stable_release
>
> Please review this at http://codereview.chromium.org/5854001/
>
> SVN Base: http://git.chromium.org/git/crosutils.git@master
>
> Affected files:
>  M bin/cbuildbot.py
>
>
> Index: bin/cbuildbot.py
> diff --git a/bin/cbuildbot.py b/bin/cbuildbot.py
> index
>
076487fb2007aadbd3b57a6c80aea8ceb3cff980..fb408eb58046ca24cabfbc2dd0b5f035c3c8744e
> 100755
> --- a/bin/cbuildbot.py
> +++ b/bin/cbuildbot.py
> @@ -27,6 +27,7 @@ ARCHIVE_BASE = '/var/www/archive'
>  ARCHIVE_COUNT = 10
>  PUBLIC_OVERLAY = '%(buildroot)s/src/third_party/chromiumos-overlay'
>  PRIVATE_OVERLAY = '%(buildroot)s/src/private-overlays/chromeos-overlay'
> +CHROME_KEYWORDS_FILE =
> ('/build/%(board)s/etc/portage/package.keywords/chrome')
>
>  # Currently, both the full buildbot and the preflight buildbot store their
>  # data in a variable named PORTAGE_BINHOST, but they're in different files.
> @@ -217,7 +218,7 @@ def _UprevFromRevisionList(buildroot, tracking_branch,
> revision_list, board,
>              cwd=cwd, enter_chroot=True)
>
>
> -def _MarkChromeAsStable(buildroot, tracking_branch, chrome_rev):
> +def _MarkChromeAsStable(buildroot, tracking_branch, chrome_rev, board):
>   """Returns the portage atom for the revved chrome ebuild - see man
> emerge."""
>   cwd = os.path.join(buildroot, 'src', 'scripts')
>   portage_atom_string = RunCommand(['bin/cros_mark_chrome_as_stable',
> @@ -228,7 +229,12 @@ def _MarkChromeAsStable(buildroot, tracking_branch,
> chrome_rev):
>     Info('Found nothing to rev.')
>     return None
>   else:
> -    return portage_atom_string.split('=')[1]
> +    chrome_atom = portage_atom_string.split('=')[1]
> +    # TODO(sosa): Workaround to build unstable chrome ebuild we uprevved.
> +    RunCommand(['sudo', 'tee', CHROME_KEYWORDS_FILE % {'board': board}],
> +               input='=%s\n' % chrome_atom, enter_chroot=True,
> +               cwd=cwd)
> +    return chrome_atom
>
>
>  def _UprevAllPackages(buildroot, tracking_branch, board, overlays):
> @@ -416,7 +422,7 @@ def _Build(buildroot, emptytree):
>  def _BuildChrome(buildroot, board, chrome_atom_to_build):
>   """Wrapper for emerge call to build Chrome."""
>   cwd = os.path.join(buildroot, 'src', 'scripts')
> -  RunCommand(['ACCEPT_KEYWORDS="* ~*"', 'emerge-%s' % board,
> +  RunCommand(['emerge-%s' % board,
>               '=%s' % chrome_atom_to_build],
>              cwd=cwd, enter_chroot=True)
>
> @@ -734,7 +740,7 @@ def main():
>     # Perform uprev.  If chrome_uprev is set, rev Chrome ebuilds.
>     if options.chrome_rev:
>       chrome_atom_to_build = _MarkChromeAsStable(buildroot, tracking_branch,
> -                                                 options.chrome_rev)
> +                                                 options.chrome_rev, board)
>       # If we found nothing to rev, we're done here.
>       if not chrome_atom_to_build:
>         return
>
>
>

Powered by Google App Engine
This is Rietveld 408576698