On 01/15/2012 05:03 PM, Antoine Beaupré wrote:
> On Sat, 07 Jan 2012 21:39:56 +0100, Julian Taylor
> <jtaylor.deb...@googlemail.com> wrote:
>> bantool: ${BANTOOL_OBJECTS}
>> - ${CC} ${CFLAGS} ${LDFLAGS} -o $@ ${BANTOOL_OBJECTS} ${IRCDLIBS}
>> + ${CC} ${CFLAGS} ${LDFLAGS} -o $@ ${BANTOOL_OBJECTS} ${IRCDLIBS} ${LIBS}
>
> I am not sure how this fixes the problem: isn't the -lsqlite3 flag still
> in ${IRCDLIBS}?
>
> If this patch was tested and works, I'll gladly commit it. :)
>
> A.
> the issue is that lsqlite3 was in LDFLAGS before which is placed before the objects I moved it to LIBS the variable where libraries are usually placed. true, it is now in IRCDLIBS and LIBS, so you can remove the additional LIBS there. (though specifying a library multiple times does no harm.) The patch was build-tested in ubuntu and debian, but I did no runtime tests.
signature.asc
Description: OpenPGP digital signature
