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

Side by Side Diff: WATCHLISTS

Issue 1589633007: [WATCHLISTS] CC v8-$arch-ports automatically on platform-specific CLs (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 the V8 project authors. All rights reserved. 1 # Copyright 2013 the V8 project authors. All rights reserved.
2 # Redistribution and use in source and binary forms, with or without 2 # Redistribution and use in source and binary forms, with or without
3 # modification, are permitted provided that the following conditions are 3 # modification, are permitted provided that the following conditions are
4 # met: 4 # met:
5 # 5 #
6 # * Redistributions of source code must retain the above copyright 6 # * Redistributions of source code must retain the above copyright
7 # notice, this list of conditions and the following disclaimer. 7 # notice, this list of conditions and the following disclaimer.
8 # * Redistributions in binary form must reproduce the above 8 # * Redistributions in binary form must reproduce the above
9 # copyright notice, this list of conditions and the following 9 # copyright notice, this list of conditions and the following
10 # disclaimer in the documentation and/or other materials provided 10 # disclaimer in the documentation and/or other materials provided
(...skipping 25 matching lines...) Expand all
36 'public_api': { 36 'public_api': {
37 'filepath': 'include/', 37 'filepath': 'include/',
38 }, 38 },
39 'snapshot': { 39 'snapshot': {
40 'filepath': 'src/snapshot/', 40 'filepath': 'src/snapshot/',
41 }, 41 },
42 'debugger': { 42 'debugger': {
43 'filepath': 'src/debug/', 43 'filepath': 'src/debug/',
44 }, 44 },
45 'interpreter': { 45 'interpreter': {
46 'filepath': 'src/interpreter/', 46 'filepath': 'src/interpreter/' \
47 'filepath': 'test/cctest/interpreter/', 47 '|test/cctest/interpreter/' \
48 'filepath': 'test/unittests/interpreter/', 48 '|test/unittests/interpreter/',
49 }, 49 },
50 'feature_shipping_status': { 50 'feature_shipping_status': {
51 'filepath': 'src/flag-definitions.h', 51 'filepath': 'src/flag-definitions.h',
52 }, 52 },
53 'gc_changes': { 53 'gc_changes': {
54 'filepath': 'src/heap/', 54 'filepath': 'src/heap/',
55 }, 55 },
56 'arm': {
57 'filepath': '/arm/',
Michael Hablich 2016/01/15 14:15:42 shouldn't it be 'src/arm'?
Jakob Kummerow 2016/01/15 14:54:33 Nope, it should match any ".../foo/arm/bar/..." pa
58 },
59 'ia32': {
60 'filepath': '/ia32/',
Michael Hablich 2016/01/15 14:15:42 same as above
61 }
56 }, 62 },
57 63
58 'WATCHLISTS': { 64 'WATCHLISTS': {
59 'public_api': [ 65 'public_api': [
60 'phajdan.jr@chromium.org', 66 'phajdan.jr@chromium.org',
61 ], 67 ],
62 'snapshot': [ 68 'snapshot': [
63 'yangguo@chromium.org', 69 'yangguo@chromium.org',
64 ], 70 ],
65 'debugger': [ 71 'debugger': [
66 'yangguo@chromium.org', 72 'yangguo@chromium.org',
67 ], 73 ],
68 'interpreter': [ 74 'interpreter': [
69 'rmcilroy@chromium.org', 75 'rmcilroy@chromium.org',
70 'oth@chromium.org', 76 'oth@chromium.org',
71 ], 77 ],
72 'feature_shipping_status': [ 78 'feature_shipping_status': [
73 'hablich@chromium.org', 79 'hablich@chromium.org',
74 ], 80 ],
75 'gc_changes': [ 81 'gc_changes': [
76 'hpayer@chromium.org', 82 'hpayer@chromium.org',
77 'ulan@chromium.org', 83 'ulan@chromium.org',
78 ], 84 ],
85 'arm': [
86 'v8-mips-ports@googlegroups.com',
87 'v8-ppc-ports@googlegroups.com',
88 ],
89 'ia32': [
90 'v8-x87-ports@googlegroups.com',
91 ],
79 }, 92 },
80 } 93 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698