diff mbox series

[meta-oe] libziparchive: fix missing fixed-width integer types in zip_writer.h

Message ID 20251029070240.3402018-1-quic_sasikuma@quicinc.com
State Under Review
Headers show
Series [meta-oe] libziparchive: fix missing fixed-width integer types in zip_writer.h | expand

Commit Message

Sasi Kumar Maddineni Oct. 29, 2025, 7:02 a.m. UTC
Compilation fails due to undefined types 'uint16_t' and 'uint32_t' in
zip_writer.h. These types are used in struct members but are not recognized
because the required header <cstdint> is not included. This results in
errors such as:
  error: 'uint16_t' does not name a type
  error: 'uint32_t' does not name a type

Added '#include <cstdint>' to zip_writer.h to ensure fixed-width integer
types are properly defined.

Signed-off-by: Sasi Kumar Maddineni <quic_sasikuma@quicinc.com>
---
 ...x-missing-fixed-width-integer-types-.patch | 38 +++++++++++++++++++
 .../android-tools/android-tools_29.0.6.r14.bb |  1 +
 2 files changed, 39 insertions(+)
 create mode 100644 meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools/debian/system/core/libziparchive-fix-missing-fixed-width-integer-types-.patch

Comments

Khem Raj Oct. 29, 2025, 5:21 p.m. UTC | #1
Which branch is it for ? it does not seem to be for master, if it is
then please rebase on latest master, secondly the subject line should
have name of recipe ( android-tools )
not the subcomponent ( libziparchive ) to denote the change

Its a good idea to read through -
https://docs.yoctoproject.org/contributor-guide/recipe-style-guide.html

