Hello community, here is the log from the commit of package ModemManager for openSUSE:Factory checked in at 2016-08-24 10:06:12 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/ModemManager (Old) and /work/SRC/openSUSE:Factory/.ModemManager.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "ModemManager" Changes: -------- --- /work/SRC/openSUSE:Factory/ModemManager/ModemManager.changes 2016-05-31 12:09:04.000000000 +0200 +++ /work/SRC/openSUSE:Factory/.ModemManager.new/ModemManager.changes 2016-08-24 10:06:14.000000000 +0200 @@ -1,0 +2,98 @@ +Thu Aug 18 14:55:57 UTC 2016 - zai...@opensuse.org + +- Add ModemManager-core-use-MM-specific-logging-methods.patch:core: + use MM-specific logging methods always instead of the generic + GLib ones. Patch from upstream stable git. + +------------------------------------------------------------------- +Mon Aug 15 09:29:09 UTC 2016 - br...@ioda-net.ch + +- Update to version 1.6.0 (boo#975211): + + New features: + - Core: + . Implemented support for suspend/resume detection, currently + working when systemd is in use. Whenever the system is + suspended, we'll flag the modems as invalid so that they + are re-probed from scratch when the system is resumed. + . Added cancellation support for the probing operations. + . Reworked and simplified the serial port response + processing. + - Location interface: + . Added A-GPS support, currently available only for QMI based + modems with PDS service. + . Added support for updating the default GPS refresh time. + - Time interface: New default implementation for all AT-based + modems. + - Voice interface: New DBus interface to allow the management + of voice calls, which currently assumes that the audio + channel is setup out of ModemManager. + - Bearer: New support for reporting statistics of the ongoing + connection with a new 'Stats' property, currently available + for QMI and MBIM based modems. + - QMI: + . Implemented support for devices which only work in "raw IP" + mode, like the Sierra MC7455. + . Implemented support for SIM related operations using the + UIM service, as newer modems with multi-SIM capabilities + don't support the legacy DMS UIM operations. + . Implemented support for detecting network-initiated + disconnections. + - MBIM: If online mode fails, try to use the 'DMS Set FCC + Authentication' QMI + - udev: Added new supported 'ID_MM_PORT_IGNORE' tag to allow + fully ignoring ports specified by the user. + - mmcli: + . Added command completion. + . Added new operations to use the Voice interface. + - Build: Added code coverage support. + - Plugins: + . haier: new plugin to support the Haier CE81B. + . thuraya: new plugin for Thuraya satellite modems. + . sierra-legacy,sierra: the implementation for Sierra modems + is now split into two different plugins: a 'legacy' one for + the old PPP and DirectIP based modems and the standard one + for the newer QMI and MBIM based ones. + . dell: new plugin for Dell rebranded devices from Novatel, + Sierra or Ericsson. + . gobi: removed the plugin. All non-vendor specific QMI + devices should now be managed by the generic plugin. + . mbm: dynamically load the list of supported modes. + . mbm: fixed several connection/disconnection issues. + . simtech: support QMI devices. + . huawei: implemented Voice call management support. + . huawei: use static IP addressing in NDISDUP capable devices + if the AT^DHCP response provides the IP details. + + Existing features backported to 1.4.x releases also present in + in ModemManager 1.6.0: + - MBIM: + . The mbim-proxy is used by default. + . Implemented support for disconnection status notification + while connected. + . Disabled CDMA capabilities, until properly supported. + - QMI: + . The qmi-proxy is used by default. + . If online mode fails, use 'DMS Set FCC Authentication', + required by some rebranded Sierra modems (e.g. Dell branded + ones). + . Implemented support for loading SIM operator id and name. + . Implemented power-cycle reset functionality. + - Plugins: + . telit: + . added support for new devices, like HE910, UE910 and + . implemented dynamic port identification. + . implemented unlock retries loading. + . implemented supported/current bands management. + . implemented supported/current modes management. + . implemented modem reset and power down. + . mbm: implemented GPS support for Ericsson HS2350 and + H5321gw modems. +- Drop upstream fixed patches: + + ModemManager-cinterion_drop_unused_constants.patch. + + ModemManager-novatel_remove_unused_custom_AT_probe_array.patch. + + ModemManager-wmc_fix_typo.patch. + + ModemManager-fix-whitespaces.patch. +- Added nmcli-bash-completion package. +- Need last version of polkit-default-priv (bsc#976945). +- build '--with-suspend-resume=systemd' argument. + +------------------------------------------------------------------- Old: ---- ModemManager-1.4.14.tar.xz ModemManager-cinterion_drop_unused_constants.patch ModemManager-fix-whitespaces.patch ModemManager-novatel_remove_unused_custom_AT_probe_array.patch ModemManager-wmc_fix_typo.patch New: ---- ModemManager-1.6.0.tar.xz ModemManager-core-use-MM-specific-logging-methods.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ ModemManager.spec ++++++ --- /var/tmp/diff_new_pack.Vv0cEt/_old 2016-08-24 10:06:15.000000000 +0200 +++ /var/tmp/diff_new_pack.Vv0cEt/_new 2016-08-24 10:06:15.000000000 +0200 @@ -18,7 +18,7 @@ %define _udevdir %(pkg-config --variable udevdir udev) Name: ModemManager -Version: 1.4.14 +Version: 1.6.0 Release: 0 Summary: DBus interface for modem handling License: GPL-2.0+ and GPL-3.0+ @@ -27,15 +27,9 @@ Source0: http://www.freedesktop.org/software/%{name}/%{name}-%{version}.tar.xz # PATCH-FIX-OPENSUSE ModemManager-1.0.0-systemd-activation.patch bnc#892682 lnus...@suse.de -- Fix systemd dbus activation Patch0: ModemManager-1.0.0-systemd-activation.patch -# PATCH-FIX-UPSTREAM ModemManager-fix-whitespaces.patch dims...@opensuse.org -- Fix whitespace usage, allows building with gcc6 -Patch1: ModemManager-fix-whitespaces.patch -# PATCH-FIX-UPSTREAM ModemManager-cinterion_drop_unused_constants.patch dims...@opensuse.org -- cinterion: drop unused constants (fix gcc6 build) -Patch2: ModemManager-cinterion_drop_unused_constants.patch -# PATCH-FIX-UPSTREAM ModemManager-novatel_remove_unused_custom_AT_probe_array.patch dims...@opensuse.org -- novatel: remove unused custom AT probe array (fix gcc6 build) -Patch3: ModemManager-novatel_remove_unused_custom_AT_probe_array.patch -# PATCH-FIX-UPSTREAM ModemManager-wmc_fix_typo.patch dims...@opensuse.org -- wmc: fix typo (fix gcc6 build) -Patch4: ModemManager-wmc_fix_typo.patch - +# PATCH-FIX-UPSTREAM ModemManager-core-use-MM-specific-logging-methods.patch zai...@opensuse.org -- Use MM specific logging. +Patch1: ModemManager-core-use-MM-specific-logging-methods.patch +BuildRequires: automake BuildRequires: dbus-1-glib-devel BuildRequires: gobject-introspection-devel >= 0.9.6 BuildRequires: hicolor-icon-theme @@ -44,9 +38,9 @@ BuildRequires: ppp-devel BuildRequires: vala-devel >= 0.18 BuildRequires: pkgconfig(gudev-1.0) >= 147 -BuildRequires: pkgconfig(mbim-glib) >= 1.10 +BuildRequires: pkgconfig(mbim-glib) >= 1.14 BuildRequires: pkgconfig(polkit-gobject-1) >= 0.97 -BuildRequires: pkgconfig(qmi-glib) >= 1.12.4 +BuildRequires: pkgconfig(qmi-glib) >= 1.16 BuildRequires: pkgconfig(systemd) BuildRequires: pkgconfig(udev) Requires: hicolor-icon-theme @@ -92,13 +86,20 @@ %lang_package +%package -n %{name}-bash-completion +Summary: Bash completion for nmcli +Group: Productivity/Networking/System +Supplements: packageand(%{name}:bash-completion) +BuildRequires: bash-completion +Requires: bash-completion + +%description -n %{name}-bash-completion +This package contain de bash completion command for nmcli tools. + %prep %setup -q %patch0 -p1 %patch1 -p1 -%patch2 -p1 -%patch3 -p1 -%patch4 -p1 %build pppddir=`ls -1d %{_libdir}/pppd/2*` @@ -106,6 +107,7 @@ %configure \ --disable-static \ --with-polkit \ + --with-suspend-resume=systemd \ --with-udev-base-dir=%{_udevdir} --with-qmi \ --with-mbim make %{?_smp_mflags} @@ -175,4 +177,7 @@ %{_libdir}/pkgconfig/ModemManager.pc %{_libdir}/pkgconfig/mm-glib.pc +%files -n %{name}-bash-completion +%{_datadir}/bash-completion/completions/mmcli + %changelog ++++++ ModemManager-1.4.14.tar.xz -> ModemManager-1.6.0.tar.xz ++++++ ++++ 106973 lines of diff (skipped) ++++++ ModemManager-core-use-MM-specific-logging-methods.patch ++++++ >From e6b99c5c1a81bb3347f76eba3b8621cf7fd78297 Mon Sep 17 00:00:00 2001 From: Aleksander Morgado <aleksan...@aleksander.es> Date: Mon, 15 Aug 2016 13:11:22 +0200 Subject: core: use MM-specific logging methods always instead of the generic GLib ones (cherry picked from commit 7fb366c41a7b62723df6ab67b611510216c14b0a) diff --git a/plugins/huawei/mm-broadband-modem-huawei.c b/plugins/huawei/mm-broadband-modem-huawei.c index c650901..fe6c217 100644 --- a/plugins/huawei/mm-broadband-modem-huawei.c +++ b/plugins/huawei/mm-broadband-modem-huawei.c @@ -1008,7 +1008,7 @@ syscfg_test_ready (MMBroadbandModemHuawei *self, combinations, (GDestroyNotify)g_array_unref); } else { - g_debug ("Error while checking ^SYSCFG format: %s", error->message); + mm_dbg ("Error while checking ^SYSCFG format: %s", error->message); /* If SIM-PIN error, don't mark as feature unsupported; we'll retry later */ if (!g_error_matches (error, MM_MOBILE_EQUIPMENT_ERROR, @@ -1066,7 +1066,7 @@ syscfgex_test_ready (MMBroadbandModemHuawei *self, /* If SIM-PIN error, don't mark as feature unsupported; we'll retry later */ if (error) { - g_debug ("Error while checking ^SYSCFGEX format: %s", error->message); + mm_dbg ("Error while checking ^SYSCFGEX format: %s", error->message); if (g_error_matches (error, MM_MOBILE_EQUIPMENT_ERROR, MM_MOBILE_EQUIPMENT_ERROR_SIM_PIN)) { @@ -1127,7 +1127,7 @@ prefmode_test_ready (MMBroadbandModemHuawei *self, combinations, (GDestroyNotify)g_array_unref); } else { - g_debug ("Error while checking ^PREFMODE format: %s", error->message); + mm_dbg ("Error while checking ^PREFMODE format: %s", error->message); /* If SIM-PIN error, don't mark as feature unsupported; we'll retry later */ if (!g_error_matches (error, MM_MOBILE_EQUIPMENT_ERROR, @@ -2163,7 +2163,7 @@ peek_port_at_for_data (MMBroadbandModemHuawei *self, g_warn_if_fail (mm_port_get_subsys (port) == MM_PORT_SUBSYS_NET); net_port_parent_path = mm_port_get_parent_path (port); if (!net_port_parent_path) { - g_warning ("(%s) no parent path for net port", mm_port_get_device (port)); + mm_warn ("(%s) no parent path for net port", mm_port_get_device (port)); return NULL; } diff --git a/plugins/mbm/mm-modem-helpers-mbm.c b/plugins/mbm/mm-modem-helpers-mbm.c index 55557f2..d7c2a58 100644 --- a/plugins/mbm/mm-modem-helpers-mbm.c +++ b/plugins/mbm/mm-modem-helpers-mbm.c @@ -25,6 +25,7 @@ #define _LIBMM_INSIDE_MM #include <libmm-glib.h> +#include "mm-log.h" #include "mm-modem-helpers.h" #include "mm-modem-helpers-mbm.h" @@ -175,7 +176,7 @@ add_supported_mode (guint32 *mask, g_assert (mask); if (mode >= 32) - g_warning ("Ignored unexpected mode in +CFUN match: %d", mode); + mm_warn ("Ignored unexpected mode in +CFUN match: %d", mode); else *mask |= (1 << mode); } @@ -236,18 +237,18 @@ mm_mbm_parse_cfun_test (const gchar *response, last_str = separator + 1; if (!mm_get_uint_from_str (first_str, &first)) - g_warning ("Couldn't match range start: '%s'", first_str); + mm_warn ("Couldn't match range start: '%s'", first_str); else if (!mm_get_uint_from_str (last_str, &last)) - g_warning ("Couldn't match range stop: '%s'", last_str); + mm_warn ("Couldn't match range stop: '%s'", last_str); else if (first >= last) - g_warning ("Couldn't match range: wrong first '%s' and last '%s' items", first_str, last_str); + mm_warn ("Couldn't match range: wrong first '%s' and last '%s' items", first_str, last_str); else { for (mode = first; mode <= last; mode++) add_supported_mode (&mask, mode); } } else { if (!mm_get_uint_from_str (supported_modes[i], &mode)) - g_warning ("Couldn't match mode: '%s'", supported_modes[i]); + mm_warn ("Couldn't match mode: '%s'", supported_modes[i]); else add_supported_mode (&mask, mode); } diff --git a/src/mm-base-bearer.c b/src/mm-base-bearer.c index f304b2a..12b1d53 100644 --- a/src/mm-base-bearer.c +++ b/src/mm-base-bearer.c @@ -175,7 +175,7 @@ reload_stats_ready (MMBaseBearer *self, guint64 tx_bytes = 0; if (!MM_BASE_BEARER_GET_CLASS (self)->reload_stats_finish (self, &rx_bytes, &tx_bytes, res, &error)) { - g_warning ("Reloading stats failed: %s", error->message); + mm_warn ("Reloading stats failed: %s", error->message); g_error_free (error); return; } diff --git a/src/mm-broadband-modem-mbim.c b/src/mm-broadband-modem-mbim.c index f1ddd6c..d68d7cb 100644 --- a/src/mm-broadband-modem-mbim.c +++ b/src/mm-broadband-modem-mbim.c @@ -1015,7 +1015,7 @@ release_qmi_client_dms_ready (QmiDevice *dev, /* Non-fatal error */ if (!qmi_device_release_client_finish (dev, res, &error)) { - g_debug ("error: couldn't release client: %s", error->message); + mm_dbg ("error: couldn't release client: %s", error->message); g_error_free (error); } @@ -1045,7 +1045,7 @@ set_fcc_authentication_ready (QmiClientDms *client, output = qmi_client_dms_set_fcc_authentication_finish (client, res, &error); if (!output || !qmi_message_dms_set_fcc_authentication_output_get_result (output, &error)) { - g_debug ("error: couldn't set FCC auth: %s", error->message); + mm_dbg ("error: couldn't set FCC auth: %s", error->message); g_error_free (error); g_assert (ctx->saved_error); g_simple_async_result_take_error (ctx->result, ctx->saved_error); @@ -1082,7 +1082,7 @@ qmi_client_dms_ready (QmiDevice *dev, ctx->qmi_client = qmi_device_allocate_client_finish (dev, res, &error); if (!ctx->qmi_client) { - g_debug ("error: couldn't create DMS client: %s", error->message); + mm_dbg ("error: couldn't create DMS client: %s", error->message); g_error_free (error); g_assert (ctx->saved_error); g_simple_async_result_take_error (ctx->result, ctx->saved_error); @@ -1116,7 +1116,7 @@ device_open_ready (QmiDevice *dev, GError *error = NULL; if (!qmi_device_open_finish (dev, res, &error)) { - g_debug ("error: couldn't open QmiDevice: %s", error->message); + mm_dbg ("error: couldn't open QmiDevice: %s", error->message); g_error_free (error); g_assert (ctx->saved_error); g_simple_async_result_take_error (ctx->result, ctx->saved_error); @@ -1151,7 +1151,7 @@ qmi_device_new_ready (GObject *unused, ctx->qmi_device = qmi_device_new_finish (res, &error); if (!ctx->qmi_device) { - g_debug ("error: couldn't create QmiDevice: %s", error->message); + mm_dbg ("error: couldn't create QmiDevice: %s", error->message); g_error_free (error); g_assert (ctx->saved_error); g_simple_async_result_take_error (ctx->result, ctx->saved_error); @@ -1221,7 +1221,7 @@ radio_state_set_up_ready (MbimDevice *device, /* Only the first attempt isn't fatal */ if (ctx->step == POWER_UP_CONTEXT_STEP_FIRST) { /* Warn and keep, will retry */ - g_warning ("%s", error->message); + mm_warn ("%s", error->message); g_assert (!ctx->saved_error); ctx->saved_error = error; ctx->step++; diff --git a/src/mm-broadband-modem-qmi.c b/src/mm-broadband-modem-qmi.c index b57499c..bc13925 100644 --- a/src/mm-broadband-modem-qmi.c +++ b/src/mm-broadband-modem-qmi.c @@ -1238,7 +1238,7 @@ dms_get_ids_ready (QmiClientDms *client, else if (len == 8) ctx->self->priv->esn = g_strdup (str); else - g_debug ("Invalid ESN reported: '%s' (unexpected length)", str); + mm_dbg ("Invalid ESN reported: '%s' (unexpected length)", str); } if (qmi_message_dms_get_ids_output_get_meid (output, &str, NULL) && @@ -1248,7 +1248,7 @@ dms_get_ids_ready (QmiClientDms *client, if (len == 14) ctx->self->priv->meid = g_strdup (str); else - g_debug ("Invalid MEID reported: '%s' (unexpected length)", str); + mm_dbg ("Invalid MEID reported: '%s' (unexpected length)", str); } if (ctx->self->priv->imei) @@ -1504,7 +1504,7 @@ uim_get_card_status_output_parse (QmiMessageUimGetCardStatusOutput *output, } if (cards->len > 1) - g_debug ("Multiple cards reported: %u", cards->len); + mm_dbg ("Multiple cards reported: %u", cards->len); /* All KNOWN applications in all cards will need to be in READY state for us * to consider UNLOCKED */ @@ -1517,25 +1517,25 @@ uim_get_card_status_output_parse (QmiMessageUimGetCardStatusOutput *output, gboolean sim_usim_found = FALSE; if (card->applications->len == 0) { - g_debug ("No applications reported in card [%u]", i); + mm_dbg ("No applications reported in card [%u]", i); n_invalid++; break; } if (card->applications->len > 1) - g_debug ("Multiple applications reported in card [%u]: %u", i, card->applications->len); + mm_dbg ("Multiple applications reported in card [%u]: %u", i, card->applications->len); for (j = 0; j < card->applications->len; j++) { app = &g_array_index (card->applications, QmiMessageUimGetCardStatusOutputCardStatusCardsElementApplicationsElement, j); if (app->type == QMI_UIM_CARD_APPLICATION_TYPE_UNKNOWN) { - g_debug ("Unknown application [%u] found in card [%u]: %s. Ignored.", - j, i, qmi_uim_card_application_state_get_string (app->state)); + mm_dbg ("Unknown application [%u] found in card [%u]: %s. Ignored.", + j, i, qmi_uim_card_application_state_get_string (app->state)); continue; } - g_debug ("Application '%s' [%u] in card [%u]: %s", - qmi_uim_card_application_type_get_string (app->type), j, i, qmi_uim_card_application_state_get_string (app->state)); + mm_dbg ("Application '%s' [%u] in card [%u]: %s", + qmi_uim_card_application_type_get_string (app->type), j, i, qmi_uim_card_application_state_get_string (app->state)); if (app->type == QMI_UIM_CARD_APPLICATION_TYPE_SIM || app->type == QMI_UIM_CARD_APPLICATION_TYPE_USIM) { /* We found the card/app pair to use! Only keep the first found, @@ -1550,7 +1550,7 @@ uim_get_card_status_output_parse (QmiMessageUimGetCardStatusOutput *output, } if (!sim_usim_found) { - g_debug ("No SIM/USIM application found in card [%u]", i); + mm_dbg ("No SIM/USIM application found in card [%u]", i); n_invalid++; } @@ -1558,7 +1558,7 @@ uim_get_card_status_output_parse (QmiMessageUimGetCardStatusOutput *output, } case QMI_UIM_CARD_STATE_ABSENT: - g_debug ("Card '%u' is absent", i); + mm_dbg ("Card '%u' is absent", i); n_absent++; break; @@ -1566,9 +1566,9 @@ uim_get_card_status_output_parse (QmiMessageUimGetCardStatusOutput *output, default: n_error++; if (qmi_uim_card_error_get_string (card->error_code) != NULL) - g_warning ("Card '%u' is unusable: %s", i, qmi_uim_card_error_get_string (card->error_code)); + mm_warn ("Card '%u' is unusable: %s", i, qmi_uim_card_error_get_string (card->error_code)); else - g_warning ("Card '%u' is unusable: unknown error", i); + mm_warn ("Card '%u' is unusable: unknown error", i); break; } @@ -1608,7 +1608,7 @@ uim_get_card_status_output_parse (QmiMessageUimGetCardStatusOutput *output, app->state != QMI_UIM_CARD_APPLICATION_STATE_PIN1_OR_UPIN_PIN_REQUIRED && app->state != QMI_UIM_CARD_APPLICATION_STATE_PUK1_OR_UPIN_PUK_REQUIRED && app->state != QMI_UIM_CARD_APPLICATION_STATE_PIN1_BLOCKED) { - g_debug ("Neither SIM nor USIM are ready"); + mm_dbg ("Neither SIM nor USIM are ready"); g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_RETRY, "SIM not ready yet (retry)"); return FALSE; @@ -1669,7 +1669,7 @@ uim_get_card_status_output_parse (QmiMessageUimGetCardStatusOutput *output, break; case QMI_UIM_PIN_STATE_PERMANENTLY_BLOCKED: - g_warning ("PUK2 permanently blocked"); + mm_warn ("PUK2 permanently blocked"); case QMI_UIM_PIN_STATE_BLOCKED: lock = MM_MODEM_LOCK_SIM_PUK2; break; @@ -1679,7 +1679,7 @@ uim_get_card_status_output_parse (QmiMessageUimGetCardStatusOutput *output, break; default: - g_warning ("Unknown SIM PIN2/PUK2 status"); + mm_warn ("Unknown SIM PIN2/PUK2 status"); break; } } @@ -7691,7 +7691,7 @@ wms_list_messages_ready (QmiClientWms *client, if (!qmi_message_wms_list_messages_output_get_result (output, &error)) { /* Ignore error, keep on */ - g_debug ("Couldn't read SMS messages: %s", error->message); + mm_dbg ("Couldn't read SMS messages: %s", error->message); g_error_free (error); ctx->step++; load_initial_sms_parts_step (ctx); @@ -9730,7 +9730,7 @@ oma_event_report_indication_cb (QmiClientNas *client, session_type = mm_oma_session_type_from_qmi_oma_session_type (network_initiated_alert_session_type); if (session_type == MM_OMA_SESSION_TYPE_UNKNOWN) - g_warning ("Unknown QMI OMA session type '%u'", network_initiated_alert_session_type); + mm_warn ("Unknown QMI OMA session type '%u'", network_initiated_alert_session_type); else mm_iface_modem_oma_add_pending_network_initiated_session ( MM_IFACE_MODEM_OMA (self), diff --git a/src/mm-charsets.c b/src/mm-charsets.c index ffdcad5..191789d 100644 --- a/src/mm-charsets.c +++ b/src/mm-charsets.c @@ -24,6 +24,7 @@ #include <libmm-glib.h> #include "mm-charsets.h" +#include "mm-log.h" typedef struct { const char *gsm_name; @@ -130,9 +131,8 @@ mm_modem_charset_byte_array_append (GByteArray *array, converted = g_convert (utf8, -1, iconv_to, "UTF-8", NULL, &written, &error); if (!converted) { if (error) { - g_warning ("%s: failed to convert '%s' to %s character set: (%d) %s", - __func__, utf8, iconv_to, - error->code, error->message); + mm_warn ("failed to convert '%s' to %s character set: (%d) %s", + utf8, iconv_to, error->code, error->message); g_error_free (error); } return FALSE; diff --git a/src/mm-error-helpers.c b/src/mm-error-helpers.c index e922cbb..a6b21b7 100644 --- a/src/mm-error-helpers.c +++ b/src/mm-error-helpers.c @@ -16,6 +16,7 @@ */ #include "mm-error-helpers.h" +#include "mm-log.h" #include <ctype.h> @@ -50,7 +51,7 @@ mm_connection_error_for_code (MMConnectionError code) break; default: - g_debug ("Invalid connection error code: %u", code); + mm_dbg ("Invalid connection error code: %u", code); /* uhm... make something up (yes, ok, lie!). */ code = MM_CONNECTION_ERROR_NO_CARRIER; msg = "No carrier"; @@ -127,7 +128,7 @@ mm_mobile_equipment_error_for_code (MMMobileEquipmentError code) } /* Not found? Then, default */ - g_debug ("Invalid mobile equipment error code: %u", (guint)code); + mm_dbg ("Invalid mobile equipment error code: %u", (guint)code); return g_error_new (MM_MOBILE_EQUIPMENT_ERROR, MM_MOBILE_EQUIPMENT_ERROR_UNKNOWN, "Unknown error"); @@ -163,8 +164,7 @@ mm_mobile_equipment_error_for_string (const gchar *str) /* Not found? Then, default */ if (!msg) { - g_debug ("Invalid mobile equipment error string: '%s' (%s)", - str, buf); + mm_dbg ("Invalid mobile equipment error string: '%s' (%s)", str, buf); code = MM_MOBILE_EQUIPMENT_ERROR_UNKNOWN; msg = "Unknown error"; } @@ -215,7 +215,7 @@ mm_message_error_for_code (MMMessageError code) } /* Not found? Then, default */ - g_debug ("Invalid message error code: %u", (guint)code); + mm_dbg ("Invalid message error code: %u", (guint)code); return g_error_new (MM_MESSAGE_ERROR, MM_MESSAGE_ERROR_UNKNOWN, "Unknown error"); @@ -251,8 +251,7 @@ mm_message_error_for_string (const gchar *str) /* Not found? Then, default */ if (!msg) { - g_debug ("Invalid message error string: '%s' (%s)", - str, buf); + mm_dbg ("Invalid message error string: '%s' (%s)", str, buf); code = MM_MESSAGE_ERROR_UNKNOWN; msg = "Unknown error"; } diff --git a/src/mm-iface-modem.c b/src/mm-iface-modem.c index f0d826c..db7afc8 100644 --- a/src/mm-iface-modem.c +++ b/src/mm-iface-modem.c @@ -3371,7 +3371,7 @@ set_power_state_load_ready (MMIfaceModem *self, ctx->previous_real_power_state = MM_IFACE_MODEM_GET_INTERFACE (self)->load_power_state_finish (self, res, &error); if (error) { - g_debug ("Couldn't reload current power state: %s", error->message); + mm_dbg ("Couldn't reload current power state: %s", error->message); g_error_free (error); /* Default to the cached one */ ctx->previous_real_power_state = ctx->previous_cached_power_state; diff --git a/src/tests/test-charsets.c b/src/tests/test-charsets.c index 1c31413..1bfe531 100644 --- a/src/tests/test-charsets.c +++ b/src/tests/test-charsets.c @@ -17,6 +17,7 @@ #include <string.h> #include "mm-modem-helpers.h" +#include "mm-log.h" static void test_def_chars (void *f, gpointer d) @@ -361,6 +362,25 @@ test_take_convert_ucs2_bad_ascii2 (void *f, gpointer d) g_assert (converted == NULL); } +void +_mm_log (const char *loc, + const char *func, + guint32 level, + const char *fmt, + ...) +{ +#if defined ENABLE_TEST_MESSAGE_TRACES + /* Dummy log function */ + va_list args; + gchar *msg; + + va_start (args, fmt); + msg = g_strdup_vprintf (fmt, args); + va_end (args); + g_print ("%s\n", msg); + g_free (msg); +#endif +} typedef GTestFixtureFunc TCFunc; -- cgit v0.10.2