[pacman-dev] [PATCH 3/4] Build vercmp without needing link to libalpm

Dan McGee dan at archlinux.org
Wed May 5 06:45:50 CEST 2010


Include the object file directly from the libalpm version comparison code as
it is the only thing we need. This drops the dependency of vercmp on
libalpm and all of the stuff we know it drags in.

Signed-off-by: Dan McGee <dan at archlinux.org>
---
 src/util/Makefile.am |    2 +-
 src/util/vercmp.c    |    6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/util/Makefile.am b/src/util/Makefile.am
index 638e276..7dce9dc 100644
--- a/src/util/Makefile.am
+++ b/src/util/Makefile.am
@@ -16,7 +16,7 @@ INCLUDES = -I$(top_srcdir)/lib/libalpm
 AM_CFLAGS = -pedantic -D_GNU_SOURCE
 
 vercmp_SOURCES = vercmp.c
-vercmp_LDADD = $(top_builddir)/lib/libalpm/.libs/libalpm.la
+vercmp_LDADD = $(top_builddir)/lib/libalpm/version.o
 
 testpkg_SOURCES = testpkg.c
 testpkg_LDADD = $(top_builddir)/lib/libalpm/.libs/libalpm.la
diff --git a/src/util/vercmp.c b/src/util/vercmp.c
index 778ac55..959dc13 100644
--- a/src/util/vercmp.c
+++ b/src/util/vercmp.c
@@ -23,12 +23,14 @@
 #include <stdio.h> /* printf */
 #include <string.h> /* strncpy */
 
-#include <alpm.h>
-
 #define BASENAME "vercmp"
 
 #define MAX_LEN 255
 
+/* forward declaration, comes from vercmp.o in libalpm source that is linked in
+ * directly so we don't have any library deps */
+int alpm_pkg_vercmp(const char *a, const char *b);
+
 static void usage()
 {
 	fprintf(stderr, "usage: %s <ver1> <ver2>\n\n", BASENAME);
-- 
1.7.1



More information about the pacman-dev mailing list