From edbb0ce21030ac574c376ecca8f56caf47628cc6 Mon Sep 17 00:00:00 2001 From: Michael Bolin Date: Tue, 2 May 2017 10:49:43 -0700 Subject: [PATCH] Build //eden/fs/service:thrift-EdenService-pyremote as part of :all target. Summary: When we say we are building "all" of Eden, we generally want to ensure we have built the pyremote service that the Hg extension needs, as well. Reviewed By: wez Differential Revision: D4982118 fbshipit-source-id: 7282266e82e662a48b5ced6da6b7daedab8b65ea --- eden/cli/TARGETS | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/eden/cli/TARGETS b/eden/cli/TARGETS index 55ad8a13cc..a3ab2c0b2d 100644 --- a/eden/cli/TARGETS +++ b/eden/cli/TARGETS @@ -28,13 +28,14 @@ python_unittest( ) for build_target, suffix in get_daemon_versions(): - # The :all rule is a convenience to ensure that both the CLI and the daemon - # are built. + # The :all rule is a convenience to ensure that the CLI, daemon, and Thrift + # Python client for the Hg extension are built. buck_genrule( name = 'all%s' % suffix, srcs = [ ':cli', build_target.replace('@/', '//'), + '//eden/fs/service:thrift-EdenService-pyremote', ], out = 'eden%s-stamp' % suffix, cmd = 'touch $OUT',