grpc 1.70.1

grpc.changes

@@ -1,3 +1,14 @@
+-------------------------------------------------------------------
+Sun Feb  9 22:37:18 UTC 2025 - Jan Engelhardt <jengelh@inai.de>
+
+- Update to release 1.70.1
+  * grpc++ begins to require C++17 or later to build and use it
+  * chttp2_server: fix a race between connection starting and it
+    being orphaned
+  * chttp2Server: fix a race between connection manager updates
+    and handshake
+- Add link-failure.patch, return-values.patch
+
 -------------------------------------------------------------------
 Sat Apr 27 09:40:46 UTC 2024 - Jan Engelhardt <jengelh@inai.de>
 

grpc.spec

@@ -16,11 +16,11 @@
 #
 
 
-%define lver 40
-%define lverp 1_63
+%define lver 45
+%define lverp 1_70
 %define src_install_dir /usr/src/%name
 Name:           grpc
-Version:        1.63.0
+Version:        1.70.1
 Release:        0
 Summary:        HTTP/2-based Remote Procedure Call implementation
 License:        Apache-2.0
@@ -29,8 +29,10 @@ URL:            https://grpc.io/
 Source:         https://github.com/grpc/grpc/archive/v%version.tar.gz
 Source2:        %name-rpmlintrc
 Patch1:         terminate.patch
-Patch4:         ARM-Unaligned-access-fixes.patch
-Patch5:         Fix-compilation-on-RHEL-7-ppc64le-gcc-4.8.patch
+Patch2:         link-failure.patch
+Patch3:         return-values.patch
+Patch14:        ARM-Unaligned-access-fixes.patch
+Patch15:        Fix-compilation-on-RHEL-7-ppc64le-gcc-4.8.patch
 BuildRequires:  abseil-cpp-devel
 BuildRequires:  cmake
 BuildRequires:  fdupes
@@ -84,7 +86,7 @@ Summary:        A small protobuf implementation in C
 Group:          System/Libraries
 
 %description -n libupb%lver
-μpb (often written 'upb') is a small protobuf implementation written in C.
+μpb (often written "upb") is a small protobuf implementation written in C.
 
 upb generates a C API for creating, parsing, and serializing messages as
 declared in .proto files. upb is heavily arena-based: all messages always live
@@ -110,7 +112,7 @@ Group:          Development/Tools/Building
 Requires:       libupb%lver = %version
 
 %description -n upb-devel
-μpb (often written 'upb') is a small protobuf implementation written in C.
+μpb (often written "upb") is a small protobuf implementation written in C.
 
 upb generates a C API for creating, parsing, and serializing messages as
 declared in .proto files. upb is heavily arena-based: all messages always live
@@ -128,7 +130,7 @@ This subpackage contains source code of the gRPC reference implementation.
 
 %prep
 %autosetup -N
-%patch -P 1 -p1
+%patch -P 1 -P 2 -P 3 -p1
 find "." -type f -exec grep -l '/usr/bin/python' {} + |
 	xargs -r perl -i -lpe \
 	's{#! ?/usr/bin/python\S*}{#!/usr/bin/python3}g;'
