On Fri, Jul 17, 2020 at 3:24 PM Niels Möller nisse@lysator.liu.se wrote:
Jeffrey Walton noloader@gmail.com writes:
Hi Everyone,
I build OpenSSH for downlevel machines, like OS X and Solaris. I install into /opt/ssh, and I use a runpath of $ORIGIN/../lib. The LDFLAGS are:
-Wl,-runpath,'$ORIGIN/../lib' -Wl,-runpath,$(prefix)/lib-Wl,--enable-new-dtags
I noticed Nettle does not handle the ORIGIN-based runpath properly:
/opt/ssh/lib/libhogweed.so.6: RUNPATH: RIGIN/../lib:/opt/ssh/lib RPATH: RIGIN/../lib:/opt/ssh/lib
Indeed looks like missing a quote/escape problem. Does it work with other packages using autconf? How? I would suggest adding the quotes in the input, i.e., set
LDFLAGS='-Wl,-runpath,'$$ORIGIN/../lib' -Wl,-runpath,$(prefix)/lib ...'
when running configure.
Since you specify $(prefix), a Makefile-level substitution, and expect that to work, it seems reasonable to me to use Makefile-style syntax, including needed escapes, also for $ORIGIN.
I believe the fix is to escape the dollar sign in the makefile. That is, when Nettle creates its makefiles, it must use:
-Wl,-runpath,'$$ORIGIN/../lib' ...Where do you suggest that substitution be made? And how would Nettle's configure script or Makefile know that you intend the $ in '$ORIGIN' to be escaped, but not the $ in '$(prefix)' ?
I admit I'm quite skeptical, and I also think it's important to stay consistent with how LDFLAGS is handled in other GNU packages. But if you can suggest a good way to do it, I'm happy to listen.
You have to fix the makefiles. If you escape the dollar signs in LDFLAGS, then Autotools will fail its conftests because they don't use a makefile. Autotools conftests use CFLAGS, LDFLAGS (and friends) directly.
Run this after you write out your Makefiles. You can probably add it to configure.ac since configure.ac is just another script.
# We want the leading single quote, and the trailing slash. origin1=$(echo "'"'$ORIGIN/' | sed -e 's/[/&]/\&/g') origin2=$(echo "'"'$$ORIGIN/' | sed -e 's/[/&]/\&/g')
IFS="" find "./" -name 'Makefile' -print | while read -r file do chmod a+w "$file" sed -e "s/$origin1/$origin2/g" \ -e "s/GZIP_ENV = --best/GZIP_ENV = -9/g" \ "$file" > "$file.fixed" mv "$file.fixed" "$file" chmod a-w "$file" done
You can add additional expressions to the sed for $LIB and $PLATFORM.
Don't use 'sed -i' because it is not portable and fails on AIX and OS X (and possibly others, like the BSDs and BusyBox). Sed to a new file, and then move the new file to the old file.
The Gzip expression is due to '--best'. That's a GNU extension, and fails for BusyBox and other non-GNU systems, like Alpine Linux.
Jeff