[Libosinfo] [osinfo-db-tools PATCH v2 7/9] validate: Simplify validate_file_directory()

2019-07-03 Thread Fabiano Fidêncio
Signed-off-by: Fabiano Fidêncio --- tools/osinfo-db-validate.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/tools/osinfo-db-validate.c b/tools/osinfo-db-validate.c index 7fd87d2..3a1717f 100644 --- a/tools/osinfo-db-validate.c +++ b/tools/osinfo-db-validate.c @@ -1

[Libosinfo] [osinfo-db-tools PATCH v2 5/9] export: use g_clear_*() functions

2019-07-03 Thread Fabiano Fidêncio
Let's use g_clear_object() & g_clear_error() as those two functions avoid us checking whether the object is NULL or not and also sets the object to NULL after unrefing it. Signed-off-by: Fabiano Fidêncio --- tools/osinfo-db-export.c | 21 +++-- 1 file changed, 7 insertions(+), 14

[Libosinfo] [osinfo-db-tools PATCH v2 8/9] validate: Use g_clear_*() functions

2019-07-03 Thread Fabiano Fidêncio
Let's use g_clear_object() & g_clear_error() as those two functions avoid us checking whether the object is NULL or not and also sets the object to NULL after unrefing it. Signed-off-by: Fabiano Fidêncio --- tools/osinfo-db-validate.c | 11 --- 1 file changed, 4 insertions(+), 7 deletion

[Libosinfo] [osinfo-db-tools PATCH v2 3/9] import: Use g_clear_*() functions

2019-07-03 Thread Fabiano Fidêncio
Let's use g_clear_object() & g_clear_error() as those two functions avoid us checking whether the object is NULL or not and also sets the object to NULL after unrefing it. Signed-off-by: Fabiano Fidêncio --- tools/osinfo-db-import.c | 26 +- 1 file changed, 9 insertions(+

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

2019-07-03 Thread Fabiano Fidêncio
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/libosi

[Libosinfo] [osinfo-db-tools PATCH v2 9/9] util: Use g_clear_*() functions

2019-07-03 Thread Fabiano Fidêncio
Let's use g_clear_object() & g_clear_error() as those two functions avoid us checking whether the object is NULL or not and also sets the object to NULL after unrefing it. Signed-off-by: Fabiano Fidêncio --- tools/osinfo-db-util.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --g

[Libosinfo] [osinfo-db-tools PATCH v2 6/9] path: Use g_clear_*() functions

2019-07-03 Thread Fabiano Fidêncio
Let's use g_clear_object() & g_clear_error() as those two functions avoid us checking whether the object is NULL or not and also sets the object to NULL after unrefing it. Signed-off-by: Fabiano Fidêncio --- tools/osinfo-db-path.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --

[Libosinfo] [osinfo-db-tools PATCH v2 1/9] import: Use libsoup for http:// & https:// requests

2019-07-03 Thread Fabiano Fidêncio
As osinfo-db-import handles non-local requests, it'd end up relying on GVFS under the hood, which would cause the app to not work when running as root. In order to avoid this situation, let's rely on libsoup for this case. It's important to mention that glib minimum version has been bumped to 2.4

[Libosinfo] [osinfo-db-tools PATCH v2 2/9] spec: Drop gvfs dependency

2019-07-03 Thread Fabiano Fidêncio
As we rely on libsoup to handle http://, https://, and ftp:// requests there's no need to depend on gvfs anymore. https://gitlab.com/libosinfo/libosinfo/issues/30 Signed-off-by: Fabiano Fidêncio --- osinfo-db-tools.spec.in | 1 - 1 file changed, 1 deletion(-) diff --git a/osinfo-db-tools.spec.

[Libosinfo] [osinfo-db-tools PATCH v2 4/9] export: Simplify osinfo_db_export_create_dir()

2019-07-03 Thread Fabiano Fidêncio
Signed-off-by: Fabiano Fidêncio --- tools/osinfo-db-export.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/tools/osinfo-db-export.c b/tools/osinfo-db-export.c index eef6688..5bf61fe 100644 --- a/tools/osinfo-db-export.c +++ b/tools/osinfo-db-export.c @@ -10