Hello community, here is the log from the commit of package libsmi for openSUSE:Factory checked in at 2015-02-06 10:53:43 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/libsmi (Old) and /work/SRC/openSUSE:Factory/.libsmi.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "libsmi" Changes: -------- --- /work/SRC/openSUSE:Factory/libsmi/libsmi.changes 2015-01-25 21:14:48.000000000 +0100 +++ /work/SRC/openSUSE:Factory/.libsmi.new/libsmi.changes 2015-02-06 10:53:44.000000000 +0100 @@ -1,0 +2,14 @@ +Tue Jan 27 22:09:31 UTC 2015 - crrodrig...@opensuse.org + +- fix missing pkgconfig buildRequires +- Run make check but ignore failures. + +------------------------------------------------------------------- +Tue Jan 27 22:05:41 UTC 2015 - crrodrig...@opensuse.org + +- libsmi-exports.patch: Export only the symbols of the public + API, specially hide and avoid using internal + vasprintf and asnprintf implementations that will + name-clash with the C library. + +------------------------------------------------------------------- New: ---- libsmi-exports.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ libsmi.spec ++++++ --- /var/tmp/diff_new_pack.yFajxY/_old 2015-02-06 10:53:45.000000000 +0100 +++ /var/tmp/diff_new_pack.yFajxY/_new 2015-02-06 10:53:45.000000000 +0100 @@ -1,7 +1,7 @@ # # spec file for package libsmi # -# Copyright (c) 2015 SUSE LINUX Products GmbH, Nuernberg, Germany. +# Copyright (c) 2015 SUSE LINUX GmbH, Nuernberg, Germany. # # All modifications and additions to the file contributed by third parties # remain the property of their copyright owners, unless otherwise agreed @@ -29,10 +29,12 @@ Patch2: libsmi-CVE-2010-2891.patch Patch3: libsmi-flex.patch Patch4: libsmi-bison-3.0.patch +Patch5: libsmi-exports.patch BuildRoot: %{_tmppath}/%{name}-%{version}-build BuildRequires: bison BuildRequires: flex BuildRequires: libtool +BuildRequires: pkg-config %description The purpose of libsmi is to @@ -82,6 +84,7 @@ %patch3 -p1 %endif %patch4 -p1 +%patch5 -p1 %build autoreconf --force --install @@ -94,9 +97,11 @@ %install make install DESTDIR=%buildroot - rm -f %{buildroot}%{_libdir}/*.la +%check +make check || cat test/test-suite.log && exit 0 + %post -n libsmi2 -p /sbin/ldconfig %postun -n libsmi2 -p /sbin/ldconfig ++++++ libsmi-exports.patch ++++++ --- libsmi-0.4.8.orig/lib/Makefile.am +++ libsmi-0.4.8/lib/Makefile.am @@ -11,6 +11,7 @@ # @(#) $Id: Makefile.am 7735 2008-02-15 08:09:17Z schoenw $ # +AM_CFLAGS = -fvisibility=hidden EXTRA_DIST = parser-sming.y parser-smi.y \ scanner-sming.l scanner-smi.l \ errormacros.h data.h check.h error.h util.h \ @@ -29,10 +30,14 @@ man_MANS = libsmi.3 smi_config.3 smi_mo smi_class.3 smi_attribute.3 smi_identity.3 \ smi_event.3 lib_LTLIBRARIES = libsmi.la -libsmi_la_SOURCES = data.c check.c error.c util.c snprintf.c smi.c \ +noinst_LTLIBRARIES = libsmi-internal.la + +libsmi_la_SOURCES = data.c check.c error.c util.c smi.c \ parser-smi.c scanner-smi.c \ parser-sming.c scanner-sming.c -libsmi_la_LDFLAGS = -version-info @VERSION_LIBTOOL@ +libsmi_la_LDFLAGS = -no-undefined -version-info @VERSION_LIBTOOL@ + +libsmi_internal_la_SOURCES = $(libsmi_la_SOURCES) parser-smi.c parser-smi.tab.h: parser-smi.y scanner-smi.h parser-smi.h $(BISON) --defines=parser-smi.tab.h -t -psmi -o parser-smi.c parser-smi.y --- libsmi-0.4.8.orig/lib/smi.h +++ libsmi-0.4.8/lib/smi.h @@ -16,14 +16,11 @@ #include <stdlib.h> #include <stdarg.h> -#ifdef HAVE_STDINT_H #include <stdint.h> -#endif -#ifdef HAVE_LIMITS_H -#include "limits.h" -#endif +#include <limits.h> #include <time.h> +#pragma GCC visibility push(default) #ifdef __cplusplus extern "C" { @@ -623,9 +620,9 @@ extern int smiPack(SmiNode *row, SmiValu * to free the allocated memory. */ -extern int smiAsprintf(char **strp, const char *format, ...); +extern int smiAsprintf(char **strp, const char *format, ...) __attribute__ ((format (printf, 2, 3))); -extern int smiVasprintf(char **strp, const char *format, va_list ap); +extern int smiVasprintf(char **strp, const char *format, va_list ap) __attribute__ ((format (printf, 2, 0))); /* @@ -653,8 +650,8 @@ extern void _smiFree(char *, int, void * #define smiFree(p) _smiFree(__FILE__, __LINE__, p) #else -extern void *smiMalloc(size_t size); -extern void *smiRealloc(void *ptr, size_t size); +extern void *smiMalloc(size_t size) __attribute((malloc, alloc_size(2))); +extern void *smiRealloc(void *ptr, size_t size) __attribute((alloc_size(2))); extern char *smiStrdup(const char *s1); extern char *smiStrndup(const char *s1, size_t n); extern void smiFree(void *ptr); @@ -664,5 +661,6 @@ extern void smiFree(void *ptr); } #endif +#pragma GCC visibility pop #endif /* _SMI_H */ --- libsmi-0.4.8.orig/lib/smi.h.in +++ libsmi-0.4.8/lib/smi.h.in @@ -24,6 +24,7 @@ #endif #include <time.h> +#pragma GCC visibility push(default) #ifdef __cplusplus extern "C" { @@ -663,6 +664,6 @@ extern void smiFree(void *ptr); #ifdef __cplusplus } #endif - +#pragma GCC visibility pop #endif /* _SMI_H */ --- libsmi-0.4.8.orig/lib/smi.c +++ libsmi-0.4.8/lib/smi.c @@ -30,7 +30,6 @@ #include "data.h" #include "error.h" #include "util.h" -#include "snprintf.h" #ifdef BACKEND_SMI #include "scanner-smi.h" --- libsmi-0.4.8.orig/lib/util.c +++ libsmi-0.4.8/lib/util.c @@ -19,8 +19,8 @@ #include <string.h> #include <time.h> +#include "smi.h" #include "util.h" -#include "snprintf.h" #ifdef HAVE_DMALLOC_H #include <dmalloc.h> --- libsmi-0.4.8.orig/tools/Makefile.am +++ libsmi-0.4.8/tools/Makefile.am @@ -29,7 +29,7 @@ smiquery_SOURCES = smiquery.c shhopt.c smiquery_LDADD = ../lib/libsmi.la smilint_SOURCES = smilint.c shhopt.c -smilint_LDADD = ../lib/libsmi.la +smilint_LDADD = ../lib/libsmi.la ../lib/libsmi-internal.la smixlate_SOURCES = smixlate.c shhopt.c dstring.h dstring.c smixlate_LDADD = ../lib/libsmi.la -- To unsubscribe, e-mail: opensuse-commit+unsubscr...@opensuse.org For additional commands, e-mail: opensuse-commit+h...@opensuse.org