From 1543f622e13b7d2c6398b2eb721ffb6e43ff1c9d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20G=C3=B3rny?= Date: Sat, 20 Dec 2025 13:57:08 +0100 Subject: [PATCH] dev-util/android-tools: Remove old MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Michał Górny --- dev-util/android-tools/Manifest | 3 - .../android-tools-33.0.3-r1.ebuild | 95 -------------- .../android-tools/android-tools-34.0.5.ebuild | 116 ----------------- .../android-tools/android-tools-35.0.1.ebuild | 116 ----------------- ...usbdevfs_urb-to-match-new-kernel-UAP.patch | 120 ------------------ .../android-tools-33.0.3-adb-gcc-13.patch | 25 ---- 6 files changed, 475 deletions(-) delete mode 100644 dev-util/android-tools/android-tools-33.0.3-r1.ebuild delete mode 100644 dev-util/android-tools/android-tools-34.0.5.ebuild delete mode 100644 dev-util/android-tools/android-tools-35.0.1.ebuild delete mode 100644 dev-util/android-tools/files/android-tools-33.0.3-adb-0023-Update-usage-of-usbdevfs_urb-to-match-new-kernel-UAP.patch delete mode 100644 dev-util/android-tools/files/android-tools-33.0.3-adb-gcc-13.patch diff --git a/dev-util/android-tools/Manifest b/dev-util/android-tools/Manifest index 3f35551e7a02..2d663b10326f 100644 --- a/dev-util/android-tools/Manifest +++ b/dev-util/android-tools/Manifest @@ -1,5 +1,2 @@ DIST android-tools-31.0.3-no-gtest.patch 3152 BLAKE2B da71e3cba87b21ffcb144602a06d7e269faabd173233363b757cd2191c051f8d3281f591117d4551d7acc029d6e2d421f702f9f836dfe0dec6c676aa39a2d9fe SHA512 a28d2264bf40c420a279acf1f4c3b4588d96ce2d6e2d6d322abbde62d773804bef10dc33f13dd52c7de80ace6c58f91ae28f5d4e8e97dfae4d2a8473d8537423 -DIST android-tools-33.0.3.tar.xz 25300756 BLAKE2B 4254f0ab8657966cf56e2a7c3e9a3d889dfa9f26e61d8b46480bb74c5a8be33494f450827e3b2db4087006b3160892e6a846ef33719870cbbd032647cfc3ce78 SHA512 78c8141f5f13f1c0e5da474e1d79760a612e911519af4e9f21c0e5005f2ce99f3aeacf16f3fcdfe1250fc795a88190fa243592e1066a2156e1fd8d2b9c3d7f71 -DIST android-tools-34.0.5.tar.xz 19389824 BLAKE2B a86ad9d22c356d2b3d0d85dd17e5540b5d105a286dd07946659dac4ab6d1e8285eecbbf0377c542ac6ca78e0af8df320a97e24ea2b735bbec66a2d2b23d2099c SHA512 2edea0c7a4c2f63531a8b5f518ed4dc9c315d08d8b23d3fac3b7024e2297860465f6df9fad43c9925fb80b5a50f5f3f4744886b552818bf3e961ba0372212496 -DIST android-tools-35.0.1.tar.xz 21460424 BLAKE2B e87a4ad13e5b8030069c87cbbe47f3698888e797e319aa3a054a668ad55f4cdd7445e5ef7825f2d185f6891958fd52f815ee0cb3909649ee3ea703061d72d0ab SHA512 2dc3831f29a2c5ffc57978e4a6fa6d1469bcb0740750779b3e4f8239eaec87864f3474254a2d649ab630062be091d8b44a4e45c7e7b7e033f3b4481f81d76296 DIST android-tools-35.0.2.tar.xz 18847376 BLAKE2B 5db4d32e4d9acca5f98f0fa76d02f2cbb9c8ad23f7138deb57aa78bcfb02e8e1998dd5649e2a7f77df99f4f4c0b47440c8f6b84d2a001ae8ec2f4840e01535a8 SHA512 391ce4d638b274d7bbae24a3df8de8b5812a982570f29b2aef37d12a3ba7ed6f66b5c0b7f908759e0b0da30d152b5319af0fef16c54bdc3b9f4074fb22f80d10 diff --git a/dev-util/android-tools/android-tools-33.0.3-r1.ebuild b/dev-util/android-tools/android-tools-33.0.3-r1.ebuild deleted file mode 100644 index ef4c369f862a..000000000000 --- a/dev-util/android-tools/android-tools-33.0.3-r1.ebuild +++ /dev/null @@ -1,95 +0,0 @@ -# Copyright 1999-2025 Gentoo Authors -# Distributed under the terms of the GNU General Public License v2 - -EAPI=8 - -PYTHON_COMPAT=( python3_11 ) - -inherit cmake python-r1 - -DESCRIPTION="Android platform tools (adb, fastboot, and mkbootimg)" -HOMEPAGE="https://github.com/nmeum/android-tools/ https://developer.android.com/" - -MY_PV="${PV//_/}" -SRC_URI="https://github.com/nmeum/android-tools/releases/download/${MY_PV}/${PN}-${MY_PV}.tar.xz - https://dev.gentoo.org/~zmedico/dist/${PN}-31.0.3-no-gtest.patch -" -S="${WORKDIR}/${PN}-${MY_PV}" - -# The entire source code is Apache-2.0, except for fastboot which is BSD-2. -LICENSE="Apache-2.0 BSD-2" -SLOT="0" -KEYWORDS="amd64 ~arm ~arm64 x86" -IUSE="python udev" -REQUIRED_USE="python? ( ${PYTHON_REQUIRED_USE} )" - -# dev-libs/libpcre only required for e2fsdroid -DEPEND=" - app-arch/brotli:= - app-arch/lz4:= - app-arch/zstd:= - dev-cpp/abseil-cpp:= - dev-libs/libpcre2:= - -Date: Mon, 10 Oct 2022 10:47:57 -0700 -Subject: [PATCH] Update usage of usbdevfs_urb to match new kernel UAPI -MIME-Version: 1.0 -Content-Type: text/plain; charset=UTF-8 -Content-Transfer-Encoding: 8bit - -Linux kernel API has been changed by commit 94dfc73e7cf4 ("treewide: uapi: Replace zero-length arrays with flexible-array members") -where zero-length array iso_frame_desc in struct usbdevfs_urb was replaced with a proper flexible-array member. - -Current USB API usage causes a compilation error at Linux 6.0: - -In file included from /home/mae/.cache/kiss/proc/121205/build/android-tools/vendor/adb/client/usb_linux.cpp:28: -/usr/include/linux/usbdevice_fs.h:134:41: error: flexible array member ‘usbdevfs_urb::iso_frame_desc’ not at end of ‘struct usb_handle’ - 134 | struct usbdevfs_iso_packet_desc iso_frame_desc[]; - | ^~~~~~~~~~~~~~ -/home/mae/.cache/kiss/proc/121205/build/android-tools/vendor/adb/client/usb_linux.cpp:76:18: note: next member ‘usbdevfs_urb usb_handle::urb_out’ declared here - 76 | usbdevfs_urb urb_out; - | ^~~~~~~ -/home/mae/.cache/kiss/proc/121205/build/android-tools/vendor/adb/client/usb_linux.cpp:61:8: note: in the definition of ‘struct usb_handle’ - 61 | struct usb_handle { - | ^~~~~~~~~~ - -Fix it by using pointers to a struct with flexible-array members. -Current fix works both with the old and the new API. - -See https://github.com/nmeum/android-tools/issues/74 for more context. - -Tested: built on Linux against kernel 5.19 and 6.0; 'adb shell' over USB -cable -Acked-by: Gustavo A. R. Silva gustavoars@kernel.org -Change-Id: I7f0f7b35d9a3ab980d3520b541b60c7857a6b101 -Signed-off-by: Anatol Pomozov ---- a/client/usb_linux.cpp -+++ b/client/usb_linux.cpp -@@ -71,8 +71,8 @@ struct usb_handle { - unsigned zero_mask; - unsigned writeable = 1; - -- usbdevfs_urb urb_in; -- usbdevfs_urb urb_out; -+ usbdevfs_urb *urb_in; -+ usbdevfs_urb *urb_out; - - bool urb_in_busy = false; - bool urb_out_busy = false; -@@ -303,7 +303,7 @@ static int usb_bulk_write(usb_handle* h, const void* data, int len) { - std::unique_lock lock(h->mutex); - D("++ usb_bulk_write ++"); - -- usbdevfs_urb* urb = &h->urb_out; -+ usbdevfs_urb* urb = h->urb_out; - memset(urb, 0, sizeof(*urb)); - urb->type = USBDEVFS_URB_TYPE_BULK; - urb->endpoint = h->ep_out; -@@ -342,7 +342,7 @@ static int usb_bulk_read(usb_handle* h, void* data, int len) { - std::unique_lock lock(h->mutex); - D("++ usb_bulk_read ++"); - -- usbdevfs_urb* urb = &h->urb_in; -+ usbdevfs_urb* urb = h->urb_in; - memset(urb, 0, sizeof(*urb)); - urb->type = USBDEVFS_URB_TYPE_BULK; - urb->endpoint = h->ep_in; -@@ -387,7 +387,7 @@ static int usb_bulk_read(usb_handle* h, void* data, int len) { - } - D("[ urb @%p status = %d, actual = %d ]", out, out->status, out->actual_length); - -- if (out == &h->urb_in) { -+ if (out == h->urb_in) { - D("[ reap urb - IN complete ]"); - h->urb_in_busy = false; - if (urb->status != 0) { -@@ -396,7 +396,7 @@ static int usb_bulk_read(usb_handle* h, void* data, int len) { - } - return urb->actual_length; - } -- if (out == &h->urb_out) { -+ if (out == h->urb_out) { - D("[ reap urb - OUT compelete ]"); - h->urb_out_busy = false; - h->cv.notify_all(); -@@ -500,10 +500,10 @@ void usb_kick(usb_handle* h) { - ** but this ensures that a reader blocked on REAPURB - ** will get unblocked - */ -- ioctl(h->fd, USBDEVFS_DISCARDURB, &h->urb_in); -- ioctl(h->fd, USBDEVFS_DISCARDURB, &h->urb_out); -- h->urb_in.status = -ENODEV; -- h->urb_out.status = -ENODEV; -+ ioctl(h->fd, USBDEVFS_DISCARDURB, h->urb_in); -+ ioctl(h->fd, USBDEVFS_DISCARDURB, h->urb_out); -+ h->urb_in->status = -ENODEV; -+ h->urb_out->status = -ENODEV; - h->urb_in_busy = false; - h->urb_out_busy = false; - h->cv.notify_all(); -@@ -519,6 +519,8 @@ int usb_close(usb_handle* h) { - - D("-- usb close %p (fd = %d) --", h, h->fd); - -+ delete h->urb_in; -+ delete h->urb_out; - delete h; - - return 0; -@@ -572,6 +574,8 @@ static void register_device(const char* dev_name, const char* dev_path, unsigned - usb->ep_out = ep_out; - usb->zero_mask = zero_mask; - usb->max_packet_size = max_packet_size; -+ usb->urb_in = new usbdevfs_urb; -+ usb->urb_out = new usbdevfs_urb; - - // Initialize mark so we don't get garbage collected after the device scan. - usb->mark = true; diff --git a/dev-util/android-tools/files/android-tools-33.0.3-adb-gcc-13.patch b/dev-util/android-tools/files/android-tools-33.0.3-adb-gcc-13.patch deleted file mode 100644 index 4ba2c9a49c31..000000000000 --- a/dev-util/android-tools/files/android-tools-33.0.3-adb-gcc-13.patch +++ /dev/null @@ -1,25 +0,0 @@ -https://bugs.gentoo.org/875575 -https://android-review.googlesource.com/c/platform/packages/modules/adb/+/2399311 - -From ddffab649b12dce1502d63711836b58d007f6a28 Mon Sep 17 00:00:00 2001 -From: Heiko Becker -Date: Mon, 23 Jan 2023 23:09:52 +0100 -Subject: [PATCH] Fix build with gcc 13 by including - -Like other versions before, gcc 13 moved some includes around and as a -result is no longer transitively included. Explicitly include -it for uint{32,64}_t. - -Test: local build -Change-Id: I05a27726b05427c486fd01b013dba4d698abac97 ---- a/file_sync_protocol.h -+++ b/file_sync_protocol.h -@@ -16,6 +16,8 @@ - - #pragma once - -+#include -+ - #define MKID(a, b, c, d) ((a) | ((b) << 8) | ((c) << 16) | ((d) << 24)) - - #define ID_LSTAT_V1 MKID('S', 'T', 'A', 'T')