[Libosinfo] [osinfo-db-tools PATCH v2 0/9] Drop GVFS dependency in favour of libsoup (+ some cleanups)

Fabiano Fidêncio fidencio at redhat.com
Wed Jul 3 10:14:01 UTC 2019


Let's rely on libsoup in order to to http://, https://, and ftp://
requests.

By doing this, GVFS (more specifically, gvfs-http) won't be dragged as
a dependency, which caused errors for running osinfo-db-import as root
(as GVFS relies on being run in the user's session).

https://gitlab.com/libosinfo/libosinfo/issues/30

The cleanups are basically in order to use g_clear_*() functions.

Changes since v1:
https://www.redhat.com/archives/libosinfo/2019-June/msg00129.html
- Adjusted commit message to only mention support for http:// and
  https://;
- Added to the manpage that only http:// and https:// protocols are
  supported for non-local files;
- Used the same requires_soup() logic used in libosinfo here, just
  for the sake of consistency;

Fabiano Fidêncio (9):
  import: Use libsoup for http:// & https:// requests
  spec: Drop gvfs dependency
  import: Use g_clear_*() functions
  export: Simplify osinfo_db_export_create_dir()
  export: use g_clear_*() functions
  path: Use g_clear_*() functions
  validate: Simplify validate_file_directory()
  validate: Use g_clear_*() functions
  util: Use g_clear_*() functions

 configure.ac               |   5 +-
 osinfo-db-tools.spec.in    |   2 +-
 tools/Makefile.am          |   2 +
 tools/osinfo-db-export.c   |  38 ++++-----
 tools/osinfo-db-import.c   | 163 ++++++++++++++++++++++++++-----------
 tools/osinfo-db-path.c     |   4 +-
 tools/osinfo-db-util.c     |   3 +-
 tools/osinfo-db-validate.c |  20 ++---
 8 files changed, 148 insertions(+), 89 deletions(-)

-- 
2.21.0




More information about the Libosinfo mailing list