@@ -139,12 +141,12 @@ find "." -type f -exec grep -l '/usr/bin/env ' {} + |
 	 s{#! ?/usr/bin/env bash}{#!/bin/bash}g;
 	 s{#! ?/usr/bin/env }{#!/usr/bin/}g;'
 pushd third_party/xxhash
-%patch -P 4 -P 5 -p1
+%patch -P 14 -P 15 -p1
 popd
 rm -Rf third_party/abseil-cpp/
 
 %build
-%if 0%{?suse_version} < 1550
+%if 0%{?suse_version} < 1600
 export CC=gcc-12
 export CXX=g++-12
 %endif

link-failure.patch

@@ -0,0 +1,33 @@
+From: Jan Engelhardt <ej@inai.de>
+Date: 2025-02-09 18:51:55.470853259 +0100
+
+happened since about 1.63 (last known good 1.62.x)
+
+[    9s] /usr/bin/cc -fPIC -O2 -Wall -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=3 -fstack-protector-strong -funwind-tables -fasynchronous-unwind-tables -fstack-clash-protection -Werror=return-type  -g -Wno-error  -O2 -g -DNDEBUG -Wl,--dependency-file=CMakeFiles/upb_textformat_lib.dir/link.d  -Wl,--as-needed -Wl,--no-undefined -Wl,-z,now -shared -Wl,-soname,libupb_textformat_lib.so.45 -o libupb_textformat_lib.so.45.0.0 CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/lex/atoi.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/lex/round_trip.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/lex/strtod.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/lex/unicode.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/message/copy.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/message/merge.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/def_pool.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/def_type.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/desc_state.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/enum_def.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/enum_reserved_range.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/enum_value_def.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/extension_range.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/field_def.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/file_def.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/internal/def_builder.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/internal/strdup2.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/message.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/message_def.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/message_reserved_range.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/method_def.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/oneof_def.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/reflection/service_def.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/text/encode.c.o CMakeFiles/upb_textformat_lib.dir/third_party/upb/upb/text/internal/encode.c.o  -Wl,-rpath,/home/abuild/rpmbuild/BUILD/grpc-1.70.1-build/grpc-1.70.1/build: -ldl -lm -lrt libupb_mini_descriptor_lib.so.45.0.0 libupb_wire_lib.so.45.0.0 libutf8_range_lib.so.45.0.0 libupb_message_lib.so.45.0.0 libupb_base_lib.so.45.0.0 libupb_mem_lib.so.45.0.0 -ldl -lm -lrt
+[    9s] /home/abuild/rpmbuild/BUILD/grpc-1.70.1-build/grpc-1.70.1/src/core/ext/upb-gen/google/protobuf/descriptor.upb.h:5195:(.text+0x3b0): undefined reference to `google__protobuf__ServiceOptions_msg_init'
+[    9s] collect2: error: ld returned 1 exit status
+
+---
+ CMakeLists.txt |    2 ++
+ 1 file changed, 2 insertions(+)
+
+Index: grpc-1.70.1/CMakeLists.txt
+===================================================================
+--- grpc-1.70.1.orig/CMakeLists.txt
++++ grpc-1.70.1/CMakeLists.txt
+@@ -4091,6 +4091,7 @@ target_link_libraries(upb_json_lib
+   ${_gRPC_ALLTARGETS_LIBRARIES}
+   upb_mini_descriptor_lib
+   upb_wire_lib
++  grpc_unsecure
+ )
+ 
+ 
+@@ -4348,6 +4349,7 @@ target_link_libraries(upb_textformat_lib
+   ${_gRPC_ALLTARGETS_LIBRARIES}
+   upb_mini_descriptor_lib
+   upb_wire_lib
++  grpc_unsecure
+ )
+ 
+ 

return-values.patch

@@ -0,0 +1,54 @@
+From: Jan Engelhardt <ej@inai.de>
+Date: 2025-02-09 21:28:54.517217439 +0100
+
+Fix some terrible code.
+
+[   28s] /home/abuild/rpmbuild/BUILD/grpc-1.70.1-build/grpc-1.70.1/src/core/lib/transport/call_state.h:157:3: error: control reaches end of non-void function [-Werror=return-type]
+---
+ src/core/lib/transport/call_state.h |    5 +++++
+ 1 file changed, 5 insertions(+)
+
+Index: grpc-1.70.1/src/core/lib/transport/call_state.h
+===================================================================
+--- grpc-1.70.1.orig/src/core/lib/transport/call_state.h
++++ grpc-1.70.1/src/core/lib/transport/call_state.h
+@@ -154,6 +154,7 @@ class CallState {
+       case ClientToServerPullState::kTerminated:
+         return "Terminated";
+     }
++    return "Undefined";
+   }
+   template <typename Sink>
+   friend void AbslStringify(Sink& out, ClientToServerPullState state) {
+@@ -184,6 +185,7 @@ class CallState {
+       case ClientToServerPushState::kFinished:
+         return "Finished";
+     }
++    return "Undefined";
+   }
+   template <typename Sink>
+   friend void AbslStringify(Sink& out, ClientToServerPushState state) {
+@@ -234,6 +236,7 @@ class CallState {
+       case ServerToClientPullState::kTerminated:
+         return "Terminated";
+     }
++    return "Undefined";
+   }
+   template <typename Sink>
+   friend void AbslStringify(Sink& out, ServerToClientPullState state) {
+@@ -274,6 +277,7 @@ class CallState {
+       case ServerToClientPushState::kFinished:
+         return "Finished";
+     }
++    return "Undefined";
+   }
+   template <typename Sink>
+   friend void AbslStringify(Sink& out, ServerToClientPushState state) {
+@@ -304,6 +308,7 @@ class CallState {
+       case ServerTrailingMetadataState::kPulledCancel:
+         return "PulledCancel";
+     }
++    return "Undefined";
+   }
+   template <typename Sink>
+   friend void AbslStringify(Sink& out, ServerTrailingMetadataState state) {