about summary refs log tree commit diff
diff options
context:
space:
mode:
authorJesus Checa Hidalgo <jchecahi@redhat.com>2025-04-11 16:57:38 +0200
committerJesus Checa Hidalgo <jchecahi@redhat.com>2025-04-11 16:57:38 +0200
commitdc0fbcab7e0673afe62b3e8e74905d9e5f5b74a4 (patch)
tree38e7624574c1f0808492e285a70bd059a1638834
parent71b68da1bd9fa6afb9f964a731e9c843ab0862bd (diff)
downloadrust-dc0fbcab7e0673afe62b3e8e74905d9e5f5b74a4.tar.gz
rust-dc0fbcab7e0673afe62b3e8e74905d9e5f5b74a4.zip
Fix profiler_builtins build script to handle full path to profiler lib
LLVM_PROFILER_RT_LIB may be set to an absolute path (e.g., in Fedora builds),
but `-l` expects a library name, not a path. After #138273, this caused builds
to fail with a "could not find native static library" error.

This patch updates the build script to split the path into directory and
filename, using `cargo::rustc-link-search` for the directory and
`cargo::rustc-link-lib=+verbatim` for the file. This allows profiler_builtins to
correctly link the static library even when an absolute path is provided.
-rw-r--r--library/profiler_builtins/build.rs10
1 files changed, 8 insertions, 2 deletions
diff --git a/library/profiler_builtins/build.rs b/library/profiler_builtins/build.rs
index dd85239fa8c..fc1a9ecc1ec 100644
--- a/library/profiler_builtins/build.rs
+++ b/library/profiler_builtins/build.rs
@@ -9,8 +9,14 @@ use std::path::PathBuf;
 
 fn main() {
     if let Ok(rt) = tracked_env_var("LLVM_PROFILER_RT_LIB") {
-        println!("cargo::rustc-link-lib=static:+verbatim={rt}");
-        return;
+        let rt = PathBuf::from(rt);
+        if let Some(lib) = rt.file_name() {
+            if let Some(dir) = rt.parent() {
+                println!("cargo::rustc-link-search=native={}", dir.display());
+            }
+            println!("cargo::rustc-link-lib=static:+verbatim={}", lib.to_str().unwrap());
+            return;
+        }
     }
 
     let target_os = env::var("CARGO_CFG_TARGET_OS").expect("CARGO_CFG_TARGET_OS was not set");