Bug#1012576: onnx: FTBFS with protobuf 3.20.1+

[László Böszörményi (GCS) <gcs@debian.org>]

Source: onnx
Version: 1.7.0+dfsg-3.1
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) <gcs@debian.org>
Forwarded: no
Last-Update: 2022-06-06

---

--- onnx-1.7.0+dfsg.orig/onnx/proto_utils.h
+++ onnx-1.7.0+dfsg/onnx/proto_utils.h
@@ -31,7 +31,7 @@ bool ParseProtoFromBytes(Proto* proto, c
   // respectively.
   ::google::protobuf::io::ArrayInputStream input_stream(buffer, static_cast<int>(length));
   ::google::protobuf::io::CodedInputStream coded_stream(&input_stream);
-  coded_stream.SetTotalBytesLimit((2048LL << 20) - 1, 512LL << 20);
+  coded_stream.SetTotalBytesLimit((2048LL << 20) - 1);
   return proto->ParseFromCodedStream(&coded_stream);
 }