On Fri, Dec 7, 2018 at 3:23 PM Felipe Borges <felipe10borges@xxxxxxxxx> wrote:
>
> Hi, thanks for working on this!
>
> On Fri, Dec 7, 2018 at 2:34 PM Fabiano Fidêncio <fidencio@xxxxxxxxxx> wrote:
> >
> > Signed-off-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx>
> > ---
> > tools/osinfo-db-import.c | 90 ++++++++++++++++++++++++++++++++++++++--
> > 1 file changed, 86 insertions(+), 4 deletions(-)
> >
> > diff --git a/tools/osinfo-db-import.c b/tools/osinfo-db-import.c
> > index 72f21ba..e4b7824 100644
> > --- a/tools/osinfo-db-import.c
> > +++ b/tools/osinfo-db-import.c
> > @@ -134,6 +134,55 @@ static GFile *osinfo_db_import_get_file(GFile *target,
> > return g_file_resolve_relative_path(target, tmp);
> > }
> >
> > +static int
> > +osinfo_db_import_download_file(const gchar *url,
> > + gchar **source_file)
> > +{
> > + GFile *in = NULL;
> > + GFile *out = NULL;
> > + GError *err = NULL;
> > + gchar *filename = NULL;
> > + GFileCopyFlags flags = G_FILE_COPY_OVERWRITE | G_FILE_COPY_BACKUP;
> > + int ret = -1;
> > +
> > + in = g_file_new_for_uri(url);
> > + if (in == NULL)
> > + return -1;
> > +
> > + filename = g_file_get_basename(in);
> > + if (filename == NULL)
> > + goto cleanup;
> > +
> > + *source_file = g_strdup_printf("%s/%s", g_get_user_cache_dir(), filename);
>
> Any reason for not using "g_build_filename" here?
No specific reason, I just didn't remember of its existence :-)
I've changed it locally, thanks!
_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo
