Closed romulasry closed 1 year ago
While redoing the patches for new release, I noticed that the @anatol's patch as not accepted here https://android-review.googlesource.com/c/platform/packages/modules/adb/+/2247053. Should it be kept?
Should it be kept?
Please drop my patch. Upstream solved the problem in a slightly different way.
It's ironic to see it being fixed that way. In some repositories, maintainers suggest to fix the issue instead of suppressing the compiler warning/error with workaround 😔
Yeah, upstream decided just suppress the compiler warning. In this case (empty payload), it is fine. But if the message payload changes in the future, disabled warnings will make it harder to catch problems.
Anyway, upstream is going to maintain the code. Let them handle the situation then.
It seems that your patch is required for gcc. Here is the compiler error in archlinux
/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[];
| ^~~~~~~~~~~~~~
I see. Could you please pull the upstream patch in this case?
Yeah, doing that exactly.
I have added a pull request to update to 34.0.0 version. Please build and test from the source tarball in CI artifact.
Thank you for your work @Biswa96. This patch has been published here https://github.com/nmeum/android-tools/releases/tag/34.0.0
Also upstream just cut a new 34.0.1 release:
34.0.1 (March 2023)
**adb**
macOS: Reverted "unstable connectivity (MacBook high speed cable)" resolution due to adb install hang (https://issuetracker.google.com/issues/270205252).
**fastboot**
Windows: Fixed "mke2fs: Illegal or malformed device name while trying to determine filesystem size" error introduced in Platform tools 34.0.0 (https://issuetracker.google.com/issues/271039230).
There is no separate platform-tools-34.0.1 tag in upstream. Wondering if those patches have to be added by distribution maintainers. Another way could be to apply those patches in CI before creating a tarball.
There is no separate platform-tools-34.0.1 tag in upstream.
Android upstream regularly misses pushing these tags, unfortunately. It happened before, e.g. https://issuetracker.google.com/issues/229569602
I filed an upstream bug for missing 34.0.1 tag here https://issuetracker.google.com/issues/271591658
The 34.0.1 tag has been pushed to the repository. android-tools
is now ready to move to this release.
@Biswa96 I see you made recent 34.0.0 bump so it will be natural to take care of 34.0.1 release as well. Could you please take care of it?
OK, I'll look into it. Did the 34.0.1 change every submodules? For example,
I thought it will change only the adb and fastboot repositories.
Every android submodule is tagged with this new release. As of specific changes you need to look into git diff
between these two tags.
Thank you @Biswa96 for your work!
Build Tools, Revision 34.0.0 (February 2023) https://developer.android.com/studio/releases/build-tools#notes