Github user robertamarton commented on a diff in the pull request:
https://github.com/apache/incubator-trafodion/pull/118#discussion_r41885829
--- Diff: core/sqf/sqenvcom.sh ---
@@ -580,6 +547,13 @@ EOF
echo "**** ERROR: Unable to determine location of HBase lib directory"
fi
+ if [[ -d $TOOLSDIR/thrift-0.9.0 ]]; then
+ # this is mostly for a build environment, where we need
+ # thrift from TOOLSDIR
+ export THRIFT_LIB_DIR=$TOOLSDIR/thrift-0.9.0/lib
+ export THRIFT_INC_DIR=$TOOLSDIR/thrift-0.9.0/include
+ fi
+
if [ -n "$HBASE_CNF_DIR" -a -n "$HADOOP_CNF_DIR" -a \
--- End diff --
You removed Thrift from the required list in bldenvchk and set the variable
only if Thrift is installed in the TOOLSDIR. What happens if Thrift is not
installed?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
