cmcfarlen commented on PR #12201:
URL: https://github.com/apache/trafficserver/pull/12201#issuecomment-2835881366
Hi! This doesn't compile for me. I added a line to
`include/tscore/ink_config.cmake.in` so `HAVE_ZSTD_H` would be truthy for the
CPP macros. This lit up that code, but then there were several compiler errors:
```
/Users/cmcfarlen/projects/oss/trafficserver/plugins/compress/compress.cc:203:9:
error: no member named 'zstd_ctx' in 'Data'; did you mean 'zstd_cctx'?
203 | data->zstd_ctx = nullptr;
| ^~~~~~~~
| zstd_cctx
/Users/cmcfarlen/projects/oss/trafficserver/plugins/compress/misc.h:93:14:
note: 'zstd_cctx' declared here
93 | ZSTD_CCtx *zstd_cctx;
| ^
/Users/cmcfarlen/projects/oss/trafficserver/plugins/compress/compress.cc:206:11:
error: no member named 'zstd_ctx' in 'Data'; did you mean 'zstd_cctx'?
206 | data->zstd_ctx = ZSTD_createCCtx();
| ^~~~~~~~
| zstd_cctx
/Users/cmcfarlen/projects/oss/trafficserver/plugins/compress/misc.h:93:14:
note: 'zstd_cctx' declared here
93 | ZSTD_CCtx *zstd_cctx;
| ^
/Users/cmcfarlen/projects/oss/trafficserver/plugins/compress/compress.cc:207:16:
error: no member named 'zstd_ctx' in 'Data'; did you mean 'zstd_cctx'?
207 | if (!data->zstd_ctx) {
| ^~~~~~~~
| zstd_cctx
/Users/cmcfarlen/projects/oss/trafficserver/plugins/compress/misc.h:93:14:
note: 'zstd_cctx' declared here
93 | ZSTD_CCtx *zstd_cctx;
| ^
```
Please include this patch in your PR to enable the ZSTD code. Thanks!
```
diff --git a/include/tscore/ink_config.h.cmake.in
b/include/tscore/ink_config.h.cmake.in
index 634ed94c3..7da0771fd 100644
--- a/include/tscore/ink_config.h.cmake.in
+++ b/include/tscore/ink_config.h.cmake.in
@@ -184,3 +184,5 @@ const int DEFAULT_STACKSIZE = @DEFAULT_STACK_SIZE@;
#cmakedefine YAMLCPP_LIB_VERSION "@YAMLCPP_LIB_VERSION@"
#cmakedefine01 TS_HAS_CRIPTS
+
+#cmakedefine HAVE_ZSTD_H 1
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
