populate THRIFT_INCLUDE_PATH

Summary:
Populate the new environment variable if relevant path for it found.

For the OSS getdeps builds of things depending on fb303 this will allow them to be able to be able to find the fb303 thrift files from the declared fb303 dependency, rather than needing duplicates/stubs inside each package.

Also adds python as dependency to fbthrift manifest,  found it needed it when testing

Reviewed By: HarveyHunt

Differential Revision: D33771979

fbshipit-source-id: 9635221c0e6a2ac225d7d0babe76f1c919b583de

build/fbcode_builder/getdeps/buildopts.py

@@ -240,6 +240,11 @@ class BuildOptions(object):
 
             add_path_entry(env, "CMAKE_PREFIX_PATH", d)
 
+            # Tell the thrift compiler about includes it needs to consider
+            thriftdir = os.path.join(d, "include/thrift-files")
+            if os.path.exists(thriftdir):
+                add_path_entry(env, "THRIFT_INCLUDE_PATH", thriftdir)
+
             # Allow resolving shared objects built earlier (eg: zstd
             # doesn't include the full path to the dylib in its linkage
             # so we need to give it an assist)

build/fbcode_builder/manifests/fbthrift

@@ -19,6 +19,7 @@ fizz
 fmt
 googletest
 libsodium
+python
 python-six
 zstd