[gnome-builder/gnome-builder-3-40] flatpak: make sure we have proper argv pointer
- From: Christian Hergert <chergert src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-builder/gnome-builder-3-40] flatpak: make sure we have proper argv pointer
- Date: Wed, 5 May 2021 04:18:32 +0000 (UTC)
commit 05cccea151c11e82a833812b27e63d3329a87d39
Author: Christian Hergert <chergert redhat com>
Date: Tue May 4 21:18:01 2021 -0700
flatpak: make sure we have proper argv pointer
src/plugins/flatpak/gbp-flatpak-subprocess-launcher.c | 17 +++++++++++++----
1 file changed, 13 insertions(+), 4 deletions(-)
---
diff --git a/src/plugins/flatpak/gbp-flatpak-subprocess-launcher.c
b/src/plugins/flatpak/gbp-flatpak-subprocess-launcher.c
index 791cf4c5e..b6dcd9db2 100644
--- a/src/plugins/flatpak/gbp-flatpak-subprocess-launcher.c
+++ b/src/plugins/flatpak/gbp-flatpak-subprocess-launcher.c
@@ -31,6 +31,15 @@ struct _GbpFlatpakSubprocessLauncher
G_DEFINE_TYPE (GbpFlatpakSubprocessLauncher, gbp_flatpak_subprocess_launcher, IDE_TYPE_SUBPROCESS_LAUNCHER)
+static const char * const *
+insert_argv (IdeSubprocessLauncher *launcher,
+ int argpos,
+ const char *xdg_runtime_dir)
+{
+ ide_subprocess_launcher_insert_argv (launcher, argpos, xdg_runtime_dir);
+ return ide_subprocess_launcher_get_argv (launcher);
+}
+
static IdeSubprocess *
gbp_flatpak_subprocess_launcher_spawn (IdeSubprocessLauncher *launcher,
GCancellable *cancellable,
@@ -122,12 +131,13 @@ gbp_flatpak_subprocess_launcher_spawn (IdeSubprocessLauncher *launcher,
* or fourth) time.
*/
if (!g_strv_contains (argv, build_dir_option))
- ide_subprocess_launcher_insert_argv (launcher, argpos, build_dir_option);
+ argv = insert_argv (launcher, argpos, build_dir_option);
if (!g_strv_contains (argv, xdg_runtime_dir))
- ide_subprocess_launcher_insert_argv (launcher, argpos, xdg_runtime_dir);
+ argv = insert_argv (launcher, argpos, xdg_runtime_dir);
apply_env:
+ argv = ide_subprocess_launcher_get_argv (launcher);
envp = ide_subprocess_launcher_get_environ (launcher);
if (envp != NULL)
@@ -139,9 +149,8 @@ apply_env:
for (guint i = 0; envp[i] != NULL; i++)
{
g_autofree gchar *arg = g_strdup_printf ("--env=%s", envp[i]);
- argv = ide_subprocess_launcher_get_argv (launcher);
if (!g_strv_contains (argv, arg))
- ide_subprocess_launcher_insert_argv (launcher, argpos, arg);
+ argv = insert_argv (launcher, argpos, arg);
}
ide_subprocess_launcher_setenv (launcher, "PATH", NULL, TRUE);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]