On Sun, Dec 24, 2017 at 08:30:11PM +0000, Stuart Henderson wrote:
> On 2017/12/24 11:35, Remi Locherer wrote:
> > Hi,
> >
> > liblognorm is a library to normalize log messages. It requires libfastjson
> > I sent in the previous mail. It can be used together with rsyslog.
> >
> > The patch is needed to run the test suite. 5 out of the 110 tests fail.
> > I informed the author about the failed tests.
> >
> > I tested on amd64.
> >
> > Reports from other platforms and OKs are welcome.
> >
> > Remi
>
> Don't hardcode /usr/local (in the patch), use $(LOCALBASE) instead and
> pass it in as a variable (or SUBST_CMD it if you prefer).
Thank you for spotting this. With "pass it in as a variable" do you mean
something like this:
TEST_ENV = INCDIR=${LOCALBASE}/include
And then use "-I$(INCDIR)" in the patch?
(But for this package I think jca@ came up with a solution that does not
need this patch.)
