Re: [pushed] Darwin, libffi, testsuite: Ensure we pick up the convenience lib.
Hi Rainer, > On 22 Jan 2023, at 09:44, Rainer Orth wrote: > >> Tested on i686, powerpc, x86_64, Arm64 Darwin, x86_64-linux-gnu, >> pushed to trunk, thanks, >> Iain >> >> --- 8< --- >> >> For several Darwin linker versions, the link paths are first traversed >> looking for shared libraries, and then again looking for archives. >> >> We want the paths to be searched for both shared and archives in order >> so that we will find the non-shared libffi in the testsuite (otherwise >> the installed version in the system will be found instead). >> >> Signed-off-by: Iain Sandoe >> >> libffi/ChangeLog: >> >> * testsuite/lib/libffi.exp: Search for both shared and archive >> in each path in turn. > > shouldn't this go upstream first? I was regarding this as specific to GCC since we build without the shared library now (but I guess upstream could also be configured with —disable-shared). so I will make a pull request, thanks for the note. Iain
Re: [pushed] Darwin, libffi, testsuite: Ensure we pick up the convenience lib.
Hi Iain, > Tested on i686, powerpc, x86_64, Arm64 Darwin, x86_64-linux-gnu, > pushed to trunk, thanks, > Iain > > --- 8< --- > > For several Darwin linker versions, the link paths are first traversed > looking for shared libraries, and then again looking for archives. > > We want the paths to be searched for both shared and archives in order > so that we will find the non-shared libffi in the testsuite (otherwise > the installed version in the system will be found instead). > > Signed-off-by: Iain Sandoe > > libffi/ChangeLog: > > * testsuite/lib/libffi.exp: Search for both shared and archive > in each path in turn. shouldn't this go upstream first? Rainer -- - Rainer Orth, Center for Biotechnology, Bielefeld University
[pushed] Darwin, libffi, testsuite: Ensure we pick up the convenience lib.
Tested on i686, powerpc, x86_64, Arm64 Darwin, x86_64-linux-gnu,
pushed to trunk, thanks,
Iain
--- 8< ---
For several Darwin linker versions, the link paths are first traversed
looking for shared libraries, and then again looking for archives.
We want the paths to be searched for both shared and archives in order
so that we will find the non-shared libffi in the testsuite (otherwise
the installed version in the system will be found instead).
Signed-off-by: Iain Sandoe
libffi/ChangeLog:
* testsuite/lib/libffi.exp: Search for both shared and archive
in each path in turn.
---
libffi/testsuite/lib/libffi.exp | 1 +
1 file changed, 1 insertion(+)
diff --git a/libffi/testsuite/lib/libffi.exp b/libffi/testsuite/lib/libffi.exp
index 7dc7b5d19a3..15d3d5ebd73 100644
--- a/libffi/testsuite/lib/libffi.exp
+++ b/libffi/testsuite/lib/libffi.exp
@@ -384,6 +384,7 @@ proc libffi_target_compile { source dest type options } {
if { [istarget "*-*-darwin9*"] || [istarget "*-*-darwin1*"]
|| [istarget "*-*-darwin2*"] } {
lappend options "additional_flags=-Wl,-allow_stack_execute"
+ lappend options "additional_flags=-Wl,-search_paths_first"
}
# If you're building the compiler with --prefix set to a place
--
2.37.1 (Apple Git-137.1)
