snappy-hwe-snaps

Merge ~alfonsosanchezbeato/snappy-hwe-snaps/+git/modem-manager:fix-proxy-path into ~snappy-hwe-team/snappy-hwe-snaps/+git/modem-manager:libmbim/xenial/1.14.0

  1. Git
  2. lp:~alfonsosanchezbeato/snappy-hwe-snaps/+git/modem-manager
  3. fix-proxy-path
  4. Merge into libmbim/xenial/1.14.0
Proposed by Alfonso Sanchez-Beato
Status: Merged
Approved by: Simon Fels
Approved revision: 4b0b294a885ffdfda4001afa7f19983078a1a562
Merged at revision: 3a02f7541da63b1fc0861590d10203dcccbd9b48
Proposed branch: ~alfonsosanchezbeato/snappy-hwe-snaps/+git/modem-manager:fix-proxy-path
Merge into: ~snappy-hwe-team/snappy-hwe-snaps/+git/modem-manager:libmbim/xenial/1.14.0
Diff against target: 32 lines (+12/-1)
1 file modified
src/libmbim-glib/mbim-device.c (+12/-1)
Reviewer Review Type Date Requested Status
System Enablement Bot continuous-integration Needs Fixing
Simon Fels Approve
Matteo Croce (community) Approve
Konrad Zapałowicz (community) code Approve
Review via email: mp+313780@code.launchpad.net

Description of the change

Use snap path to start mbim-proxy

To post a comment you must log in.
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

FAILED: Continuous integration, rev:fe9613f01564583dbcb93173d271f837a726085c
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/542/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-enablement/job/generic-run-snap-spread-tests/279/console
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/450/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/542/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Simon Fels (morphis) :
review: Needs Fixing
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) :
Revision history for this message
Simon Fels (morphis) :
Revision history for this message
Konrad Zapałowicz (kzapalowicz) wrote :

lgtm

review: Approve (code)
Revision history for this message
Matteo Croce (teknoraver) :
review: Approve
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

@morphis great that you agree in removing that var. Pushed.

Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

FAILED: Continuous integration, rev:7e4d74274592707e59f0cf6f6a2f3061ec9737d1
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/550/
Executed test runs:
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/458/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/550/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

FAILED: Continuous integration, rev:4b0b294a885ffdfda4001afa7f19983078a1a562
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/552/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-enablement/job/generic-run-snap-spread-tests/288/console
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/460/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/552/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

FAILED: Continuous integration, rev:4b0b294a885ffdfda4001afa7f19983078a1a562
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/553/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-enablement/job/generic-run-snap-spread-tests/289/console
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/461/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/553/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Simon Fels (morphis) :
review: Approve
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

FAILED: Continuous integration, rev:4b0b294a885ffdfda4001afa7f19983078a1a562
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/591/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-enablement/job/generic-run-snap-spread-tests/319/console
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/499/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/591/rebuild

review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/src/libmbim-glib/mbim-device.c b/src/libmbim-glib/mbim-device.c
index a001940..97faa05 100644
--- a/src/libmbim-glib/mbim-device.c
+++ b/src/libmbim-glib/mbim-device.c
@@ -1100,6 +1100,8 @@ create_iochannel_with_socket (CreateIoChannelContext *ctx)
1100 if (!ctx->self->priv->socket_connection) {1100 if (!ctx->self->priv->socket_connection) {
1101 gchar **argc;1101 gchar **argc;
1102 GSource *source;1102 GSource *source;
1103 const gchar *snap_path;
1104 gchar *mbim_proxy_path;
11031105
1104 g_debug ("cannot connect to proxy: %s", error->message);1106 g_debug ("cannot connect to proxy: %s", error->message);
1105 g_clear_error (&error);1107 g_clear_error (&error);
@@ -1118,8 +1120,17 @@ create_iochannel_with_socket (CreateIoChannelContext *ctx)
11181120
1119 g_debug ("spawning new mbim-proxy (try %u)...", ctx->spawn_retries);1121 g_debug ("spawning new mbim-proxy (try %u)...", ctx->spawn_retries);
11201122
1123 mbim_proxy_path = g_malloc (PATH_MAX);
1124 snap_path = g_getenv ("SNAP");
1125 if (snap_path)
1126 g_snprintf(mbim_proxy_path, PATH_MAX, "%s%s/mbim-proxy",
1127 snap_path, LIBEXEC_PATH);
1128 else
1129 g_snprintf(mbim_proxy_path, PATH_MAX, "%s/mbim-proxy",
1130 LIBEXEC_PATH);
1131
1121 argc = g_new0 (gchar *, 2);1132 argc = g_new0 (gchar *, 2);
1122 argc[0] = g_strdup (LIBEXEC_PATH "/mbim-proxy");1133 argc[0] = mbim_proxy_path; /* Freed by g_strfreev */
1123 if (!g_spawn_async (NULL, /* working directory */1134 if (!g_spawn_async (NULL, /* working directory */
1124 argc,1135 argc,
1125 NULL, /* envp */1136 NULL, /* envp */

Subscribers

People subscribed via source and target branches