On Wed, Oct 29, 2025 at 5:44 AM Sasi Kumar Maddineni via
lists.openembedded.org
<quic_sasikuma=quicinc.com@lists.openembedded.org> wrote:
>
> Compilation fails due to undefined types 'uint16_t' and 'uint32_t' in
> zip_writer.h. These types are used in struct members but are not recognized
> because the required header <cstdint> is not included. This results in
> errors such as:
>   error: 'uint16_t' does not name a type
>   error: 'uint32_t' does not name a type
>
> Added '#include <cstdint>' to zip_writer.h to ensure fixed-width integer
> types are properly defined.
>
> Signed-off-by: Sasi Kumar Maddineni <quic_sasikuma@quicinc.com>
> ---
>  ...x-missing-fixed-width-integer-types-.patch | 38 +++++++++++++++++++
>  .../android-tools/android-tools_29.0.6.r14.bb |  1 +
>  2 files changed, 39 insertions(+)
>  create mode 100644 meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools/debian/system/core/libziparchive-fix-missing-fixed-width-integer-types-.patch
>
> diff --git a/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools/debian/system/core/libziparchive-fix-missing-fixed-width-integer-types-.patch b/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools/debian/system/core/libziparchive-fix-missing-fixed-width-integer-types-.patch
> new file mode 100644
> index 0000000000..0b5087f283
> --- /dev/null
> +++ b/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools/debian/system/core/libziparchive-fix-missing-fixed-width-integer-types-.patch
> @@ -0,0 +1,38 @@
> +From f6874a2f7de30b6eeefb2d509b895c58b93da8f7 Mon Sep 17 00:00:00 2001
> +From: Sasi Kumar Maddineni <quic_sasikuma@quicinc.com>
> +Date: Wed, 29 Oct 2025 09:38:17 +0530
> +Subject: [PATCH] libziparchive: fix missing fixed-width integer types in
> + zip_writer.h
> +
> +Compilation fails due to undefined types 'uint16_t' and 'uint32_t' in
> +zip_writer.h. These types are used in struct members but are not recognized
> +because the required header <cstdint> is not included. This results in
> +errors such as:
> +  error: 'uint16_t' does not name a type
> +  error: 'uint32_t' does not name a type
> +
> +Added '#include <cstdint>' to zip_writer.h to ensure fixed-width integer
> +types are properly defined.
> +
> +Upstream-Status: Pending
> +
> +Signed-off-by: Sasi Kumar Maddineni <quic_sasikuma@quicinc.com>
> +---
> + system/core/libziparchive/include/ziparchive/zip_writer.h | 1 +
> + 1 file changed, 1 insertion(+)
> +
> +diff --git a/system/core/libziparchive/include/ziparchive/zip_writer.h b/system/core/libziparchive/include/ziparchive/zip_writer.h
> +index d68683df..7a05e349 100644
> +--- a/system/core/libziparchive/include/ziparchive/zip_writer.h
> ++++ b/system/core/libziparchive/include/ziparchive/zip_writer.h
> +@@ -18,6 +18,7 @@
> +
> + #include <cstdio>
> + #include <ctime>
> ++#include <cstdint>
> +
> + #include <gtest/gtest_prod.h>
> + #include <memory>
> +--
> +2.34.1
> +
> diff --git a/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools_29.0.6.r14.bb b/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools_29.0.6.r14.bb
> index 47c8e08b62..74928ed171 100644
> --- a/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools_29.0.6.r14.bb
> +++ b/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools_29.0.6.r14.bb
> @@ -40,6 +40,7 @@ SRC_URI += " \
>      file://debian/system/core/fix-standard-namespace-errors.patch \
>      file://debian/system/core/Add-riscv64-support.patch \
>      file://debian/system/core/android-tools-scope-write-macro-to-prevent-conflict-.patch \
> +    file://debian/system/core/libziparchive-fix-missing-fixed-width-integer-types-.patch \
>      \
>  "
>
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#121150): https://lists.openembedded.org/g/openembedded-devel/message/121150
> Mute This Topic: https://lists.openembedded.org/mt/116011337/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools/debian/system/core/libziparchive-fix-missing-fixed-width-integer-types-.patch b/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools/debian/system/core/libziparchive-fix-missing-fixed-width-integer-types-.patch
new file mode 100644
index 0000000000..0b5087f283
--- /dev/null
+++ b/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools/debian/system/core/libziparchive-fix-missing-fixed-width-integer-types-.patch
@@ -0,0 +1,38 @@ 
+From f6874a2f7de30b6eeefb2d509b895c58b93da8f7 Mon Sep 17 00:00:00 2001
+From: Sasi Kumar Maddineni <quic_sasikuma@quicinc.com>
+Date: Wed, 29 Oct 2025 09:38:17 +0530
+Subject: [PATCH] libziparchive: fix missing fixed-width integer types in
+ zip_writer.h
+
+Compilation fails due to undefined types 'uint16_t' and 'uint32_t' in
+zip_writer.h. These types are used in struct members but are not recognized
+because the required header <cstdint> is not included. This results in
+errors such as:
+  error: 'uint16_t' does not name a type
+  error: 'uint32_t' does not name a type
+
+Added '#include <cstdint>' to zip_writer.h to ensure fixed-width integer
+types are properly defined.
+
+Upstream-Status: Pending
+
+Signed-off-by: Sasi Kumar Maddineni <quic_sasikuma@quicinc.com>
+---
+ system/core/libziparchive/include/ziparchive/zip_writer.h | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/system/core/libziparchive/include/ziparchive/zip_writer.h b/system/core/libziparchive/include/ziparchive/zip_writer.h
+index d68683df..7a05e349 100644
+--- a/system/core/libziparchive/include/ziparchive/zip_writer.h
++++ b/system/core/libziparchive/include/ziparchive/zip_writer.h
+@@ -18,6 +18,7 @@
+ 
+ #include <cstdio>
+ #include <ctime>
++#include <cstdint>
+ 
+ #include <gtest/gtest_prod.h>
+ #include <memory>
+-- 
+2.34.1
+
diff --git a/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools_29.0.6.r14.bb b/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools_29.0.6.r14.bb
index 47c8e08b62..74928ed171 100644
--- a/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools_29.0.6.r14.bb
+++ b/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools_29.0.6.r14.bb
@@ -40,6 +40,7 @@  SRC_URI += " \
     file://debian/system/core/fix-standard-namespace-errors.patch \
     file://debian/system/core/Add-riscv64-support.patch \
     file://debian/system/core/android-tools-scope-write-macro-to-prevent-conflict-.patch \
+    file://debian/system/core/libziparchive-fix-missing-fixed-width-integer-types-.patch \
     \
 "