[Libosinfo] [PATCH 10/11] configure: bump libsoup version to 2.41

[Christophe Fergeau]

On Sat, Mar 23, 2013 at 06:42:07PM +0200, Zeeshan Ali (Khattak) wrote:
> On Sat, Mar 23, 2013 at 4:09 PM, Fabiano Fidêncio <fidencio at redhat.com> wrote:
> > Bumping libsoup version we are fixing a few deprecated warnings in our tests
> > (SOUP_TYPE_PROXY_RESOLVER_GNOME -> SOUP_TYPE_PROXY_RESOLVER_DEFAULT) and as a
> > side effect we don't need libsoup-gnome anymore.
> 
> Since we are only bumping the micro version and benefit is worth it,
> ACK from my side but I think you should ask Daniel too.

I couldn't find the exact release date for libsoup 2.41.0, but it's from
the second half of 2012 as 2.40.0 and 2.41.1 were released in this time
frame. Fedora 18 only has 2.40.3 not 2.41.
It should be doable to hide the GNOME stuff behind some -compat/#ifdef
magic...

Not a strong opposition to this patch if everyone agrees it's ok to have
this as is, but in my opinion libsoup 2.41 is a bit too new to depend on
now.

Christophe

> 
> > ---
> >  configure.ac          | 3 +--
> >  test/Makefile.am      | 2 --
> >  test/test-mediauris.c | 6 +-----
> >  test/test-treeuris.c  | 6 +-----
> >  4 files changed, 3 insertions(+), 14 deletions(-)
> >
> > diff --git a/configure.ac b/configure.ac
> > index 3fefdba..51aa5f5 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -39,8 +39,7 @@ m4_ifdef([AM_SILENT_RULES],[AM_SILENT_RULES([yes])])
> >
> >  PKG_CHECK_MODULES([GOBJECT], [gobject-2.0])
> >  PKG_CHECK_MODULES([GIO], [gio-2.0])
> > -PKG_CHECK_MODULES([SOUP], [libsoup-2.4])
> > -PKG_CHECK_MODULES([SOUP_GNOME], [libsoup-gnome-2.4])
> > +PKG_CHECK_MODULES([SOUP], [libsoup-2.4 >= 2.41])
> >  PKG_CHECK_MODULES([LIBXML], [libxml-2.0 >= 2.6.0])
> >  PKG_CHECK_MODULES([LIBXSLT], [libxslt >= 1.0.0])
> >
> > diff --git a/test/Makefile.am b/test/Makefile.am
> > index 5c05162..ef63154 100644
> > --- a/test/Makefile.am
> > +++ b/test/Makefile.am
> > @@ -25,7 +25,6 @@ COMMON_LDADD = \
> >         $(COVERAGE_LDFLAGS) \
> >         $(GOBJECT_LIBS) \
> >         $(SOUP_LIBS) \
> > -       $(SOUP_GNOME_LIBS) \
> >         $(CHECK_LIBS) \
> >         ../osinfo/libosinfo-1.0.la
> >  COMMON_CFLAGS = \
> > @@ -33,7 +32,6 @@ COMMON_CFLAGS = \
> >         $(COVERAGE_CFLAGS) \
> >         $(GOBJECT_CFLAGS) \
> >         $(SOUP_CFLAGS) \
> > -       $(SOUP_GNOME_CFLAGS) \
> >         -I$(top_srcdir) \
> >         -DSRCDIR="\"$(abs_top_srcdir)\"" \
> >         -DBUILDDIR="\"$(abs_top_builddir)\"" \
> > diff --git a/test/test-mediauris.c b/test/test-mediauris.c
> > index 191e8c6..8c3514d 100644
> > --- a/test/test-mediauris.c
> > +++ b/test/test-mediauris.c
> > @@ -26,8 +26,6 @@
> >  #include <osinfo/osinfo.h>
> >  #include <check.h>
> >  #include <libsoup/soup.h>
> > -#define HAVE_LIBSOUP_GNOME
> > -#include <libsoup/soup-gnome.h>
> >
> >  static void test_media(OsinfoMediaList *medialist, GError **error, SoupSession *session)
> >  {
> > @@ -69,10 +67,8 @@ START_TEST(test_uris)
> >      gchar *debugstr;
> >
> >      session = soup_session_async_new_with_options(
> > -#ifdef HAVE_LIBSOUP_GNOME
> >                                                    SOUP_SESSION_ADD_FEATURE_BY_TYPE,
> > -                                                  SOUP_TYPE_PROXY_RESOLVER_GNOME,
> > -#endif
> > +                                                  SOUP_TYPE_PROXY_RESOLVER_DEFAULT,
> >                                                    NULL);
> >      if ((debugstr = getenv("LIBOSINFO_TEST_DEBUG"))) {
> >          SoupLogger *logger;
> > diff --git a/test/test-treeuris.c b/test/test-treeuris.c
> > index 07f14bf..84ebc68 100644
> > --- a/test/test-treeuris.c
> > +++ b/test/test-treeuris.c
> > @@ -26,8 +26,6 @@
> >  #include <osinfo/osinfo.h>
> >  #include <check.h>
> >  #include <libsoup/soup.h>
> > -#define HAVE_LIBSOUP_GNOME
> > -#include <libsoup/soup-gnome.h>
> >
> >  static void test_tree(OsinfoTreeList *treelist, GError **error, SoupSession *session)
> >  {
> > @@ -69,10 +67,8 @@ START_TEST(test_uris)
> >      gchar *debugstr;
> >
> >      session = soup_session_async_new_with_options(
> > -#ifdef HAVE_LIBSOUP_GNOME
> >                                                    SOUP_SESSION_ADD_FEATURE_BY_TYPE,
> > -                                                  SOUP_TYPE_PROXY_RESOLVER_GNOME,
> > -#endif
> > +                                                  SOUP_TYPE_PROXY_RESOLVER_DEFAULT,
> >                                                    NULL);
> >      if ((debugstr = getenv("LIBOSINFO_TEST_DEBUG"))) {
> >          SoupLogger *logger;
> > --
> > 1.8.1.4
> >
> > _______________________________________________
> > Libosinfo mailing list
> > Libosinfo at redhat.com
> > https://www.redhat.com/mailman/listinfo/libosinfo
> 
> 
> 
> -- 
> Regards,
> 
> Zeeshan Ali (Khattak)
> FSF member#5124
> 
> _______________________________________________
> Libosinfo mailing list
> Libosinfo at redhat.com
> https://www.redhat.com/mailman/listinfo/libosinfo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libosinfo/attachments/20130325/9210eb70/attachment.sig>