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 _______________________________________________ Libosinfo mailing list Libosinfo@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libosinfo
