[pacman-dev] [PATCH] Removed multiple definitions of pkgfrom_t

Diogo Sousa diogogsousa at gmail.com
Sat Aug 27 01:25:09 EDT 2011


libalpm now exports type alpm_pkgfrom_t in alpm.h, which may be
used by frontends. 
Pacman now uses alpm_pkgfrom_t instead
of replicating that type (pkg_from as was in
src/pacman/package.h)

Signed-off-by: Diogo Sousa <diogogsousa at gmail.com>
---
 lib/libalpm/alpm.h    |    6 ++++++
 lib/libalpm/package.h |    6 ------
 src/pacman/package.c  |    2 +-
 src/pacman/package.h  |    9 +--------
 4 files changed, 8 insertions(+), 15 deletions(-)

diff --git a/lib/libalpm/alpm.h b/lib/libalpm/alpm.h
index fc8f0bc..b1e9a88 100644
--- a/lib/libalpm/alpm.h
+++ b/lib/libalpm/alpm.h
@@ -197,6 +197,12 @@ typedef struct _alpm_file_t {
 	mode_t mode;
 } alpm_file_t;
 
+typedef enum _alpm_pkgfrom_t {
+	PKG_FROM_FILE = 1,
+	PKG_FROM_LOCALDB,
+	PKG_FROM_SYNCDB
+} alpm_pkgfrom_t;
+
 /** Package filelist container */
 typedef struct _alpm_filelist_t {
 	size_t count;
diff --git a/lib/libalpm/package.h b/lib/libalpm/package.h
index 25d1b1a..a75f6ae 100644
--- a/lib/libalpm/package.h
+++ b/lib/libalpm/package.h
@@ -34,12 +34,6 @@
 #include "db.h"
 #include "signing.h"
 
-typedef enum _alpm_pkgfrom_t {
-	PKG_FROM_FILE = 1,
-	PKG_FROM_LOCALDB,
-	PKG_FROM_SYNCDB
-} alpm_pkgfrom_t;
-
 /** Package operations struct. This struct contains function pointers to
  * all methods used to access data in a package to allow for things such
  * as lazy package intialization (such as used by the file backend). Each
diff --git a/src/pacman/package.c b/src/pacman/package.c
index 80c6bf2..ef483ef 100644
--- a/src/pacman/package.c
+++ b/src/pacman/package.c
@@ -61,7 +61,7 @@ static void deplist_display(const char *title,
  * @param from the type of package we are dealing with
  * @param extra should we show extra information
  */
-void dump_pkg_full(alpm_pkg_t *pkg, enum pkg_from from, int extra)
+void dump_pkg_full(alpm_pkg_t *pkg, alpm_pkgfrom_t from, int extra)
 {
 	const char *reason;
 	time_t bdate, idate;
diff --git a/src/pacman/package.h b/src/pacman/package.h
index 6f71d20..4bbc2ba 100644
--- a/src/pacman/package.h
+++ b/src/pacman/package.h
@@ -22,14 +22,7 @@
 
 #include <alpm.h>
 
-/* TODO it would be nice if we didn't duplicate a backend type */
-enum pkg_from {
-	PKG_FROM_FILE = 1,
-	PKG_FROM_LOCALDB,
-	PKG_FROM_SYNCDB
-};
-
-void dump_pkg_full(alpm_pkg_t *pkg, enum pkg_from from, int extra);
+void dump_pkg_full(alpm_pkg_t *pkg, alpm_pkgfrom_t from, int extra);
 
 void dump_pkg_backups(alpm_pkg_t *pkg);
 void dump_pkg_files(alpm_pkg_t *pkg, int quiet);
-- 
1.7.6.1



More information about the pacman-dev mailing list