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

Issue 10413004: Try to fix failing tests after 1386. (Closed)

Created:
8 years, 7 months ago by Nico
Modified:
8 years, 7 months ago
Reviewers:
scottmg
CC:
gyp-developer_googlegroups.com
Visibility:
Public.

Description

Try to fix failing tests after 1386. TBR=scottmg Committed: https://code.google.com/p/gyp/source/detail?r=1387

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M test/defines/defines.c View 1 chunk +2 lines, -0 lines 0 comments Download
M test/defines/gyptest-defines-env-regyp.py View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Nico
The file I changed is used by several tests, not all of which define HASH_VALUE.
8 years, 7 months ago (2012-05-18 13:46:56 UTC) #1
scottmg
8 years, 7 months ago (2012-05-18 22:56:25 UTC) #2
On 2012/05/18 13:46:56, Nico wrote:
> The file I changed is used by several tests, not all of which define
HASH_VALUE.

looks like this is failing on 2010 only? 
http://build.chromium.org/p/tryserver.nacl/builders/gyp-win64/builds/335/step...

managing to overcome your octalizing and trying *extra hard* to convert # to =.

separate test and !msvs seems reasonable to me.

Powered by Google App Engine
This is Rietveld 408576698