You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2022/01/05 20:27:34 UTC

[trafficserver] branch 9.1.x updated: Partial of revert "Cleanup generated LDFLAGS for jemalloc (#8285)" (#8533)

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 9.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.1.x by this push:
     new 3d84d92  Partial of revert "Cleanup generated LDFLAGS for jemalloc (#8285)" (#8533)
3d84d92 is described below

commit 3d84d92a4d3454ebb0b2780b43acdd1428bbf208
Author: Randall Meyer <rr...@apache.org>
AuthorDate: Tue Dec 7 09:12:42 2021 -0800

    Partial of revert "Cleanup generated LDFLAGS for jemalloc (#8285)" (#8533)
    
    This reverts commit 9b3cde501d9f72d250a34f6d4742c816e1e7797b.
    
    Restoring --add-needed, but keeping the corrected path
    
    (cherry picked from commit 8de830b0bb3ec1f8acf37b79990aa229f9713de8)
---
 build/jemalloc.m4 | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/build/jemalloc.m4 b/build/jemalloc.m4
index e341f2f..ba70c90 100644
--- a/build/jemalloc.m4
+++ b/build/jemalloc.m4
@@ -55,7 +55,8 @@ if test "$enable_jemalloc" != "no"; then
   jemalloc_have_libs=0
   if test "$jemalloc_base_dir" != "/usr"; then
     TS_ADDTO(CPPFLAGS, [-I${jemalloc_include}])
-    TS_ADDTO(LDFLAGS, [-L${jemalloc_ldflags} -Wl,-rpath,${jemalloc_ldflags}])
+    TS_ADDTO(LDFLAGS, [-L${jemalloc_ldflags}])
+    TS_ADDTO(LDFLAGS, [-Wl,--add-needed -L${jemalloc_ldflags} -Wl,-rpath,${jemalloc_ldflags} -Wl,--no-as-needed])
     TS_ADDTO_RPATH(${jemalloc_ldflags})
   fi
   # On Darwin, jemalloc symbols are prefixed with je_. Search for that first, then fall back