nix/pkgs/envoy: fix under new nixpkgs
This commit is contained in:
parent
db5e1646d7
commit
0c713bbf64
2 changed files with 101 additions and 3 deletions
|
@ -0,0 +1,91 @@
|
||||||
|
From 8b531c41f956b27e4be32b430db2e7a44e0cdd3e Mon Sep 17 00:00:00 2001
|
||||||
|
From: Luke Granger-Brown <git@lukegb.com>
|
||||||
|
Date: Thu, 7 Jan 2021 11:09:18 +0000
|
||||||
|
Subject: [PATCH] Add upb patch to make it compile under GCC10
|
||||||
|
|
||||||
|
---
|
||||||
|
bazel/repositories.bzl | 5 +++-
|
||||||
|
bazel/upb2.patch | 55 ++++++++++++++++++++++++++++++++++++++++++
|
||||||
|
2 files changed, 59 insertions(+), 1 deletion(-)
|
||||||
|
create mode 100644 bazel/upb2.patch
|
||||||
|
|
||||||
|
diff --git a/bazel/repositories.bzl b/bazel/repositories.bzl
|
||||||
|
index 64d61ea49..c6cadc9df 100644
|
||||||
|
--- a/bazel/repositories.bzl
|
||||||
|
+++ b/bazel/repositories.bzl
|
||||||
|
@@ -811,7 +811,10 @@ def _com_github_grpc_grpc():
|
||||||
|
def _upb():
|
||||||
|
_repository_impl(
|
||||||
|
name = "upb",
|
||||||
|
- patches = ["@envoy//bazel:upb.patch"],
|
||||||
|
+ patches = [
|
||||||
|
+ "@envoy//bazel:upb.patch",
|
||||||
|
+ "@envoy//bazel:upb2.patch",
|
||||||
|
+ ],
|
||||||
|
patch_args = ["-p1"],
|
||||||
|
)
|
||||||
|
|
||||||
|
diff --git a/bazel/upb2.patch b/bazel/upb2.patch
|
||||||
|
new file mode 100644
|
||||||
|
index 000000000..6e436c61b
|
||||||
|
--- /dev/null
|
||||||
|
+++ b/bazel/upb2.patch
|
||||||
|
@@ -0,0 +1,55 @@
|
||||||
|
+From 9bd23dab4240b015321a53c45b3c9e4847fbf020 Mon Sep 17 00:00:00 2001
|
||||||
|
+From: Joshua Haberman <jhaberman@gmail.com>
|
||||||
|
+Date: Tue, 7 Apr 2020 15:22:11 -0700
|
||||||
|
+Subject: [PATCH] Changed upb status to suit GCC10's warning about strncpy().
|
||||||
|
+ (#268)
|
||||||
|
+
|
||||||
|
+Added tests for all cases. Also removed ellipses from truncated
|
||||||
|
+messages, they were more trouble than they are worth.
|
||||||
|
+---
|
||||||
|
+ tests/test_generated_code.c | 33 +++++++++++++++++++++++++++++++++
|
||||||
|
+ upb/upb.c | 17 +++--------------
|
||||||
|
+ 2 files changed, 36 insertions(+), 14 deletions(-)
|
||||||
|
+
|
||||||
|
+diff --git a/upb/upb.c b/upb/upb.c
|
||||||
|
+index cb2cdfd9d..258192d79 100644
|
||||||
|
+--- a/upb/upb.c
|
||||||
|
++++ b/upb/upb.c
|
||||||
|
+@@ -11,17 +11,6 @@
|
||||||
|
+
|
||||||
|
+ #include "upb/port_def.inc"
|
||||||
|
+
|
||||||
|
+-/* Guarantee null-termination and provide ellipsis truncation.
|
||||||
|
+- * It may be tempting to "optimize" this by initializing these final
|
||||||
|
+- * four bytes up-front and then being careful never to overwrite them,
|
||||||
|
+- * this is safer and simpler. */
|
||||||
|
+-static void nullz(upb_status *status) {
|
||||||
|
+- const char *ellipsis = "...";
|
||||||
|
+- size_t len = strlen(ellipsis);
|
||||||
|
+- UPB_ASSERT(sizeof(status->msg) > len);
|
||||||
|
+- memcpy(status->msg + sizeof(status->msg) - len, ellipsis, len);
|
||||||
|
+-}
|
||||||
|
+-
|
||||||
|
+ /* upb_status *****************************************************************/
|
||||||
|
+
|
||||||
|
+ void upb_status_clear(upb_status *status) {
|
||||||
|
+@@ -37,8 +26,8 @@ const char *upb_status_errmsg(const upb_status *status) { return status->msg; }
|
||||||
|
+ void upb_status_seterrmsg(upb_status *status, const char *msg) {
|
||||||
|
+ if (!status) return;
|
||||||
|
+ status->ok = false;
|
||||||
|
+- strncpy(status->msg, msg, sizeof(status->msg));
|
||||||
|
+- nullz(status);
|
||||||
|
++ strncpy(status->msg, msg, UPB_STATUS_MAX_MESSAGE - 1);
|
||||||
|
++ status->msg[UPB_STATUS_MAX_MESSAGE - 1] = '\0';
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ void upb_status_seterrf(upb_status *status, const char *fmt, ...) {
|
||||||
|
+@@ -52,7 +41,7 @@ void upb_status_vseterrf(upb_status *status, const char *fmt, va_list args) {
|
||||||
|
+ if (!status) return;
|
||||||
|
+ status->ok = false;
|
||||||
|
+ _upb_vsnprintf(status->msg, sizeof(status->msg), fmt, args);
|
||||||
|
+- nullz(status);
|
||||||
|
++ status->msg[UPB_STATUS_MAX_MESSAGE - 1] = '\0';
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ /* upb_alloc ******************************************************************/
|
||||||
|
--
|
||||||
|
2.29.2
|
||||||
|
|
|
@ -11,7 +11,7 @@ with pkgs;
|
||||||
owner = "envoyproxy";
|
owner = "envoyproxy";
|
||||||
repo = "envoy";
|
repo = "envoy";
|
||||||
rev = "v${version}";
|
rev = "v${version}";
|
||||||
hash = "sha256:1b6nplf0pby1yjnsawbr5a6cnvs2ss38kxccgs4fvxv4b1m8h5d9";
|
hash = "sha256:1afk55ysjr0kl9vap6fjlqcg2ldzm18v4xdb5i2srig531czn9jy";
|
||||||
|
|
||||||
extraPostFetch = ''
|
extraPostFetch = ''
|
||||||
chmod -R +w $out
|
chmod -R +w $out
|
||||||
|
@ -24,6 +24,7 @@ with pkgs;
|
||||||
|
|
||||||
patches = [
|
patches = [
|
||||||
./0001-quiche-update-QUICHE-tar-13949.patch
|
./0001-quiche-update-QUICHE-tar-13949.patch
|
||||||
|
./0002-Add-upb-patch-to-make-it-compile-under-GCC10.patch
|
||||||
];
|
];
|
||||||
|
|
||||||
nativeBuildInputs = [
|
nativeBuildInputs = [
|
||||||
|
@ -34,7 +35,7 @@ with pkgs;
|
||||||
];
|
];
|
||||||
|
|
||||||
fetchAttrs = {
|
fetchAttrs = {
|
||||||
sha256 = "sha256:0s8gzfmpwjrv5v1l9spbkpa03x561n98qwli7qbbq4rvdsq1j45p";
|
sha256 = "sha256:1i96r6zza7kqmfsyh2x4rlkd1bf4hsf52asmch8ryi8iqr4z6m9c";
|
||||||
dontUseCmakeConfigure = true;
|
dontUseCmakeConfigure = true;
|
||||||
};
|
};
|
||||||
buildAttrs = {
|
buildAttrs = {
|
||||||
|
@ -53,7 +54,13 @@ with pkgs;
|
||||||
removeLocalConfigCc = false;
|
removeLocalConfigCc = false;
|
||||||
removeLocal = false;
|
removeLocal = false;
|
||||||
bazelTarget = "//source/exe:envoy-static";
|
bazelTarget = "//source/exe:envoy-static";
|
||||||
bazelBuildFlags = [ "-c opt" "--spawn_strategy=standalone" "--verbose_failures" "--noexperimental_strict_action_env" ''--cxxopt=-Wno-maybe-uninitialized'' ];
|
bazelBuildFlags = [
|
||||||
|
"-c opt"
|
||||||
|
"--spawn_strategy=standalone"
|
||||||
|
"--noexperimental_strict_action_env"
|
||||||
|
"--cxxopt=-Wno-maybe-uninitialized"
|
||||||
|
"--cxxopt=-Wno-uninitialized"
|
||||||
|
];
|
||||||
|
|
||||||
meta = with stdenv.lib; {
|
meta = with stdenv.lib; {
|
||||||
homepage = "https://envoyproxy.io";
|
homepage = "https://envoyproxy.io";
|
||||||
|
|
Loading…
Reference in a new issue