Skip to content

Commit 57d0bad

Browse files
committed
build: use internal libsqlite.la
Problem: content-sqlite and job-archive use external libsqlite3 but we now have an internal copy of the library. Drop configure requirement and change the build rules for those broker modules to use the internal copy of libsqlite3.
1 parent 7314c71 commit 57d0bad

File tree

4 files changed

+5
-8
lines changed

4 files changed

+5
-8
lines changed

configure.ac

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -349,7 +349,6 @@ if test "$have_libsystemd" = yes -a "$have_sd_bus_h" = yes; then
349349
fi
350350
PKG_CHECK_MODULES([HWLOC], [hwloc >= 1.11.1], [], [])
351351
PKG_CHECK_MODULES([LZ4], [liblz4], [], [])
352-
PKG_CHECK_MODULES([SQLITE], [sqlite3], [], [])
353352
PKG_CHECK_MODULES([LIBUUID], [uuid], [], [])
354353
PKG_CHECK_MODULES([CURSES], [ncursesw], [], [])
355354
PKG_CHECK_MODULES([LIBARCHIVE], [libarchive], [], [])

src/modules/Makefile.am

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -104,12 +104,11 @@ content_sqlite_la_SOURCES = \
104104
content-sqlite/content-sqlite.c
105105
content_sqlite_la_CPPFLAGS = \
106106
$(AM_CPPFLAGS) \
107-
$(SQLITE_CFLAGS) \
108107
$(LZ4_CFLAGS)
109108
content_sqlite_la_LIBADD = \
110109
$(top_builddir)/src/common/libflux-internal.la \
111110
$(top_builddir)/src/common/libflux-core.la \
112-
$(SQLITE_LIBS) \
111+
$(top_builddir)/src/common/libsqlite3/libsqlite3.la \
113112
$(LZ4_LIBS)
114113
content_sqlite_la_LDFLAGS = $(fluxmod_ldflags) -module
115114

@@ -140,13 +139,12 @@ job_archive_la_SOURCES = \
140139
job-archive/job-archive.c
141140
job_archive_la_CPPFLAGS = \
142141
$(AM_CPPFLAGS) \
143-
$(SQLITE_CFLAGS) \
144142
$(FLUX_SECURITY_CFLAGS)
145143
job_archive_la_LIBADD = \
146144
$(top_builddir)/src/common/libflux-internal.la \
147145
$(top_builddir)/src/common/libflux-core.la \
148-
$(JANSSON_LIBS) \
149-
$(SQLITE_LIBS)
146+
$(top_builddir)/src/common/libsqlite3/libsqlite3.la \
147+
$(JANSSON_LIBS)
150148
job_archive_la_LDFLAGS = $(fluxmod_ldflags) -module
151149

152150
job_exec_la_SOURCES =

src/modules/content-sqlite/content-sqlite.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,12 @@
1818
#include <unistd.h>
1919
#include <unistd.h>
2020
#include <sys/statvfs.h>
21-
#include <sqlite3.h>
2221
#include <lz4.h>
2322
#include <flux/core.h>
2423
#include <jansson.h>
2524
#include <assert.h>
2625

26+
#include "src/common/libsqlite3/sqlite3.h"
2727
#include "src/common/libutil/blobref.h"
2828
#include "src/common/libutil/log.h"
2929
#include "src/common/libutil/errno_safe.h"

src/modules/job-archive/job-archive.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@
2222
#include <stdbool.h>
2323
#include <flux/core.h>
2424
#include <jansson.h>
25-
#include <sqlite3.h>
2625

26+
#include "src/common/libsqlite3/sqlite3.h"
2727
#include "src/common/libutil/log.h"
2828
#include "src/common/libutil/fsd.h"
2929
#include "src/common/libutil/tstat.h"

0 commit comments

Comments
 (0)