public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: Josephine Pfeiffer <josie@redhat.com>
To: git-commits@fedoraproject.org
Subject: [rpms/NetworkManager] rawhide: Update to 1.58-rc1 release
Date: Fri, 03 Jul 2026 08:25:35 GMT	[thread overview]
Message-ID: <178306713582.1.2570515173205361491.rpms-NetworkManager-161a6e6ad632@fedoraproject.org> (raw)

A new commit has been pushed.

Repo   : rpms/NetworkManager
Branch : rawhide
Commit : 161a6e6ad6326a369ea6778bc3290a1e09af4681
Author : Josephine Pfeiffer <josie@redhat.com>
Date   : 2026-07-03T09:15:36+02:00
Stats  : +9/-122 in 5 file(s)
URL    : https://src.fedoraproject.org/rpms/NetworkManager/c/161a6e6ad6326a369ea6778bc3290a1e09af4681?branch=rawhide

Log:
Update to 1.58-rc1 release

---
diff --git a/.gitignore b/.gitignore
index 6f2643e..d02b5b8 100644
--- a/.gitignore
+++ b/.gitignore
@@ -457,3 +457,4 @@ prepare_sources_result*/
 /NetworkManager-1.56.0.tar.xz
 /NetworkManager-1.57.3-dev.tar.xz
 /NetworkManager-1.57.4-dev.tar.xz
+/NetworkManager-1.58-rc1.tar.xz

