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

Issue 8836001: Add syzygy files to archives and symbol processing. (Closed)

Created:
9 years ago by Michael Moss
Modified:
8 years, 8 months ago
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

Add syzygy files to archives and symbol processing. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113482

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -0 lines) Patch
M chrome/tools/build/win/FILES.cfg View 2 chunks +22 lines, -0 lines 1 comment Download
M chrome/tools/build/win/SYMBOLS View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Michael Moss
9 years ago (2011-12-07 03:15:01 UTC) #1
Sigurður Ásgeirsson
lgtm, thanks! http://codereview.chromium.org/8836001/diff/1/chrome/tools/build/win/FILES.cfg File chrome/tools/build/win/FILES.cfg (right): http://codereview.chromium.org/8836001/diff/1/chrome/tools/build/win/FILES.cfg#newcode16 chrome/tools/build/win/FILES.cfg:16: # archive: The name of the archive ...
9 years ago (2011-12-07 14:30:35 UTC) #2
nsylvain
Hello. This change adds ~100MB to the archive of the chromium builds. This used to ...
8 years, 8 months ago (2012-04-06 01:21:12 UTC) #3
Sigurður Ásgeirsson
I don't think we want to archive these files except for official builds. The chromium ...
8 years, 8 months ago (2012-04-06 02:57:59 UTC) #4
Michael Moss
Unlike FILES.cfg, SYMBOLS doesn't distinguish between official and non. Maybe this is reason to finally ...
8 years, 8 months ago (2012-04-06 03:33:20 UTC) #5
nsylvain
8 years, 8 months ago (2012-04-06 05:09:55 UTC) #6
On 2012/04/06 03:33:20, Michael Moss wrote:
> Unlike FILES.cfg, SYMBOLS doesn't distinguish between official and non.
> Maybe this is reason to finally bite the bullet and transition all these
> file lists to the FILES.cfg format. I can start working on it if that's
> what we want.

That would certainly be great to be able to not archive those files for
chromium, but I don't know how much work it is, so I'm not sure about the
tradeoff.  I'm open to anything, and will continue looking at the reasons why gs
upload is so slow.

thanks
> On Apr 5, 2012 7:57 PM, "Sigurður Ásgeirsson" <mailto:siggi@chromium.org>
wrote:
> 
> > I don't think we want to archive these files except for official builds.
> > The chromium files are randomized, and serve only as a smoke test.
> >
> > On Thu, Apr 5, 2012 at 21:21, <mailto:nsylvain@google.com> wrote:
> >
> >> Hello.
> >>
> >> This change adds ~100MB to the archive of the chromium builds.  This used
> >> to not
> >> be a big deal because it was fast, but now it takes close to 25 minutes to
> >> upload our 500MB archive to google storage, and this is slowing down the
> >> cycle
> >> of the "Win" bot on the main waterfall considerably.
> >>
> >> Is this really needed for chromium builds?
> >>
> >> thanks!
> >>
> >>
>
http://codereview.chromium.**org/8836001/%3Chttp://codereview.chromium.org/88...>
> >>
> >
> >

Powered by Google App Engine
This is Rietveld 408576698