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

Issue 2574653002: Service Manager: Override DLL search path when launching services in component builds (Closed)

Created:
4 years ago by Ken Rockot(use gerrit already)
Modified:
4 years ago
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Service Manager: Override DLL search path when launching services in component builds Service executables don't live alongside other build artifacts, so in component builds Windows doesn't normally know where to look for them at runtime. This overrides the DLL search path temporarily before launching any service child process. BUG=None R=ben@chromium.org Committed: https://crrev.com/386c2e2e5d37dba2144029fc990faeedf35733a2 Cr-Commit-Position: refs/heads/master@{#438086}

Patch Set 1 #

Patch Set 2 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -0 lines) Patch
M services/service_manager/runner/host/child_process_host.cc View 1 3 chunks +25 lines, -0 lines 0 comments Download

Messages

Total messages: 18 (13 generated)
Ken Rockot(use gerrit already)
This at least fixes the DLL loading error. I don't know what I can run ...
4 years ago (2016-12-13 02:48:58 UTC) #3
Ben Goodger (Google)
lgtm I've been running mojo_runner --service=mash_session --use-test-config --window-manager=simple_wm
4 years ago (2016-12-13 02:51:22 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2574653002/20001
4 years ago (2016-12-13 06:49:05 UTC) #13
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years ago (2016-12-13 06:53:41 UTC) #16
commit-bot: I haz the power
4 years ago (2016-12-13 06:56:20 UTC) #18
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/386c2e2e5d37dba2144029fc990faeedf35733a2
Cr-Commit-Position: refs/heads/master@{#438086}

Powered by Google App Engine
This is Rietveld 408576698