|
|
Created:
6 years, 9 months ago by Avi (use Gerrit) Modified:
6 years, 8 months ago CC:
chromium-reviews, chromoting-reviews_chromium.org Base URL:
svn://svn.chromium.org/chrome/trunk/src Visibility:
Public. |
DescriptionFix remoting dependency.
BUG=357715
TEST=none
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260462
Patch Set 1 #
Total comments: 2
Patch Set 2 : better #Messages
Total messages: 22 (0 generated)
thanks for fixing it! LGTM after my comment is addressed. https://codereview.chromium.org/217133007/diff/1/remoting/remoting.gyp File remoting/remoting.gyp (right): https://codereview.chromium.org/217133007/diff/1/remoting/remoting.gyp#newcod... remoting/remoting.gyp:263: '../base/base.gyp:base_i18n', I think this dependency should be added in remoting_host, not remoting_base
https://codereview.chromium.org/217133007/diff/1/remoting/remoting.gyp File remoting/remoting.gyp (right): https://codereview.chromium.org/217133007/diff/1/remoting/remoting.gyp#newcod... remoting/remoting.gyp:263: '../base/base.gyp:base_i18n', On 2014/03/28 19:44:06, Sergey Ulanov wrote: > I think this dependency should be added in remoting_host, not remoting_base Done.
The CQ bit was checked by avi@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avi@chromium.org/217133007/20001
The CQ bit was unchecked by commit-bot@chromium.org
Commit queue rejected this change because the description was changed between the time the change entered the commit queue and the time it was ready to commit. You can safely check the commit box again.
The CQ bit was checked by sergeyu@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avi@chromium.org/217133007/20001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: tryserver.chromium on linux_chromium_chromeos_rel
The CQ bit was checked by avi@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avi@chromium.org/217133007/20001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: tryserver.chromium on linux_chromium_chromeos_rel
The CQ bit was checked by avi@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avi@chromium.org/217133007/20001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: tryserver.chromium on linux_chromium_chromeos_rel
The CQ bit was checked by avi@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avi@chromium.org/217133007/20001
Message was sent while issue was closed.
Change committed as 260462 |