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

bneradt pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/trafficserver-ci.git


The following commit(s) were added to refs/heads/main by this push:
     new 2ae0f7d  Remove the setting of LD_LIBRARY_PATH for boring (#201)
2ae0f7d is described below

commit 2ae0f7d89f1d6822ba7a001a84e7c06afe346139
Author: Brian Neradt <brian.ner...@gmail.com>
AuthorDate: Fri Jul 28 09:56:58 2023 -0500

    Remove the setting of LD_LIBRARY_PATH for boring (#201)
    
    Now that Damian's rpath fix for 
https://github.com/cloudflare/quiche/pull/1508
    has landed, we no longer need to explicitly set LD_LIBRARY_PATH for our
    quiche builds.
---
 jenkins/branch/quiche.pipeline | 9 ---------
 jenkins/github/rocky.pipeline  | 5 -----
 2 files changed, 14 deletions(-)

diff --git a/jenkins/branch/quiche.pipeline b/jenkins/branch/quiche.pipeline
index b483a2f..0ed6d0b 100644
--- a/jenkins/branch/quiche.pipeline
+++ b/jenkins/branch/quiche.pipeline
@@ -70,10 +70,6 @@ pipeline {
 
                                        source /opt/rh/gcc-toolset-11/enable
 
-                                       # We'll need this until Damian's quiche 
patch lands which updates rpath:
-                                       # 
https://github.com/cloudflare/quiche/pull/1508
-                                       export 
LD_LIBRARY_PATH=/opt/boringssl/lib64:${LD_LIBRARY_PATH}
-
                                        # Change permissions so that all files 
are readable
                                        # (default user umask may change and 
make these unreadable)
                                        sudo chmod -R o+r .
@@ -101,7 +97,6 @@ pipeline {
                                                set -x
                                                set -e
                                                source 
/opt/rh/gcc-toolset-11/enable
-                                               export 
LD_LIBRARY_PATH=/opt/boringssl/lib64:${LD_LIBRARY_PATH}
                                                make -j4 V=1 Q= check
                                                /tmp/ats/bin/traffic_server -K 
-k -R 1
                                                '''
@@ -120,10 +115,6 @@ pipeline {
                                                export PATH=/opt/bin:${PATH}
                                                export PATH=/opt/go/bin:${PATH}
 
-                                               # We'll need this until 
Damian's quiche patch lands which updates rpath:
-                                               # 
https://github.com/cloudflare/quiche/pull/1508
-                                               export 
LD_LIBRARY_PATH=/opt/boringssl/lib64:${LD_LIBRARY_PATH}
-
                                                
export_dir="${WORKSPACE}/output/${GITHUB_PR_BRANCH}"
                                                mkdir -p ${export_dir}
 
diff --git a/jenkins/github/rocky.pipeline b/jenkins/github/rocky.pipeline
index 5c81ecf..ff60e71 100644
--- a/jenkins/github/rocky.pipeline
+++ b/jenkins/github/rocky.pipeline
@@ -48,11 +48,6 @@ pipeline {
                         set -x
                         set -e
                         source /opt/rh/gcc-toolset-11/enable
-
-                        # We'll need this until Damian's quiche patch lands 
which updates rpath:
-                        # https://github.com/cloudflare/quiche/pull/1508
-                        export 
LD_LIBRARY_PATH=/opt/boringssl/lib64:${LD_LIBRARY_PATH}
-
                         autoreconf -fiv
                         ./configure \
                           --with-quiche=/opt/quiche \

Reply via email to