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

Issue 27207: net links v8, but didn't have a dep on it, so clean builds failed. (Closed)

Created:
11 years, 10 months ago by TVL
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

net links v8, but didn't have a dep on it, so clean builds failed. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=10461

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -0 lines) Patch
M net/net.xcodeproj/project.pbxproj View 3 chunks +26 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
TVL
11 years, 10 months ago (2009-02-26 14:18:24 UTC) #1
Amanda Walker
lgtm
11 years, 10 months ago (2009-02-26 14:26:59 UTC) #2
TVL
Actually, should the link only be needed when someone links net? (they would also have ...
11 years, 10 months ago (2009-02-26 14:29:20 UTC) #3
TVL
11 years, 10 months ago (2009-02-26 14:30:41 UTC) #4
On 2009/02/26 14:29:20, TVL wrote:
> Actually, should the link only be needed when someone links net?  (they would
> also have to link v8, but libnet shouldn't have to link v8 in a static lib
> world, the unittest might need it)

landing w/ awalkers LG to get the bots happy, mark - my question still stand for
folling cleanup.

Powered by Google App Engine
This is Rietveld 408576698