Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(6359)

Issue 4842043: Move syzygy binary release to public repo. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 8 months ago by Roger McFarlane
Modified:
12 years, 8 months ago
Reviewers:
chrisha, M-A, jeffbailey, Siggi
CC:
sawbuck-changes_googlegroups.com
Base URL:
http://sawbuck.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Move syzygy binary release to public repo. Committed: http://code.google.com/p/sawbuck/source/browse/#svn/trunk392

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+76 lines, -0 lines) Patch
A syzygy/binaries/Benchmark_Chrome-0.1_r380-py2.6.egg View Binary file 0 comments Download
A syzygy/binaries/ETW-0.6.5.0-py2.6.egg View Binary file 0 comments Download
A syzygy/binaries/ETW_Db-0.1_r365-py2.6.egg View Binary file 0 comments Download
A syzygy/binaries/benchmark.bat View 1 chunk +38 lines, -0 lines 0 comments Download
A syzygy/binaries/optimize.bat View 1 chunk +38 lines, -0 lines 0 comments Download
A syzygy/binaries/setuptools-0.6c11-py2.6.egg View Binary file 0 comments Download

Messages

Total messages: 4
Roger McFarlane
PTAL
12 years, 8 months ago (2011-08-02 17:11:55 UTC) #1
M-A
I feel like the binaries shouldn't be intermixed with the source tree, e.g. it should ...
12 years, 8 months ago (2011-08-02 17:14:43 UTC) #2
Siggi
lgtm
12 years, 8 months ago (2011-08-02 17:30:57 UTC) #3
Siggi
12 years, 8 months ago (2011-08-02 17:32:37 UTC) #4
I think there'd be a strong argument for doing this if we had a per-project
DEPS file, as then we could avoid syncing the binary release dir when
working on sources.
As-is, everyone syncs /trunk/, so there's not much benefit. I'd like to move
Syzygy to a different repo eventually, at which point we should revisit
this.

On Tue, Aug 2, 2011 at 1:14 PM, <maruel@chromium.org> wrote:

> I feel like the binaries shouldn't be intermixed with the source tree,
> e.g. it should be in a directory directly under /trunk/.
>
> I don't have a strong opinion.
>
>
>
http://codereview.appspot.com/**4842043/<http://codereview.appspot.com/4842043/>
>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b