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

Issue 6093002: Add experimental support for Portage 2.1.9.25. (Closed)

Created:
10 years ago by davidjames
Modified:
9 years, 4 months ago
CC:
chromium-os-reviews_chromium.org, msb+crosoverlay_chromium.org, adlr+crosoverlay_chromium.org, anush
Visibility:
Public.

Description

Add experimental support for Portage 2.1.9.25. Most of our patches for Portage 2.1.7 have been integrated into Portage 2.1.9, so we can substantially reduce our patch count. There's still two patches. There are three differences between this ebuild and the upstream ebuild: 1. All platforms are masked, so that this ebuild isn't used yet. 2. epatch "${FILESDIR}/${P}-fastbuild".patch (needed for parallel_emerge) 3. epatch "${FILESDIR}/${P}-crossdev".patch (needed for cross-compiling packages) See http://codereview.chromium.org/6083008/ for original patch adding unmodified ebuild (except for the masking). See http://codereview.chromium.org/6094001/ for patch updating parallel_emerge to work with new version of Portage. BUG=chromium-os:10574 TEST=./build_packages --board=x86-generic --oldchromebinary Change-Id: I767d8d1ca1630f4c6fa710e5c49cf0ac3f3b04d9

Patch Set 1 #

Patch Set 2 : Re-upload #

Unified diffs Side-by-side diffs Delta from patch set Stats (+156 lines, -0 lines) Patch
A sys-apps/portage/files/portage-2.1.9.25-crossdev.patch View 1 chunk +128 lines, -0 lines 0 comments Download
A sys-apps/portage/files/portage-2.1.9.25-fastbuild.patch View 1 chunk +26 lines, -0 lines 0 comments Download
A + sys-apps/portage/portage-2.1.9.25-r1.ebuild View 1 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
davidjames
10 years ago (2010-12-23 20:50:42 UTC) #1
Mandeep Singh Baines
10 years ago (2010-12-23 21:17:18 UTC) #2
LGTM but please commit a vanilla version of the ebuild first.

Powered by Google App Engine
This is Rietveld 408576698