On Fri, Mar 12, 2021 at 08:38:41AM +0530, Dilip Kumar wrote: > On Thu, Mar 11, 2021 at 11:55 PM Justin Pryzby <pry...@telsasoft.com> wrote: > > On Wed, Mar 10, 2021 at 08:28:58PM -0600, Justin Pryzby wrote: > > > This includes a patch to use pkgconfig, in an attempt to build on mac, > > > which > > > currently fails like: > > > > > > https://cirrus-ci.com/task/5993712963551232?command=build#L126 > > > checking for LZ4_compress in -llz4... no > > > configure: error: library 'lz4' is required for LZ4 support > > > > This includes a 2nd attempt to use pkg-config to build on mac. > > > > If this doesn't work, we should ask for help from a mac user who wants to > > take > > on a hopefully-quick project. > > Thanks for your help. I did not understand the reason for removal of > lz4.h header check?
It can stay. I tried to base this on ICU and LIBXML, but I see now they both have that. > -if test "$with_lz4" = yes; then > - AC_CHECK_HEADERS(lz4/lz4.h, [], > - [AC_CHECK_HEADERS(lz4.h, [], [AC_MSG_ERROR([lz4.h header file > is required for LZ4])])]) > -fi > - -- Justin
