Source: android-platform-frameworks-base
Version: 1:10.0.0+r36-5
Severity: important
Usertags: protobuf3_20
Tags: ftbfs upstream bookworm sid patch
Hi,
I would like to start the Protobuf 3.20.1 transition in a few days.
Your package is currently FTBFS for a simple reason. The function
SetTotalBytesLimit doesn't have a second argument for long (protobuf
3.6) and it was ignored previously. Now it's finally removed and hence
your package doesn't build anymore.
As it was ignored for a long time, the fix is easy, just remove that
argument when calling the mentioned function. Patch is attached,
please apply it soon.
Thanks,
Laszlo/GCS
Description: fix Protobuf FTBFS
SetTotalBytesLimit no longer has second argument.
Author: Laszlo Boszormenyi (GCS) <g...@debian.org>
Forwarded: no
Last-Update: 2022-06-06
---
--- android-platform-frameworks-base-10.0.0+r36.orig/tools/aapt2/io/Util.h
+++ android-platform-frameworks-base-10.0.0+r36/tools/aapt2/io/Util.h
@@ -131,8 +131,7 @@ class ProtoInputStreamReader {
template <typename T> bool ReadMessage(T *message_lite) {
ZeroCopyInputAdaptor adapter(in_);
google::protobuf::io::CodedInputStream coded_stream(&adapter);
- coded_stream.SetTotalBytesLimit(std::numeric_limits<int32_t>::max(),
- coded_stream.BytesUntilTotalBytesLimit());
+ coded_stream.SetTotalBytesLimit(std::numeric_limits<int32_t>::max());
return message_lite->ParseFromCodedStream(&coded_stream);
}
