Fixed docker-push target.

Avinash Sridharan · asridharan · commit 8e5c5a26ca31 · 2018-10-30T23:02:00.000-07:00
`docker-push` didn't have a dependency on the `dockerize` target because of which it was not building the deployment image before pushing it to the registry. Introduced the dependency to make `docker-push` work.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -119,6 +119,7 @@ add_custom_target(dockerpush
     COMMAND docker tag ${deployment_image_name}:${deployment_image_version} ${deployment_push_prefix}/${deployment_image_name}:latest
     COMMAND docker push ${deployment_push_prefix}/${deployment_image_name}:${deployment_image_version}
     COMMAND docker push ${deployment_push_prefix}/${deployment_image_name}:latest
+    DEPENDS dockerize
     COMMENT "pushing to container registry"
 )
 

Original file line number	Diff line number	Diff line change
`@@ -119,6 +119,7 @@ add_custom_target(dockerpush`
`119`	`119`	`COMMAND docker tag ${deployment_image_name}:${deployment_image_version} ${deployment_push_prefix}/${deployment_image_name}:latest`
`120`	`120`	`COMMAND docker push ${deployment_push_prefix}/${deployment_image_name}:${deployment_image_version}`
`121`	`121`	`COMMAND docker push ${deployment_push_prefix}/${deployment_image_name}:latest`
	`122`	`+ DEPENDS dockerize`
`122`	`123`	`COMMENT "pushing to container registry"`
`123`	`124`	`)`
`124`	`125`