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

Unified Diff: remoting/remoting.gyp

Issue 146623004: Avoid rebuilding remoting host every time gyp runs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/remoting.gyp
diff --git a/remoting/remoting.gyp b/remoting/remoting.gyp
index 713ee4c3e00314f5a6c558d3600fb0e117d522cf..da3a51f6e5b7eb01e141ec1868a68d4a1964b798 100644
--- a/remoting/remoting.gyp
+++ b/remoting/remoting.gyp
@@ -59,16 +59,17 @@
['OS=="win"', {
'host_plugin_extension': 'dll',
'host_plugin_prefix': '',
- # Use auto-generated CLSIDs to make sure that the newly installed COM
- # classes will be used during/after upgrade even if there are old
- # instances running already.
- # The parameter at the end is ignored, but needed to make sure that the
- # script will be invoked separately for each CLSID. Otherwise GYP will
- # reuse the value returned by the first invocation of the script.
+
+ # Each CLSID is a hash of the current version string salted with an
+ # arbitrary GUID. This ensures that the newly installed COM classes will
+ # be used during/after upgrade even if there are old instances running
+ # already.
+ # The IDs are not random to avoid rebuilding host when it's not
+ # necessary.
'daemon_controller_clsid':
- '<!(python -c "import uuid; print uuid.uuid4()" 1)',
+ '<!(python -c "import uuid; print uuid.uuid5(uuid.UUID(\'655bd819-c08c-4b04-80c2-f160739ff6ef\'), \'<(version_full)\')")',
'rdp_desktop_session_clsid':
- '<!(python -c "import uuid; print uuid.uuid4()" 2)',
+ '<!(python -c "import uuid; print uuid.uuid5(uuid.UUID(\'6a7699f0-ee43-43e7-aa30-a6738f9bd470\'), \'<(version_full)\')")',
}],
],
'remoting_locales': [
« 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