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

Issue 124048: Add Makeself sources to o3d/third_party. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 9 months ago by noah.maxwell
Modified:
15 years, 8 months ago
CC:
o3d-review_googlegroups.com
Base URL:
http://o3d.googlecode.com/svn/trunk/googleclient/third_party/
Visibility:
Public.

Description

Add Makeself sources to o3d/third_party.

Patch Set 1 #

Total comments: 4

Patch Set 2 : '' #

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1913 lines, -0 lines) Patch
A makeself_2_1_5/COPYING View 1 chunk +341 lines, -0 lines 0 comments Download
A makeself_2_1_5/LICENSE View 1 chunk +341 lines, -0 lines 0 comments Download
A makeself_2_1_5/README View 1 2 1 chunk +307 lines, -0 lines 0 comments Download
A makeself_2_1_5/README.google View 1 chunk +18 lines, -0 lines 0 comments Download
A makeself_2_1_5/TODO View 1 2 1 chunk +6 lines, -0 lines 0 comments Download
A makeself_2_1_5/makeself.1 View 1 2 1 chunk +76 lines, -0 lines 0 comments Download
A makeself_2_1_5/makeself.lsm View 1 2 1 chunk +16 lines, -0 lines 0 comments Download
A makeself_2_1_5/makeself.sh View 1 2 1 chunk +407 lines, -0 lines 0 comments Download
A makeself_2_1_5/makeself-header.sh View 1 chunk +401 lines, -0 lines 0 comments Download

Messages

Total messages: 6
noah.maxwell
15 years, 9 months ago (2009-09-25 22:08:51 UTC) #1
Antoine Labour
LGTM, but please remove the 2 Google internal files, they aren't needed in the public ...
15 years, 9 months ago (2009-09-25 22:31:28 UTC) #2
tschmelcher
LGTM Modulo what Antoine said.
15 years, 9 months ago (2009-09-25 22:44:23 UTC) #3
noah.maxwell
2 files removed. PTAL. http://codereview.appspot.com/124048/diff/1/2 File makeself_2_1_5/BUILD (right): http://codereview.appspot.com/124048/diff/1/2#newcode1 Line 1: # Copyright 2009 Google ...
15 years, 8 months ago (2009-09-28 22:48:30 UTC) #4
Antoine Labour
LGTM
15 years, 8 months ago (2009-09-28 22:49:10 UTC) #5
noah.maxwell
15 years, 8 months ago (2009-10-14 21:36:51 UTC) #6
FYI, I am still unable to submit this due to this bug:

http://code.google.com/p/gclient/issues/detail?id=13

Still no response on the bug, so I think Im going to create this as a new issue
using my new nmaxwell@chromium account to see if I can submit it that way.
Sign in to reply to this message.

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