Re: [libosinfo PATCH 0/2] Switch from libsoup to libcurl

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Finally pushed this series.

Christophe

On Mon, Aug 07, 2017 at 06:18:33PM +0200, Pino Toscano wrote:
> Hi,
> 
> currently, libosinfo requires [1] libsoup for two tests, to check the
> validity of all the URLs in osinfo-db; while simple to use, libsoup is
> HTTP only, and thus cannot be used to validate other protocols used in
> osinfo-db (like ftp).
> 
> This series proposes the switch from libsoup to libcurl -- it has the
> following advantages:
> - ftp (and other) URLs can be tested
> - slightly more flexible than libsoup
> - stable library, widely available, portable, and with a liberal
>   license (MIT)
> 
> [1] even if just for two tests, not for the main library nor the tools
>     -- maybe a followup patch could be loosen the requirement, and
>     conditionalize the build of those tests only when libsoup/libcurl
>     is present
> 
> Thanks,
> 
> Pino Toscano (2):
>   Switch from libsoup to libcurl
>   Revert "tests: test-mediauris: skip non-http URLs"
> 
>  README                  |  2 +-
>  configure.ac            | 12 +++---------
>  libosinfo.spec.in       |  2 +-
>  mingw-libosinfo.spec.in |  4 ++--
>  tests/Makefile.am       |  4 ++--
>  tests/test-mediauris.c  | 42 +++++++++++++++++++++++++-----------------
>  tests/test-treeuris.c   | 39 ++++++++++++++++++++++++---------------
>  7 files changed, 58 insertions(+), 47 deletions(-)
> 
> -- 
> 2.13.4
> 
> _______________________________________________
> Libosinfo mailing list
> Libosinfo@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libosinfo

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux