diff mbox series

[meta-arago,master] ti-llvm: Fix for buildpaths errors

Message ID 20240719025328.28669-1-reatmon@ti.com
State Accepted
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,master] ti-llvm: Fix for buildpaths errors | expand

Commit Message

Ryan Eatmon July 19, 2024, 2:53 a.m. UTC
Make sure that llvm-config does not have any absolute build paths within
the compiled binary.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 meta-arago-extras/recipes-core/llvm/ti-llvm3.6_3.6.bb | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Denys Dmytriyenko July 20, 2024, 6 p.m. UTC | #1
On Thu, Jul 18, 2024 at 09:53:28PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> Make sure that llvm-config does not have any absolute build paths within
> the compiled binary.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  meta-arago-extras/recipes-core/llvm/ti-llvm3.6_3.6.bb | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/meta-arago-extras/recipes-core/llvm/ti-llvm3.6_3.6.bb b/meta-arago-extras/recipes-core/llvm/ti-llvm3.6_3.6.bb
> index 4b990b7f..395651d3 100644
> --- a/meta-arago-extras/recipes-core/llvm/ti-llvm3.6_3.6.bb
> +++ b/meta-arago-extras/recipes-core/llvm/ti-llvm3.6_3.6.bb
> @@ -72,6 +72,10 @@ do_compile:class-native() {
>  }
>  
>  do_compile:class-nativesdk() {
> +    # Fix for buildpaths error referring to TMPDIR
> +    sed -i 's:@LLVM_SRC_ROOT@::g' ${S}/tools/llvm-config/BuildVariables.inc.in
> +    sed -i 's:@LLVM_OBJ_ROOT@::g' ${S}/tools/llvm-config/BuildVariables.inc.in

Yeah, I was also looking at patching this .in file, but got pre-occupied with 
upstream breakage lately... Thanks for taking care of this.


>      cd ${LLVM_BUILD_DIR}
>  
>      # Fix libdir for multilib
> -- 
> 2.17.1
diff mbox series

Patch

diff --git a/meta-arago-extras/recipes-core/llvm/ti-llvm3.6_3.6.bb b/meta-arago-extras/recipes-core/llvm/ti-llvm3.6_3.6.bb
index 4b990b7f..395651d3 100644
--- a/meta-arago-extras/recipes-core/llvm/ti-llvm3.6_3.6.bb
+++ b/meta-arago-extras/recipes-core/llvm/ti-llvm3.6_3.6.bb
@@ -72,6 +72,10 @@  do_compile:class-native() {
 }
 
 do_compile:class-nativesdk() {
+    # Fix for buildpaths error referring to TMPDIR
+    sed -i 's:@LLVM_SRC_ROOT@::g' ${S}/tools/llvm-config/BuildVariables.inc.in
+    sed -i 's:@LLVM_OBJ_ROOT@::g' ${S}/tools/llvm-config/BuildVariables.inc.in
+
     cd ${LLVM_BUILD_DIR}
 
     # Fix libdir for multilib