r4204 - trunk/src/target/opkg/libopkg
thomas at sita.openmoko.org
thomas at sita.openmoko.org
Thu Mar 13 16:38:41 CET 2008
Author: thomas
Date: 2008-03-13 16:38:39 +0100 (Thu, 13 Mar 2008)
New Revision: 4204
Modified:
trunk/src/target/opkg/libopkg/Makefile.am
trunk/src/target/opkg/libopkg/file_util.c
trunk/src/target/opkg/libopkg/opkg_conf.c
trunk/src/target/opkg/libopkg/opkg_configure.c
trunk/src/target/opkg/libopkg/pkg.c
trunk/src/target/opkg/libopkg/pkg.h
trunk/src/target/opkg/libopkg/pkg_vec.c
trunk/src/target/opkg/libopkg/pkg_vec.h
Log:
opkg: enable -Werror in libopkg and fix resulting build failures
Modified: trunk/src/target/opkg/libopkg/Makefile.am
===================================================================
--- trunk/src/target/opkg/libopkg/Makefile.am 2008-03-13 14:44:42 UTC (rev 4203)
+++ trunk/src/target/opkg/libopkg/Makefile.am 2008-03-13 15:38:39 UTC (rev 4204)
@@ -1,5 +1,5 @@
-ALL_CFLAGS=-g -O -Wall -DHOST_CPU_STR=\"@host_cpu@\" -DBUILD_CPU=@build_cpu@ -DLIBDIR=\"@libdir@\" -DOPKGLIBDIR=\"@opkglibdir@\" -DDATADIR=\"@datadir@\" -I$(top_srcdir)
+ALL_CFLAGS=-g -O -Wall -Werror -DHOST_CPU_STR=\"@host_cpu@\" -DBUILD_CPU=@build_cpu@ -DLIBDIR=\"@libdir@\" -DOPKGLIBDIR=\"@opkglibdir@\" -DDATADIR=\"@datadir@\" -I$(top_srcdir)
libopkg_includedir=$(includedir)/libopkg
Modified: trunk/src/target/opkg/libopkg/file_util.c
===================================================================
--- trunk/src/target/opkg/libopkg/file_util.c 2008-03-13 14:44:42 UTC (rev 4203)
+++ trunk/src/target/opkg/libopkg/file_util.c 2008-03-13 15:38:39 UTC (rev 4204)
@@ -140,7 +140,7 @@
int i, err;
FILE *file;
- unsigned char *md5sum_hex;
+ char *md5sum_hex;
unsigned char md5sum_bin[md5sum_bin_len];
md5sum_hex = malloc(md5sum_hex_len + 1);
Modified: trunk/src/target/opkg/libopkg/opkg_conf.c
===================================================================
--- trunk/src/target/opkg/libopkg/opkg_conf.c 2008-03-13 14:44:42 UTC (rev 4203)
+++ trunk/src/target/opkg/libopkg/opkg_conf.c 2008-03-13 15:38:39 UTC (rev 4204)
@@ -551,14 +551,14 @@
if (strcmp(type, "option") == 0) {
opkg_conf_set_option(options, name, value);
} else if (strcmp(type, "src") == 0) {
- if (!nv_pair_list_find(pkg_src_list, name)) {
+ if (!nv_pair_list_find((nv_pair_list_t*) pkg_src_list, name)) {
pkg_src_list_append (pkg_src_list, name, value, extra, 0);
} else {
opkg_message(conf, OPKG_ERROR, "ERROR: duplicate src declaration. Skipping:\n\t src %s %s\n",
name, value);
}
} else if (strcmp(type, "src/gz") == 0) {
- if (!nv_pair_list_find(pkg_src_list, name)) {
+ if (!nv_pair_list_find((nv_pair_list_t*) pkg_src_list, name)) {
pkg_src_list_append (pkg_src_list, name, value, extra, 1);
} else {
opkg_message(conf, OPKG_ERROR, "ERROR: duplicate src declaration. Skipping:\n\t src %s %s\n",
Modified: trunk/src/target/opkg/libopkg/opkg_configure.c
===================================================================
--- trunk/src/target/opkg/libopkg/opkg_configure.c 2008-03-13 14:44:42 UTC (rev 4203)
+++ trunk/src/target/opkg/libopkg/opkg_configure.c 2008-03-13 15:38:39 UTC (rev 4204)
@@ -16,7 +16,7 @@
*/
#include "opkg.h"
-
+#include "sprintf_alloc.h"
#include "opkg_configure.h"
#include "opkg_state.h"
Modified: trunk/src/target/opkg/libopkg/pkg.c
===================================================================
--- trunk/src/target/opkg/libopkg/pkg.c 2008-03-13 14:44:42 UTC (rev 4203)
+++ trunk/src/target/opkg/libopkg/pkg.c 2008-03-13 15:38:39 UTC (rev 4204)
@@ -1159,10 +1159,10 @@
return 0;
}
-int pkg_name_version_and_architecture_compare(void *p1, void *p2)
+int pkg_name_version_and_architecture_compare(const void *p1, const void *p2)
{
- const pkg_t *a = *(const pkg_t **)p1;
- const pkg_t *b = *(const pkg_t **)p2;
+ const pkg_t *a = *(const pkg_t**) p1;
+ const pkg_t *b = *(const pkg_t**) p2;
int namecmp;
int vercmp;
if (!a->name || !b->name) {
@@ -1189,7 +1189,7 @@
return 0;
}
-int abstract_pkg_name_compare(void *p1, void *p2)
+int abstract_pkg_name_compare(const void *p1, const void *p2)
{
const abstract_pkg_t *a = *(const abstract_pkg_t **)p1;
const abstract_pkg_t *b = *(const abstract_pkg_t **)p2;
Modified: trunk/src/target/opkg/libopkg/pkg.h
===================================================================
--- trunk/src/target/opkg/libopkg/pkg.h 2008-03-13 14:44:42 UTC (rev 4203)
+++ trunk/src/target/opkg/libopkg/pkg.h 2008-03-13 15:38:39 UTC (rev 4204)
@@ -199,8 +199,8 @@
char *pkg_version_str_alloc(pkg_t *pkg);
int pkg_compare_versions(const pkg_t *pkg, const pkg_t *ref_pkg);
-int pkg_name_version_and_architecture_compare(void *a, void *b);
-int abstract_pkg_name_compare(void *a, void *b);
+int pkg_name_version_and_architecture_compare(const void *a, const void *b);
+int abstract_pkg_name_compare(const void *a, const void *b);
char * pkg_formatted_info(pkg_t *pkg );
char * pkg_formatted_field(pkg_t *pkg, const char *field );
Modified: trunk/src/target/opkg/libopkg/pkg_vec.c
===================================================================
--- trunk/src/target/opkg/libopkg/pkg_vec.c 2008-03-13 14:44:42 UTC (rev 4203)
+++ trunk/src/target/opkg/libopkg/pkg_vec.c 2008-03-13 15:38:39 UTC (rev 4204)
@@ -101,7 +101,6 @@
void pkg_vec_insert(pkg_vec_t *vec, const pkg_t *pkg)
{
- int i;
int found = 0;
#if 0
@@ -132,10 +131,9 @@
return 0;
}
-typedef int (*compare_fcn_t)(const void *, const void *);
-void pkg_vec_sort(pkg_vec_t *vec, int (*compar)(pkg_t *, pkg_t *))
+void pkg_vec_sort(pkg_vec_t *vec, compare_fcn_t compar)
{
- qsort(vec->pkgs, vec->len, sizeof(pkg_t *), (compare_fcn_t)compar);
+ qsort(vec->pkgs, vec->len, sizeof(pkg_t *), compar);
}
int pkg_vec_clear_marks(pkg_vec_t *vec)
@@ -191,7 +189,6 @@
*/
void abstract_pkg_vec_insert(abstract_pkg_vec_t *vec, abstract_pkg_t *pkg)
{
- int i;
#if 0
/* look for a duplicate pkg by name */
@@ -229,8 +226,8 @@
return 0;
}
-void abstract_pkg_vec_sort(pkg_vec_t *vec, int (*compar)(abstract_pkg_t *, abstract_pkg_t *))
+void abstract_pkg_vec_sort(pkg_vec_t *vec, compare_fcn_t compar)
{
- qsort(vec->pkgs, vec->len, sizeof(pkg_t *), (compare_fcn_t)compar);
+ qsort(vec->pkgs, vec->len, sizeof(pkg_t *), compar);
}
Modified: trunk/src/target/opkg/libopkg/pkg_vec.h
===================================================================
--- trunk/src/target/opkg/libopkg/pkg_vec.h 2008-03-13 14:44:42 UTC (rev 4203)
+++ trunk/src/target/opkg/libopkg/pkg_vec.h 2008-03-13 15:38:39 UTC (rev 4204)
@@ -47,8 +47,10 @@
/* this one never munges pkg */
void pkg_vec_insert(pkg_vec_t *vec, const pkg_t *pkg);
int pkg_vec_contains(pkg_vec_t *vec, pkg_t *apkg);
-void pkg_vec_sort(pkg_vec_t *vec, int (*compar)(pkg_t *, pkg_t *));
+typedef int (*compare_fcn_t)(const void *, const void *);
+void pkg_vec_sort(pkg_vec_t *vec, compare_fcn_t compar);
+
int pkg_vec_clear_marks(pkg_vec_t *vec);
int pkg_vec_mark_if_matches(pkg_vec_t *vec, const char *pattern);
@@ -57,6 +59,6 @@
void abstract_pkg_vec_insert(abstract_pkg_vec_t *vec, abstract_pkg_t *pkg);
abstract_pkg_t * abstract_pkg_vec_get(abstract_pkg_vec_t *vec, int i);
int abstract_pkg_vec_contains(abstract_pkg_vec_t *vec, abstract_pkg_t *apkg);
-void abstract_pkg_vec_sort(pkg_vec_t *vec, int (*compar)(abstract_pkg_t *, abstract_pkg_t *));
+void abstract_pkg_vec_sort(pkg_vec_t *vec, compare_fcn_t compar);
#endif
More information about the commitlog
mailing list