New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 884763 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Sep 17
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

M-70 failing to compile.

Project Member Reported by newcomer@chromium.org, Sep 17

Issue description


Someone merged a CL without testing it probably. Just ccing a few people who touched the file.

In file included from ../../chromeos/services/assistant/assistant_manager_service_impl.cc:5:                                                                                                                                                                                              
In file included from ../../chromeos/services/assistant/assistant_manager_service_impl.h:20:                                                                                                                                                                                              
In file included from ../../chromeos/services/assistant/platform_api_impl.h:17:                                                                                                                                                                                                           
../../chromeos/services/assistant/platform/system_provider_impl.h:29:19: error: unknown type name 'TpmProcessingType'                                                                                                                                                                     
  void ProcessTpm(TpmProcessingType type,                                                                                                                                                                                                                                                 
                  ^                                                                                                                                                                                                                                                                       
../../chromeos/services/assistant/platform/system_provider_impl.h:31:19: error: unknown type name 'TpmCallback'                                                                                                                                                                           
                  TpmCallback on_done) override;                                                                                                                                                                                                                                          
                  ^                                                                                                                                                                                                                                                                       
2 errors generated.        


 
Labels: -Pri-3 M-70 Pri-1
Cc: wutao@chromium.org
Components: UI>Shell>Assistant
Labels: OS-Chrome
Is this on M70 branch or HEAD?
M70 branch
This should be HEAD.
Xiaohui,

Does libassistant code got merged back to M70 or it automatically merged?
ProcessTpm was removed in new libassistant code. If libassistant code got merged back to M70, then we need to merge updated code [1] back to M70.

[1]
https://chromium-review.googlesource.com/c/chromium/src/+/1226203
Talked to newcomer@ and it should be a .gclient file setup issue. 
Status: Fixed (was: Assigned)
Yep! Thanks much.

Sign in to add a comment