Merge ~lucaskanashiro/ubuntu/+source/ethtool:fix-ethtool-module-info-lunar into ubuntu/+source/ethtool:ubuntu/lunar-devel

Proposed by Lucas Kanashiro
Status: Merged
Approved by: git-ubuntu bot
Approved revision: not available
Merge reported by: Andreas Hasenack
Merged at revision: eab2d41d1084a761f70b678fcdccaa434fda4b50
Proposed branch: ~lucaskanashiro/ubuntu/+source/ethtool:fix-ethtool-module-info-lunar
Merge into: ubuntu/+source/ethtool:ubuntu/lunar-devel
Diff against target: 67 lines (+36/-1)
4 files modified
debian/changelog (+8/-0)
debian/control (+2/-1)
debian/patches/0001-Fix-ethtool-module-info-in-human-readable-mode.patch (+25/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
git-ubuntu bot Approve
Andreas Hasenack Approve
Canonical Server Reporter Pending
Review via email: mp+457365@code.launchpad.net

Description of the change

Import patch proposed by upstream to fix ethtool module info in human-readable mode. See LP: #2043983.

PPA: https://launchpad.net/~lucaskanashiro/+archive/ubuntu/ethtool/+packages

To post a comment you must log in.
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

+1, straightforward

review: Approve
Revision history for this message
git-ubuntu bot (git-ubuntu-bot) wrote :

Approvers: lucaskanashiro, ahasenack
Uploaders: lucaskanashiro, ahasenack
MP auto-approved

review: Approve
Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

Thanks Andreas, package uploaded:

Uploading ethtool_6.1-1ubuntu0.1.dsc
Uploading ethtool_6.1-1ubuntu0.1.debian.tar.xz
Uploading ethtool_6.1-1ubuntu0.1_source.changes

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

This was uploaded to lunar-proposed, and imported into git-ubuntu, but the package was not verified in time for lunar's EOL and was then removed from lunar-proposed. In terms of branches, this was merged, so I'll mark it as such anyway.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/changelog b/debian/changelog
index f58a8ee..045e938 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,11 @@
1ethtool (1:6.1-1ubuntu0.1) lunar; urgency=medium
2
3 * d/p/0001-Fix-ethtool-module-info-in-human-readable-mode.patch: add
4 upstream patch to avoid hex dump report even in human-readable mode
5 (LP: #2043983).
6
7 -- Lucas Kanashiro <kanashiro@ubuntu.com> Tue, 12 Dec 2023 17:44:20 -0300
8
1ethtool (1:6.1-1) unstable; urgency=medium9ethtool (1:6.1-1) unstable; urgency=medium
210
3 * New upstream release: 6.111 * New upstream release: 6.1
diff --git a/debian/control b/debian/control
index ed4d896..d7dbefb 100644
--- a/debian/control
+++ b/debian/control
@@ -1,7 +1,8 @@
1Source: ethtool1Source: ethtool
2Section: net2Section: net
3Priority: optional3Priority: optional
4Maintainer: Debian Kernel Team <debian-kernel@lists.debian.org>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5XSBC-Original-Maintainer: Debian Kernel Team <debian-kernel@lists.debian.org>
5Uploaders: Ben Hutchings <benh@debian.org>, Salvatore Bonaccorso <carnil@debian.org>6Uploaders: Ben Hutchings <benh@debian.org>, Salvatore Bonaccorso <carnil@debian.org>
6Standards-Version: 4.6.27Standards-Version: 4.6.2
7Build-Depends: debhelper-compat (= 13), autoconf, automake, libmnl-dev, pkg-config8Build-Depends: debhelper-compat (= 13), autoconf, automake, libmnl-dev, pkg-config
diff --git a/debian/patches/0001-Fix-ethtool-module-info-in-human-readable-mode.patch b/debian/patches/0001-Fix-ethtool-module-info-in-human-readable-mode.patch
8new file mode 1006449new file mode 100644
index 0000000..77916d8
--- /dev/null
+++ b/debian/patches/0001-Fix-ethtool-module-info-in-human-readable-mode.patch
@@ -0,0 +1,25 @@
1From: Lucas Kanashiro <lucas.kanashiro@canonical.com>
2Date: Mon, 11 Dec 2023 18:49:23 -0300
3Subject: Fix ethtool module info in human readable mode
4
5Origin: upstream, https://lists.openwall.net/netdev/2023/11/23/118
6Forwarded: https://salsa.debian.org/kernel-team/ethtool/-/merge_requests/3
7Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/ethtool/+bug/2043983
8Reviewed-By: Lucas Kanashiro <kanashiro@ubuntu.com>
9Last-Updated: 2023-12-11
10---
11 netlink/module-eeprom.c | 1 +
12 1 file changed, 1 insertion(+)
13
14diff --git a/netlink/module-eeprom.c b/netlink/module-eeprom.c
15index 09ad580..e9dd270 100644
16--- a/netlink/module-eeprom.c
17+++ b/netlink/module-eeprom.c
18@@ -216,6 +216,7 @@ static int eeprom_parse(struct cmd_context *ctx)
19
20 switch (request.data[0]) {
21 #ifdef ETHTOOL_ENABLE_PRETTY_DUMP
22+ case SFF8024_ID_SOLDERED_MODULE:
23 case SFF8024_ID_SFP:
24 return sff8079_show_all_nl(ctx);
25 case SFF8024_ID_QSFP:
diff --git a/debian/patches/series b/debian/patches/series
index e69de29..d500446 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -0,0 +1 @@
10001-Fix-ethtool-module-info-in-human-readable-mode.patch

Subscribers

People subscribed via source and target branches