Merge pull request #2698 from brave/enable_vmp_on_macos

Add widevine sig deps into proper deps list
brave · Jun 14, 2019 · 6b3e379 · 6b3e379
2 parents 4811707 + 22dffae
commit 6b3e379
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/build/mac/BUILD.gn b/build/mac/BUILD.gn
@@ -81,9 +81,10 @@ action("create_pkg") {
     "unsigned/$brave_pkg",
   ]
 
-  deps = [":sign_app"]
   if (enable_widevine_cdm_host_verification) {
-    deps += [ ":sign_chrome_framework_for_widevine" ]
+    deps = [ ":sign_chrome_framework_for_widevine" ]
+  } else {
+    deps = [":sign_app"]
   }
 }
 
@@ -128,7 +129,11 @@ action("create_dmg") {
     "--symlink", "/Applications",
   ]
 
-  deps = [":sign_app"]
+  if (enable_widevine_cdm_host_verification) {
+    deps = [ ":sign_chrome_framework_for_widevine" ]
+  } else {
+    deps = [":sign_app"]
+  }
 }
 
 action("create_unsigned_dmg") {
@@ -238,10 +243,5 @@ if (enable_widevine_cdm_host_verification) {
 }
 
 group("create_dist_mac") {
-  deps = [ ":create_dmg" ]
-
-  deps += [
-    ":sign_app",
-    ":sign_dmg",
-  ]
+  deps = [ ":sign_dmg" ]
 }