diff --git a/0001-libnm-use-correct-directory-permissions-in-nm_utils_.patch b/0001-libnm-use-correct-directory-permissions-in-nm_utils_.patch
deleted file mode 100644
index bbc1f00..0000000
--- a/0001-libnm-use-correct-directory-permissions-in-nm_utils_.patch
+++ /dev/null
@@ -1,38 +0,0 @@
-From 0d0b03e7d3f3082540a0dbc7ec05de835b92563d Mon Sep 17 00:00:00 2001
-From: Christian Krause <chkr@plauener.de>
-Date: Tue, 2 Jun 2026 22:28:51 +0200
-Subject: [PATCH] libnm: use correct directory permissions in
- nm_utils_copy_cert_as_user
-
-This patch fixes https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/work_items/1957.
-
-At the moment, libnm creates the temporary certificate directory
-with permission 0600.
-
-Some NM plugins drop most of their capabilities (including CAP_DAC_OVERRIDE)
-on startup and so, even when running as root, they can't create files
-in that directory (since the execute permission is missing).
-
-This patch sets the permission to 0700 to fix the problem.
-
-Fixes: 1a52bbe7c9dc ('libnm: add function to copy a certificate or key as user')
----
- src/libnm-core-impl/nm-utils.c | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/src/libnm-core-impl/nm-utils.c b/src/libnm-core-impl/nm-utils.c
-index ad7eea438b..5086270803 100644
---- a/src/libnm-core-impl/nm-utils.c
-+++ b/src/libnm-core-impl/nm-utils.c
-@@ -6506,7 +6506,7 @@ nm_utils_copy_cert_as_user(const char *filename, const char *user, GError **erro
-     nm_str_buf_destroy(&info.output_buffer);
-     nm_str_buf_destroy(&info.error_buffer);
- 
--    mkdir(RUN_CERT_DIR, 0600);
-+    mkdir(RUN_CERT_DIR, 0700);
-     fd = mkstemp(dst_path);
-     if (fd < 0) {
-         g_set_error_literal(error,
--- 
-2.54.0
-

diff --git a/0001-nmtui-fix-wrong-use-of-assertions-in-bond-page.patch b/0001-nmtui-fix-wrong-use-of-assertions-in-bond-page.patch
deleted file mode 100644
index a4295f6..0000000
--- a/0001-nmtui-fix-wrong-use-of-assertions-in-bond-page.patch
+++ /dev/null
@@ -1,65 +0,0 @@
-From 77c1cf9624e5eb788c47ff3a4feb61f4ed28a48b Mon Sep 17 00:00:00 2001
-From: Beniamino Galvani <bgalvani@redhat.com>
-Date: Wed, 27 May 2026 23:40:31 +0200
-Subject: [PATCH] nmtui: fix wrong use of assertions in bond page
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-When building in release mode the compiler complains with:
-
-  ../src/nmtui/nmt-page-bond.c: In function ‘other_options_widget_changed’:
-  ../src/nmtui/nmt-page-bond.c:373:13: error: ‘name’ may be used uninitialized [-Werror=maybe-uninitialized]
-    373 |         if (_is_other_option(name)) {
-        |             ^~~~~~~~~~~~~~~~~~~~~~
-  ../src/nmtui/nmt-page-bond.c:357:25: note: ‘name’ was declared here
-    357 |     const char         *name;
-        |                         ^~~~
-
-That happens because "name" is initialized inside an assertion, which
-is removed in release builds. Assertions must not have side effects.
-
-Move the initialization out of the assertion.
-
-Fixes: 6a841072ec3b ('nmtui/bond: introduce "other options" list')
----
- src/nmtui/nmt-page-bond.c | 8 ++++++--
- 1 file changed, 6 insertions(+), 2 deletions(-)
-
-diff --git a/src/nmtui/nmt-page-bond.c b/src/nmtui/nmt-page-bond.c
-index 962a95096b..eeea0b0b73 100644
---- a/src/nmtui/nmt-page-bond.c
-+++ b/src/nmtui/nmt-page-bond.c
-@@ -87,8 +87,10 @@ _bond_update_other_options(NMSettingBond *s_bond, NmtList *list)
-     for (i = 0; i < num_opts; i++) {
-         const char *opt_name;
-         const char *opt_value;
-+        gboolean    ret;
- 
--        nm_assert(nm_setting_bond_get_option(s_bond, i, &opt_name, &opt_value));
-+        ret = nm_setting_bond_get_option(s_bond, i, &opt_name, &opt_value);
-+        nm_assert(ret);
- 
-         if (_is_other_option(opt_name)) {
-             g_ptr_array_add(arr, g_strdup_printf("%s=%s", opt_name, opt_value));
-@@ -355,6 +357,7 @@ other_options_widget_changed(GObject *object, GParamSpec *pspec, gpointer user_d
-     NmtPageBondPrivate *priv          = NMT_PAGE_BOND_GET_PRIVATE(bond);
-     gs_strfreev char  **other_options = NULL;
-     const char         *name;
-+    gboolean            ret;
-     guint               num;
-     guint               i;
- 
-@@ -368,7 +371,8 @@ other_options_widget_changed(GObject *object, GParamSpec *pspec, gpointer user_d
- again:
-     num = nm_setting_bond_get_num_options(priv->s_bond);
-     for (i = 0; i < num; i++) {
--        nm_assert(nm_setting_bond_get_option(priv->s_bond, i, &name, NULL));
-+        ret = nm_setting_bond_get_option(priv->s_bond, i, &name, NULL);
-+        nm_assert(ret);
- 
-         if (_is_other_option(name)) {
-             nm_setting_bond_remove_option(priv->s_bond, name);
--- 
-2.53.0
-

diff --git a/NetworkManager.spec b/NetworkManager.spec
index 99fc3b6..f9be29d 100644
--- a/NetworkManager.spec
+++ b/NetworkManager.spec
@@ -5,8 +5,8 @@ URL:     https://networkmanager.dev/
 Group:   System Environment/Base
 
 Epoch:   1
-Version: 1.57.4~dev
-Release: 3%{?dist}
+Version: 1.58~rc1
+Release: 1%{?dist}
 
 ###############################################################################
 
@@ -34,11 +34,6 @@ Release: 3%{?dist}
 %global systemd_units_cloud_setup nm-cloud-setup.service nm-cloud-setup.timer
 
 ###############################################################################
-%if 0%{?fedora} > 40 || 0%{?rhel} >= 10
-%bcond_with dhclient
-%else
-%bcond_without dhclient
-%endif
 %bcond_without adsl
 %bcond_without bluetooth
 %bcond_without wwan
@@ -164,12 +159,6 @@ Source8: readme-ifcfg-rh.txt
 Source9: readme-ifcfg-rh-migrated.txt
 Source10: 24-clat-auto.conf
 
-# Keep until next rebase
-Patch1: 0001-nmtui-fix-wrong-use-of-assertions-in-bond-page.patch
-
-# Keep until next rebase, https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/commit/0d0b03e7d3f3082540a0dbc7ec05de835b92563d
-Patch2: 0001-libnm-use-correct-directory-permissions-in-nm_utils_.patch
-
 Requires(post): systemd
 Requires(post): systemd-udev
 Requires(post): /usr/sbin/update-alternatives
@@ -582,11 +571,6 @@ Preferably use nmcli instead.
 	-Dnft=%{_sbindir}/nft \
 	-Diptables=%{_sbindir}/iptables \
 	-Dip6tables=%{_sbindir}/ip6tables \
-%if %{with dhclient}
-	-Ddhclient=%{_sbindir}/dhclient \
-%else
-	-Ddhclient=no \
-%endif
 	-Ddhcpcd=no \
 	-Dcrypto=gnutls \
 %if %{with debug}
@@ -1076,6 +1060,11 @@ fi
 
 
 %changelog
+* Thu Jul 02 2026 Josephine Pfeiffer <josie@redhat.com> - 1:1.58~rc1-1
+- Update to 1.58-rc1 release
+- Drop patches now merged upstream (nmtui bond assertions, libnm cert dir permissions)
+- Drop the dhclient meson option, removed upstream in 1.58
+
 * Thu Jun 11 2026 Christian Krause <chkr@fedoraproject.org> - 1:1.57.4-3
 - Apply upstream patch to fix strongSwan VPN connection problem (rhbz#2461399)
 

diff --git a/sources b/sources
index 400e2b3..ae9dcc2 100644
--- a/sources
+++ b/sources
@@ -1 +1 @@
-SHA512 (NetworkManager-1.57.4-dev.tar.xz) = a6219479ed133a55e3cc207bfad10a8bf54dd74c4d5581dd731f406c46210d35bba497d24adbb05890673e4d14d0f762431a14a5bea2a4c2f79307c55ce62792
+SHA512 (NetworkManager-1.58-rc1.tar.xz) = 35a4ada775e411bfb74345d1d492b9297a3a8c42412d2ca16d74bb118ffe2dc982749761f8b682336e5724213455769032889c4670efe1bfe4a479640b6af328

                 reply	other threads:[~2026-07-03  8:25 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=178306713582.1.2570515173205361491.rpms-NetworkManager-161a6e6ad632@fedoraproject.org \
    --to=josie@redhat.com \
    --cc=git-commits@fedoraproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox