Glad to hear it. A PR would be much appreciated, but I think we just need to declare the function static inline, or move its body to a .cc file (since it is used).
Which compiler/version are you using? (Presumably some variant of Clang on MacOS?) On Wed, Feb 2, 2022, at 11:24, Li Jin wrote: > David Li - Thanks! -DBUILD_WARNING_LEVEL=PRODUCTION does the trick! I am > happy to submit a PR to remove the used function if that's the proper way > to fix this. > > Benson - I think I got this on fresh build too with -DARROW_COMPUTE=ON (I > am using the preset ninja-debug-basic) > > On Wed, Feb 2, 2022 at 11:01 AM Benson Muite <benson_mu...@emailplus.org> > wrote: > >> Perhaps try a fresh build in a clean build directory. Most testing has >> been on Clang 11. Also check your other options, is -DARROW_COMPUTE=ON set? >> >> On 2/2/22 6:46 PM, Li Jin wrote: >> > Thanks! >> > >> > -DARROW_DEPENDENCY_SOURCE=BUNDLED >> > >> > seems to do the trick - I can build without adding find_package! >> > >> > Although, any idea how to get pass this? >> > >> > " >> > >> > >> */Users/icexelloss/workspace/arrow/cpp/src/arrow/compute/kernels/scalar_string_internal.h:216:20: >> > **error: **unused function 'StringClassifyDoc' >> [-Werror,-Wunused-function]* >> > >> > static FunctionDoc StringClassifyDoc(std::string class_summary, >> std::string >> > class_desc, >> > >> > " >> > >> > On Wed, Feb 2, 2022 at 10:11 AM Benson Muite <benson_mu...@emailplus.org >> > >> > wrote: >> > >> >> Can you try using one of the CMAKE options: >> >> -DARROW_DEPENDENCY_SOURCE=BREW >> >> -DARROW_DEPENDENCY_SOURCE=BUNDLED >> >> >> >> see https://arrow.apache.org/docs/developers/cpp/building.html >> >> >>
