* Re: android-tools: update to 30.0.5.
2021-01-25 0:56 [PR PATCH] android-tools: update to 30.0.5 sgn
@ 2021-01-25 1:24 ` sgn
2021-01-25 1:29 ` [PR PATCH] [Updated] " sgn
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: sgn @ 2021-01-25 1:24 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 263 bytes --]
New comment by sgn on void-packages repository
https://github.com/void-linux/void-packages/pull/28209#issuecomment-766482724
Comment:
Oops https://github.com/void-linux/void-packages/pull/28209/checks?check_run_id=1758983334 I couldn't write a correct CMake :|
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PR PATCH] [Updated] android-tools: update to 30.0.5.
2021-01-25 0:56 [PR PATCH] android-tools: update to 30.0.5 sgn
2021-01-25 1:24 ` sgn
@ 2021-01-25 1:29 ` sgn
2021-01-25 1:39 ` sgn
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: sgn @ 2021-01-25 1:29 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1677 bytes --]
There is an updated pull request by sgn against master on the void-packages repository
https://github.com/sgn/void-packages android-tools-30.0.5
https://github.com/void-linux/void-packages/pull/28209
android-tools: update to 30.0.5.
<!-- Mark items with [x] where applicable -->
#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
The re-packaging efforts was done by me but I only used on x86_64-musl.
A patch file from https://github.com/void-linux/void-packages/pull/28209.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-android-tools-30.0.5-28209.patch --]
[-- Type: text/x-diff, Size: 5527 bytes --]
From 6b1fdf7112b270ca8a83019736c2ebac1ae0c122 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
<congdanhqx@gmail.com>
Date: Mon, 25 Jan 2021 07:41:21 +0700
Subject: [PATCH] android-tools: update to 30.0.5.
---
...ypto_defaults-depends-on-protobuf-he.patch | 58 +++++++++++++++++++
srcpkgs/android-tools/template | 18 +++---
srcpkgs/android-tools/update | 2 +-
3 files changed, 66 insertions(+), 12 deletions(-)
create mode 100644 srcpkgs/android-tools/patches/0001-vendor-libadb_crypto_defaults-depends-on-protobuf-he.patch
diff --git a/srcpkgs/android-tools/patches/0001-vendor-libadb_crypto_defaults-depends-on-protobuf-he.patch b/srcpkgs/android-tools/patches/0001-vendor-libadb_crypto_defaults-depends-on-protobuf-he.patch
new file mode 100644
index 00000000000..f6a813892a0
--- /dev/null
+++ b/srcpkgs/android-tools/patches/0001-vendor-libadb_crypto_defaults-depends-on-protobuf-he.patch
@@ -0,0 +1,58 @@
+From e284901aa2ce180164cbf4f2d613fb8f484e4f5e Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
+ <congdanhqx@gmail.com>
+Date: Mon, 25 Jan 2021 08:25:31 +0700
+Subject: [PATCH] vendor: libadb_crypto_defaults: depends on protobuf header
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+While we're at it, simplify OBJECT_DEPENDS as pointed out in
+https://cmake.org/cmake/help/v3.18/prop_sf/OBJECT_DEPENDS.html
+
+This fix a broken build because of a race as shown in:
+https://github.com/void-linux/void-packages/pull/28209/checks?check_run_id=1758983334
+
+Signed-off-by: Đoàn Trần Công Danh <congdanhqx@gmail.com>
+---
+ vendor/CMakeLists.adb.txt | 20 ++++++--------------
+ 1 file changed, 6 insertions(+), 14 deletions(-)
+
+diff --git vendor/CMakeLists.adb.txt vendor/CMakeLists.adb.txt
+index 1495a5d..ace2558 100644
+--- vendor/CMakeLists.adb.txt
++++ vendor/CMakeLists.adb.txt
+@@ -34,19 +34,10 @@ add_library(libadb STATIC
+ core/adb/sysdeps_unix.cpp
+ core/adb/sysdeps/errno.cpp
+ core/adb/sysdeps/posix/network.cpp
+- ${ADB_APP_PROCESSES_PROTO_SRCS}
+- ${ADB_KNOWN_HOSTS_PROTO_SRCS}
+- ${ADB_KEY_TYPE_PROTO_SRCS}
+- ${ADB_PAIRING_PROTO_SRCS})
+-
+-set_property(SOURCE core/adb/client/commandline.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_APP_PROCESSES_PROTO_HDRS})
+-set_property(SOURCE core/adb/client/adb_wifi.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_KNOWN_HOSTS_PROTO_HDRS})
+-set_property(SOURCE core/adb/client/auth.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_KEY_TYPE_PROTO_HDRS})
+-set_property(SOURCE core/adb/pairing_connection/pairing_connection.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_PAIRING_PROTO_HDRS})
++ ${ADB_APP_PROCESSES_PROTO_SRCS} ${ADB_APP_PROCESSES_PROTO_HDRS}
++ ${ADB_KNOWN_HOSTS_PROTO_SRCS} ${ADB_KNOWN_HOSTS_PROTO_HDRS}
++ ${ADB_KEY_TYPE_PROTO_SRCS} ${ADB_KEY_TYPE_PROTO_HDRS}
++ ${ADB_PAIRING_PROTO_SRCS} ${ADB_PAIRING_PROTO_HDRS})
+
+ target_compile_definitions(libadb PRIVATE -D_GNU_SOURCE)
+ target_compile_definitions(libadb PUBLIC -DADB_HOST=1)
+@@ -68,7 +59,8 @@ target_include_directories(libadb PUBLIC
+ add_library(libadb_crypto_defaults STATIC
+ core/adb/crypto/key.cpp
+ core/adb/crypto/rsa_2048_key.cpp
+- core/adb/crypto/x509_generator.cpp)
++ core/adb/crypto/x509_generator.cpp
++ ${ADB_KEY_TYPE_PROTO_HDRS})
+
+ target_include_directories(libadb_crypto_defaults PUBLIC
+ core/adb
diff --git a/srcpkgs/android-tools/template b/srcpkgs/android-tools/template
index 343a543a9a2..f4104cb07d1 100644
--- a/srcpkgs/android-tools/template
+++ b/srcpkgs/android-tools/template
@@ -1,20 +1,22 @@
# Template file for 'android-tools'
pkgname=android-tools
-version=29.0.6
-revision=3
+version=30.0.5
+revision=1
archs="armv* aarch64* x86_64* i686* ppc64le*"
build_style=cmake
-hostmakedepends="perl go"
-makedepends="gtest-devel zlib-devel libressl-devel libusb-devel pcre2-devel"
+hostmakedepends="perl go protobuf"
+makedepends="gtest-devel zlib-devel libressl-devel libusb-devel pcre2-devel
+ liblz4-devel libzstd-devel protobuf-devel brotli-devel"
short_desc="Android platform tools (adb and fastboot)"
maintainer="John <me@johnnynator.dev>"
license="Apache-2.0, ISC, GPL-2.0-only, MIT"
homepage="http://developer.android.com/tools/help/adb.html"
distfiles="https://github.com/nmeum/android-tools/releases/download/${version}/android-tools-${version}.tar.xz"
-checksum=7fb1c127c36b0752657593838b6823743bf8e5730f9f8b0f7ba2c185424cf376
+checksum=1b870a3f28d75e4907c95dbe8e71b4014f56be636d089a75257ef3922ad39d62
if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
makedepends+=" libatomic-devel"
+ configure_args="-DCMAKE_CXX_STANDARD_LIBRARIES=-latomic"
fi
pre_configure() {
@@ -31,16 +33,10 @@ pre_configure() {
_EOF
configure_args+=" ${boring_ssl_cmake_args}"
fi
-
- if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
- sed -i vendor/CMakeLists.adb.txt \
- -e "/target_link_libraries/s;$; atomic;"
- fi
}
post_install() {
vlicense vendor/boringssl/LICENSE boringssl.LICENSE
vlicense vendor/boringssl/third_party/fiat/LICENSE fiat.LICENSE
- vlicense vendor/boringssl/third_party/googletest/LICENSE gtest.LICENSE
vsv adb
}
diff --git a/srcpkgs/android-tools/update b/srcpkgs/android-tools/update
index 49895605477..f69b1910ad9 100644
--- a/srcpkgs/android-tools/update
+++ b/srcpkgs/android-tools/update
@@ -1,2 +1,2 @@
-site="https://android.googlesource.com/platform/system/core"
+site="https://android.googlesource.com/platform/system/core/+refs"
pattern='platform-tools-\K[\d._r]+'
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PR PATCH] [Updated] android-tools: update to 30.0.5.
2021-01-25 0:56 [PR PATCH] android-tools: update to 30.0.5 sgn
2021-01-25 1:24 ` sgn
2021-01-25 1:29 ` [PR PATCH] [Updated] " sgn
@ 2021-01-25 1:39 ` sgn
2021-01-25 2:18 ` sgn
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: sgn @ 2021-01-25 1:39 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1677 bytes --]
There is an updated pull request by sgn against master on the void-packages repository
https://github.com/sgn/void-packages android-tools-30.0.5
https://github.com/void-linux/void-packages/pull/28209
android-tools: update to 30.0.5.
<!-- Mark items with [x] where applicable -->
#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
The re-packaging efforts was done by me but I only used on x86_64-musl.
A patch file from https://github.com/void-linux/void-packages/pull/28209.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-android-tools-30.0.5-28209.patch --]
[-- Type: text/x-diff, Size: 5694 bytes --]
From 232ae40f459ecfc35d9b84e552cfd7255e9e4f45 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
<congdanhqx@gmail.com>
Date: Mon, 25 Jan 2021 07:41:21 +0700
Subject: [PATCH] android-tools: update to 30.0.5.
---
...s-fix-dependencies-on-protobuf-heade.patch | 63 +++++++++++++++++++
srcpkgs/android-tools/template | 18 +++---
srcpkgs/android-tools/update | 2 +-
3 files changed, 71 insertions(+), 12 deletions(-)
create mode 100644 srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch
diff --git a/srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch b/srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch
new file mode 100644
index 00000000000..0d883b4f0fb
--- /dev/null
+++ b/srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch
@@ -0,0 +1,63 @@
+From 41f92b797aa6bf4609b6b31e42bd37df710ae7aa Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
+ <congdanhqx@gmail.com>
+Date: Mon, 25 Jan 2021 08:25:31 +0700
+Subject: [PATCH] vendor: CMakeLists: fix dependencies on protobuf header
+
+While we're at it, simplify OBJECT_DEPENDS as pointed out in
+https://cmake.org/cmake/help/v3.18/prop_sf/OBJECT_DEPENDS.html
+
+This fix a broken build because of a race as shown in:
+https://github.com/void-linux/void-packages/pull/28209/checks?check_run_id=1758983334
+---
+ vendor/CMakeLists.adb.txt | 23 ++++++++---------------
+ 1 file changed, 8 insertions(+), 15 deletions(-)
+
+diff --git vendor/CMakeLists.adb.txt vendor/CMakeLists.adb.txt
+index 1495a5d..5206a80 100644
+--- vendor/CMakeLists.adb.txt
++++ vendor/CMakeLists.adb.txt
+@@ -34,19 +34,10 @@ add_library(libadb STATIC
+ core/adb/sysdeps_unix.cpp
+ core/adb/sysdeps/errno.cpp
+ core/adb/sysdeps/posix/network.cpp
+- ${ADB_APP_PROCESSES_PROTO_SRCS}
+- ${ADB_KNOWN_HOSTS_PROTO_SRCS}
+- ${ADB_KEY_TYPE_PROTO_SRCS}
+- ${ADB_PAIRING_PROTO_SRCS})
+-
+-set_property(SOURCE core/adb/client/commandline.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_APP_PROCESSES_PROTO_HDRS})
+-set_property(SOURCE core/adb/client/adb_wifi.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_KNOWN_HOSTS_PROTO_HDRS})
+-set_property(SOURCE core/adb/client/auth.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_KEY_TYPE_PROTO_HDRS})
+-set_property(SOURCE core/adb/pairing_connection/pairing_connection.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_PAIRING_PROTO_HDRS})
++ ${ADB_APP_PROCESSES_PROTO_SRCS} ${ADB_APP_PROCESSES_PROTO_HDRS}
++ ${ADB_KNOWN_HOSTS_PROTO_SRCS} ${ADB_KNOWN_HOSTS_PROTO_HDRS}
++ ${ADB_KEY_TYPE_PROTO_SRCS} ${ADB_KEY_TYPE_PROTO_HDRS}
++ ${ADB_PAIRING_PROTO_SRCS} ${ADB_PAIRING_PROTO_HDRS})
+
+ target_compile_definitions(libadb PRIVATE -D_GNU_SOURCE)
+ target_compile_definitions(libadb PUBLIC -DADB_HOST=1)
+@@ -68,7 +59,8 @@ target_include_directories(libadb PUBLIC
+ add_library(libadb_crypto_defaults STATIC
+ core/adb/crypto/key.cpp
+ core/adb/crypto/rsa_2048_key.cpp
+- core/adb/crypto/x509_generator.cpp)
++ core/adb/crypto/x509_generator.cpp
++ ${ADB_KEY_TYPE_PROTO_HDRS})
+
+ target_include_directories(libadb_crypto_defaults PUBLIC
+ core/adb
+@@ -173,7 +165,8 @@ add_executable(adb
+ core/adb/transport_fd.cpp
+ core/adb/client/transport_local.cpp
+ core/adb/client/transport_usb.cpp
+- core/adb/types.cpp)
++ core/adb/types.cpp
++ ${ADB_KEY_TYPE_PROTO_HDRS})
+
+ target_compile_definitions(adb PRIVATE
+ -DPLATFORM_TOOLS_VERSION="${ANDROID_VERSION}"
diff --git a/srcpkgs/android-tools/template b/srcpkgs/android-tools/template
index 343a543a9a2..f4104cb07d1 100644
--- a/srcpkgs/android-tools/template
+++ b/srcpkgs/android-tools/template
@@ -1,20 +1,22 @@
# Template file for 'android-tools'
pkgname=android-tools
-version=29.0.6
-revision=3
+version=30.0.5
+revision=1
archs="armv* aarch64* x86_64* i686* ppc64le*"
build_style=cmake
-hostmakedepends="perl go"
-makedepends="gtest-devel zlib-devel libressl-devel libusb-devel pcre2-devel"
+hostmakedepends="perl go protobuf"
+makedepends="gtest-devel zlib-devel libressl-devel libusb-devel pcre2-devel
+ liblz4-devel libzstd-devel protobuf-devel brotli-devel"
short_desc="Android platform tools (adb and fastboot)"
maintainer="John <me@johnnynator.dev>"
license="Apache-2.0, ISC, GPL-2.0-only, MIT"
homepage="http://developer.android.com/tools/help/adb.html"
distfiles="https://github.com/nmeum/android-tools/releases/download/${version}/android-tools-${version}.tar.xz"
-checksum=7fb1c127c36b0752657593838b6823743bf8e5730f9f8b0f7ba2c185424cf376
+checksum=1b870a3f28d75e4907c95dbe8e71b4014f56be636d089a75257ef3922ad39d62
if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
makedepends+=" libatomic-devel"
+ configure_args="-DCMAKE_CXX_STANDARD_LIBRARIES=-latomic"
fi
pre_configure() {
@@ -31,16 +33,10 @@ pre_configure() {
_EOF
configure_args+=" ${boring_ssl_cmake_args}"
fi
-
- if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
- sed -i vendor/CMakeLists.adb.txt \
- -e "/target_link_libraries/s;$; atomic;"
- fi
}
post_install() {
vlicense vendor/boringssl/LICENSE boringssl.LICENSE
vlicense vendor/boringssl/third_party/fiat/LICENSE fiat.LICENSE
- vlicense vendor/boringssl/third_party/googletest/LICENSE gtest.LICENSE
vsv adb
}
diff --git a/srcpkgs/android-tools/update b/srcpkgs/android-tools/update
index 49895605477..f69b1910ad9 100644
--- a/srcpkgs/android-tools/update
+++ b/srcpkgs/android-tools/update
@@ -1,2 +1,2 @@
-site="https://android.googlesource.com/platform/system/core"
+site="https://android.googlesource.com/platform/system/core/+refs"
pattern='platform-tools-\K[\d._r]+'
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PR PATCH] [Updated] android-tools: update to 30.0.5.
2021-01-25 0:56 [PR PATCH] android-tools: update to 30.0.5 sgn
` (2 preceding siblings ...)
2021-01-25 1:39 ` sgn
@ 2021-01-25 2:18 ` sgn
2021-01-25 16:26 ` sgn
2021-01-26 12:46 ` [PR PATCH] [Merged]: " sgn
5 siblings, 0 replies; 7+ messages in thread
From: sgn @ 2021-01-25 2:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1677 bytes --]
There is an updated pull request by sgn against master on the void-packages repository
https://github.com/sgn/void-packages android-tools-30.0.5
https://github.com/void-linux/void-packages/pull/28209
android-tools: update to 30.0.5.
<!-- Mark items with [x] where applicable -->
#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
The re-packaging efforts was done by me but I only used on x86_64-musl.
A patch file from https://github.com/void-linux/void-packages/pull/28209.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-android-tools-30.0.5-28209.patch --]
[-- Type: text/x-diff, Size: 7059 bytes --]
From 7c1d05a2671b7ef9eddd24a4189b1688fa8f9e87 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
<congdanhqx@gmail.com>
Date: Mon, 25 Jan 2021 07:41:21 +0700
Subject: [PATCH 1/2] android-tools: update to 30.0.5.
---
...s-fix-dependencies-on-protobuf-heade.patch | 63 +++++++++++++++++++
srcpkgs/android-tools/template | 18 +++---
srcpkgs/android-tools/update | 2 +-
3 files changed, 71 insertions(+), 12 deletions(-)
create mode 100644 srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch
diff --git a/srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch b/srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch
new file mode 100644
index 00000000000..0d883b4f0fb
--- /dev/null
+++ b/srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch
@@ -0,0 +1,63 @@
+From 41f92b797aa6bf4609b6b31e42bd37df710ae7aa Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
+ <congdanhqx@gmail.com>
+Date: Mon, 25 Jan 2021 08:25:31 +0700
+Subject: [PATCH] vendor: CMakeLists: fix dependencies on protobuf header
+
+While we're at it, simplify OBJECT_DEPENDS as pointed out in
+https://cmake.org/cmake/help/v3.18/prop_sf/OBJECT_DEPENDS.html
+
+This fix a broken build because of a race as shown in:
+https://github.com/void-linux/void-packages/pull/28209/checks?check_run_id=1758983334
+---
+ vendor/CMakeLists.adb.txt | 23 ++++++++---------------
+ 1 file changed, 8 insertions(+), 15 deletions(-)
+
+diff --git vendor/CMakeLists.adb.txt vendor/CMakeLists.adb.txt
+index 1495a5d..5206a80 100644
+--- vendor/CMakeLists.adb.txt
++++ vendor/CMakeLists.adb.txt
+@@ -34,19 +34,10 @@ add_library(libadb STATIC
+ core/adb/sysdeps_unix.cpp
+ core/adb/sysdeps/errno.cpp
+ core/adb/sysdeps/posix/network.cpp
+- ${ADB_APP_PROCESSES_PROTO_SRCS}
+- ${ADB_KNOWN_HOSTS_PROTO_SRCS}
+- ${ADB_KEY_TYPE_PROTO_SRCS}
+- ${ADB_PAIRING_PROTO_SRCS})
+-
+-set_property(SOURCE core/adb/client/commandline.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_APP_PROCESSES_PROTO_HDRS})
+-set_property(SOURCE core/adb/client/adb_wifi.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_KNOWN_HOSTS_PROTO_HDRS})
+-set_property(SOURCE core/adb/client/auth.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_KEY_TYPE_PROTO_HDRS})
+-set_property(SOURCE core/adb/pairing_connection/pairing_connection.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_PAIRING_PROTO_HDRS})
++ ${ADB_APP_PROCESSES_PROTO_SRCS} ${ADB_APP_PROCESSES_PROTO_HDRS}
++ ${ADB_KNOWN_HOSTS_PROTO_SRCS} ${ADB_KNOWN_HOSTS_PROTO_HDRS}
++ ${ADB_KEY_TYPE_PROTO_SRCS} ${ADB_KEY_TYPE_PROTO_HDRS}
++ ${ADB_PAIRING_PROTO_SRCS} ${ADB_PAIRING_PROTO_HDRS})
+
+ target_compile_definitions(libadb PRIVATE -D_GNU_SOURCE)
+ target_compile_definitions(libadb PUBLIC -DADB_HOST=1)
+@@ -68,7 +59,8 @@ target_include_directories(libadb PUBLIC
+ add_library(libadb_crypto_defaults STATIC
+ core/adb/crypto/key.cpp
+ core/adb/crypto/rsa_2048_key.cpp
+- core/adb/crypto/x509_generator.cpp)
++ core/adb/crypto/x509_generator.cpp
++ ${ADB_KEY_TYPE_PROTO_HDRS})
+
+ target_include_directories(libadb_crypto_defaults PUBLIC
+ core/adb
+@@ -173,7 +165,8 @@ add_executable(adb
+ core/adb/transport_fd.cpp
+ core/adb/client/transport_local.cpp
+ core/adb/client/transport_usb.cpp
+- core/adb/types.cpp)
++ core/adb/types.cpp
++ ${ADB_KEY_TYPE_PROTO_HDRS})
+
+ target_compile_definitions(adb PRIVATE
+ -DPLATFORM_TOOLS_VERSION="${ANDROID_VERSION}"
diff --git a/srcpkgs/android-tools/template b/srcpkgs/android-tools/template
index 343a543a9a2..f4104cb07d1 100644
--- a/srcpkgs/android-tools/template
+++ b/srcpkgs/android-tools/template
@@ -1,20 +1,22 @@
# Template file for 'android-tools'
pkgname=android-tools
-version=29.0.6
-revision=3
+version=30.0.5
+revision=1
archs="armv* aarch64* x86_64* i686* ppc64le*"
build_style=cmake
-hostmakedepends="perl go"
-makedepends="gtest-devel zlib-devel libressl-devel libusb-devel pcre2-devel"
+hostmakedepends="perl go protobuf"
+makedepends="gtest-devel zlib-devel libressl-devel libusb-devel pcre2-devel
+ liblz4-devel libzstd-devel protobuf-devel brotli-devel"
short_desc="Android platform tools (adb and fastboot)"
maintainer="John <me@johnnynator.dev>"
license="Apache-2.0, ISC, GPL-2.0-only, MIT"
homepage="http://developer.android.com/tools/help/adb.html"
distfiles="https://github.com/nmeum/android-tools/releases/download/${version}/android-tools-${version}.tar.xz"
-checksum=7fb1c127c36b0752657593838b6823743bf8e5730f9f8b0f7ba2c185424cf376
+checksum=1b870a3f28d75e4907c95dbe8e71b4014f56be636d089a75257ef3922ad39d62
if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
makedepends+=" libatomic-devel"
+ configure_args="-DCMAKE_CXX_STANDARD_LIBRARIES=-latomic"
fi
pre_configure() {
@@ -31,16 +33,10 @@ pre_configure() {
_EOF
configure_args+=" ${boring_ssl_cmake_args}"
fi
-
- if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
- sed -i vendor/CMakeLists.adb.txt \
- -e "/target_link_libraries/s;$; atomic;"
- fi
}
post_install() {
vlicense vendor/boringssl/LICENSE boringssl.LICENSE
vlicense vendor/boringssl/third_party/fiat/LICENSE fiat.LICENSE
- vlicense vendor/boringssl/third_party/googletest/LICENSE gtest.LICENSE
vsv adb
}
diff --git a/srcpkgs/android-tools/update b/srcpkgs/android-tools/update
index 49895605477..f69b1910ad9 100644
--- a/srcpkgs/android-tools/update
+++ b/srcpkgs/android-tools/update
@@ -1,2 +1,2 @@
-site="https://android.googlesource.com/platform/system/core"
+site="https://android.googlesource.com/platform/system/core/+refs"
pattern='platform-tools-\K[\d._r]+'
From 3c0955916470d6195ccc8204a4edc8a8195277ab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
<congdanhqx@gmail.com>
Date: Mon, 25 Jan 2021 09:16:36 +0700
Subject: [PATCH 2/2] fixup! android-tools: update to 30.0.5.
---
srcpkgs/android-tools/template | 16 ----------------
1 file changed, 16 deletions(-)
diff --git a/srcpkgs/android-tools/template b/srcpkgs/android-tools/template
index f4104cb07d1..a75a37d7076 100644
--- a/srcpkgs/android-tools/template
+++ b/srcpkgs/android-tools/template
@@ -19,22 +19,6 @@ if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
configure_args="-DCMAKE_CXX_STANDARD_LIBRARIES=-latomic"
fi
-pre_configure() {
- # Don't check for CROSS_COMPILING, since i686 container/chroot inside
- # x86_64 host will failed to build this.
- if [ "${XBPS_TARGET_MACHINE}" = i686 ] && [ "$(uname -m)" = "x86_64" ]; then
- boring_ssl_cmake_args="-DCMAKE_TOOLCHAIN_FILE=cross_boring_ssl.cmake"
- cat > cross_boring_ssl.cmake <<-_EOF
- SET(CMAKE_SYSTEM_NAME Linux)
- SET(CMAKE_SYSTEM_VERSION 1)
- SET(CMAKE_C_COMPILER ${CC})
- SET(CMAKE_CXX_COMPILER ${CXX})
- SET(CMAKE_SYSTEM_PROCESSOR x86)
- _EOF
- configure_args+=" ${boring_ssl_cmake_args}"
- fi
-}
-
post_install() {
vlicense vendor/boringssl/LICENSE boringssl.LICENSE
vlicense vendor/boringssl/third_party/fiat/LICENSE fiat.LICENSE
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PR PATCH] [Updated] android-tools: update to 30.0.5.
2021-01-25 0:56 [PR PATCH] android-tools: update to 30.0.5 sgn
` (3 preceding siblings ...)
2021-01-25 2:18 ` sgn
@ 2021-01-25 16:26 ` sgn
2021-01-26 12:46 ` [PR PATCH] [Merged]: " sgn
5 siblings, 0 replies; 7+ messages in thread
From: sgn @ 2021-01-25 16:26 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1677 bytes --]
There is an updated pull request by sgn against master on the void-packages repository
https://github.com/sgn/void-packages android-tools-30.0.5
https://github.com/void-linux/void-packages/pull/28209
android-tools: update to 30.0.5.
<!-- Mark items with [x] where applicable -->
#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
The re-packaging efforts was done by me but I only used on x86_64-musl.
A patch file from https://github.com/void-linux/void-packages/pull/28209.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-android-tools-30.0.5-28209.patch --]
[-- Type: text/x-diff, Size: 6132 bytes --]
From b5b4cf98870f9f4c20ff1d9bb82b74f63fb33ca6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
<congdanhqx@gmail.com>
Date: Mon, 25 Jan 2021 07:41:21 +0700
Subject: [PATCH] android-tools: update to 30.0.5.
---
...s-fix-dependencies-on-protobuf-heade.patch | 63 +++++++++++++++++++
srcpkgs/android-tools/template | 34 +++-------
srcpkgs/android-tools/update | 2 +-
3 files changed, 71 insertions(+), 28 deletions(-)
create mode 100644 srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch
diff --git a/srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch b/srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch
new file mode 100644
index 00000000000..0d883b4f0fb
--- /dev/null
+++ b/srcpkgs/android-tools/patches/0001-vendor-CMakeLists-fix-dependencies-on-protobuf-heade.patch
@@ -0,0 +1,63 @@
+From 41f92b797aa6bf4609b6b31e42bd37df710ae7aa Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
+ <congdanhqx@gmail.com>
+Date: Mon, 25 Jan 2021 08:25:31 +0700
+Subject: [PATCH] vendor: CMakeLists: fix dependencies on protobuf header
+
+While we're at it, simplify OBJECT_DEPENDS as pointed out in
+https://cmake.org/cmake/help/v3.18/prop_sf/OBJECT_DEPENDS.html
+
+This fix a broken build because of a race as shown in:
+https://github.com/void-linux/void-packages/pull/28209/checks?check_run_id=1758983334
+---
+ vendor/CMakeLists.adb.txt | 23 ++++++++---------------
+ 1 file changed, 8 insertions(+), 15 deletions(-)
+
+diff --git vendor/CMakeLists.adb.txt vendor/CMakeLists.adb.txt
+index 1495a5d..5206a80 100644
+--- vendor/CMakeLists.adb.txt
++++ vendor/CMakeLists.adb.txt
+@@ -34,19 +34,10 @@ add_library(libadb STATIC
+ core/adb/sysdeps_unix.cpp
+ core/adb/sysdeps/errno.cpp
+ core/adb/sysdeps/posix/network.cpp
+- ${ADB_APP_PROCESSES_PROTO_SRCS}
+- ${ADB_KNOWN_HOSTS_PROTO_SRCS}
+- ${ADB_KEY_TYPE_PROTO_SRCS}
+- ${ADB_PAIRING_PROTO_SRCS})
+-
+-set_property(SOURCE core/adb/client/commandline.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_APP_PROCESSES_PROTO_HDRS})
+-set_property(SOURCE core/adb/client/adb_wifi.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_KNOWN_HOSTS_PROTO_HDRS})
+-set_property(SOURCE core/adb/client/auth.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_KEY_TYPE_PROTO_HDRS})
+-set_property(SOURCE core/adb/pairing_connection/pairing_connection.cpp
+- PROPERTY OBJECT_DEPENDS ${ADB_PAIRING_PROTO_HDRS})
++ ${ADB_APP_PROCESSES_PROTO_SRCS} ${ADB_APP_PROCESSES_PROTO_HDRS}
++ ${ADB_KNOWN_HOSTS_PROTO_SRCS} ${ADB_KNOWN_HOSTS_PROTO_HDRS}
++ ${ADB_KEY_TYPE_PROTO_SRCS} ${ADB_KEY_TYPE_PROTO_HDRS}
++ ${ADB_PAIRING_PROTO_SRCS} ${ADB_PAIRING_PROTO_HDRS})
+
+ target_compile_definitions(libadb PRIVATE -D_GNU_SOURCE)
+ target_compile_definitions(libadb PUBLIC -DADB_HOST=1)
+@@ -68,7 +59,8 @@ target_include_directories(libadb PUBLIC
+ add_library(libadb_crypto_defaults STATIC
+ core/adb/crypto/key.cpp
+ core/adb/crypto/rsa_2048_key.cpp
+- core/adb/crypto/x509_generator.cpp)
++ core/adb/crypto/x509_generator.cpp
++ ${ADB_KEY_TYPE_PROTO_HDRS})
+
+ target_include_directories(libadb_crypto_defaults PUBLIC
+ core/adb
+@@ -173,7 +165,8 @@ add_executable(adb
+ core/adb/transport_fd.cpp
+ core/adb/client/transport_local.cpp
+ core/adb/client/transport_usb.cpp
+- core/adb/types.cpp)
++ core/adb/types.cpp
++ ${ADB_KEY_TYPE_PROTO_HDRS})
+
+ target_compile_definitions(adb PRIVATE
+ -DPLATFORM_TOOLS_VERSION="${ANDROID_VERSION}"
diff --git a/srcpkgs/android-tools/template b/srcpkgs/android-tools/template
index 343a543a9a2..a75a37d7076 100644
--- a/srcpkgs/android-tools/template
+++ b/srcpkgs/android-tools/template
@@ -1,46 +1,26 @@
# Template file for 'android-tools'
pkgname=android-tools
-version=29.0.6
-revision=3
+version=30.0.5
+revision=1
archs="armv* aarch64* x86_64* i686* ppc64le*"
build_style=cmake
-hostmakedepends="perl go"
-makedepends="gtest-devel zlib-devel libressl-devel libusb-devel pcre2-devel"
+hostmakedepends="perl go protobuf"
+makedepends="gtest-devel zlib-devel libressl-devel libusb-devel pcre2-devel
+ liblz4-devel libzstd-devel protobuf-devel brotli-devel"
short_desc="Android platform tools (adb and fastboot)"
maintainer="John <me@johnnynator.dev>"
license="Apache-2.0, ISC, GPL-2.0-only, MIT"
homepage="http://developer.android.com/tools/help/adb.html"
distfiles="https://github.com/nmeum/android-tools/releases/download/${version}/android-tools-${version}.tar.xz"
-checksum=7fb1c127c36b0752657593838b6823743bf8e5730f9f8b0f7ba2c185424cf376
+checksum=1b870a3f28d75e4907c95dbe8e71b4014f56be636d089a75257ef3922ad39d62
if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
makedepends+=" libatomic-devel"
+ configure_args="-DCMAKE_CXX_STANDARD_LIBRARIES=-latomic"
fi
-pre_configure() {
- # Don't check for CROSS_COMPILING, since i686 container/chroot inside
- # x86_64 host will failed to build this.
- if [ "${XBPS_TARGET_MACHINE}" = i686 ] && [ "$(uname -m)" = "x86_64" ]; then
- boring_ssl_cmake_args="-DCMAKE_TOOLCHAIN_FILE=cross_boring_ssl.cmake"
- cat > cross_boring_ssl.cmake <<-_EOF
- SET(CMAKE_SYSTEM_NAME Linux)
- SET(CMAKE_SYSTEM_VERSION 1)
- SET(CMAKE_C_COMPILER ${CC})
- SET(CMAKE_CXX_COMPILER ${CXX})
- SET(CMAKE_SYSTEM_PROCESSOR x86)
- _EOF
- configure_args+=" ${boring_ssl_cmake_args}"
- fi
-
- if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
- sed -i vendor/CMakeLists.adb.txt \
- -e "/target_link_libraries/s;$; atomic;"
- fi
-}
-
post_install() {
vlicense vendor/boringssl/LICENSE boringssl.LICENSE
vlicense vendor/boringssl/third_party/fiat/LICENSE fiat.LICENSE
- vlicense vendor/boringssl/third_party/googletest/LICENSE gtest.LICENSE
vsv adb
}
diff --git a/srcpkgs/android-tools/update b/srcpkgs/android-tools/update
index 49895605477..f69b1910ad9 100644
--- a/srcpkgs/android-tools/update
+++ b/srcpkgs/android-tools/update
@@ -1,2 +1,2 @@
-site="https://android.googlesource.com/platform/system/core"
+site="https://android.googlesource.com/platform/system/core/+refs"
pattern='platform-tools-\K[\d._r]+'
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PR PATCH] [Merged]: android-tools: update to 30.0.5.
2021-01-25 0:56 [PR PATCH] android-tools: update to 30.0.5 sgn
` (4 preceding siblings ...)
2021-01-25 16:26 ` sgn
@ 2021-01-26 12:46 ` sgn
5 siblings, 0 replies; 7+ messages in thread
From: sgn @ 2021-01-26 12:46 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1516 bytes --]
There's a merged pull request on the void-packages repository
android-tools: update to 30.0.5.
https://github.com/void-linux/void-packages/pull/28209
Description:
<!-- Mark items with [x] where applicable -->
#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
The re-packaging efforts was done by me but I only used on x86_64-musl.
^ permalink raw reply [flat|nested] 7+ messages in thread