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

Issue 1153703002: Update shared_library test after c0cf1f22eb (Closed)

Created:
5 years, 7 months ago by scottmg
Modified:
5 years, 7 months ago
Reviewers:
Nico
CC:
gyp-developer_googlegroups.com
Base URL:
https://chromium.googlesource.com/external/gyp.git@master
Target Ref:
refs/heads/master
Project:
gyp
Visibility:
Public.

Description

Update shared_library test after c0cf1f22eb Forgot to update test after removing shared_library case from target types that aren't allowed to have duplicate basenames. R=thakis@chromium.org BUG=gyp:384 Committed: https://chromium.googlesource.com/external/gyp/+/4dd5d3c614fb851b0ccda15bf66abec87b38ceb9

Patch Set 1 #

Patch Set 2 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -36 lines) Patch
A + test/same-source-file-name/gyptest-pass-shared.py View 1 1 chunk +3 lines, -5 lines 0 comments Download
D test/same-source-file-name/gyptest-shared.py View 1 chunk +0 lines, -31 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
scottmg
5 years, 7 months ago (2015-05-22 00:27:27 UTC) #1
Nico
lgtm
5 years, 7 months ago (2015-05-22 01:21:15 UTC) #2
scottmg
5 years, 7 months ago (2015-05-22 01:44:22 UTC) #3
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
4dd5d3c614fb851b0ccda15bf66abec87b38ceb9.

Powered by Google App Engine
This is Rietveld 408576698