mailing list of musl libc
 help / color / mirror / code / Atom feed
From: Isaac Dunham <idunham@lavabit.com>
To: musl@lists.openwall.com
Subject: Re: Pending issues for next release
Date: Wed, 3 Apr 2013 21:00:00 -0700	[thread overview]
Message-ID: <20130403210000.b60ba35a.idunham@lavabit.com> (raw)
In-Reply-To: <20130401233144.GA29587@brightrain.aerifal.cx>

[-- Attachment #1: Type: text/plain, Size: 400 bytes --]

On Mon, 1 Apr 2013 19:31:44 -0400
Rich Felker <dalias@aerifal.cx> wrote:

> Here's a quick list of open issues that we might want to take care of
> before another release:
>
(snipping several interesting improvements) 
> - switching libc.so/ld-musl.so symlink direction

Does this patch look suitable?
I know it does work, including in the DESTDIR=./.../ case.
-- 
Isaac Dunham <idunham@lavabit.com>

[-- Attachment #2: buildsys.diff --]
[-- Type: text/x-diff, Size: 871 bytes --]

diff --git a/Makefile b/Makefile
index 997c5bb..478e698 100644
--- a/Makefile
+++ b/Makefile
@@ -118,7 +118,10 @@ tools/musl-gcc: config.mak
 $(DESTDIR)$(bindir)/%: tools/%
 	install -D $< $@
 
-$(DESTDIR)$(libdir)/%.so: lib/%.so
+$(DESTDIR)$(libdir)/libc.so: $(DESTDIR)$(LDSO_PATHNAME)
+	ln -sf $(LDSO_PATHNAME) $@
+
+$(DESTDIR)$(LDSO_PATHNAME): lib/libc.so
 	install -D -m 755 $< $@
 
 $(DESTDIR)$(libdir)/%: lib/%
@@ -127,12 +130,6 @@ $(DESTDIR)$(libdir)/%: lib/%
 $(DESTDIR)$(includedir)/%: include/%
 	install -D -m 644 $< $@
 
-$(DESTDIR)$(LDSO_PATHNAME): $(DESTDIR)$(syslibdir)
-	ln -sf $(libdir)/libc.so $@ || true
-
-$(DESTDIR)$(syslibdir):
-	install -d -m 755 $(DESTDIR)$(syslibdir)
-
 install-libs: $(ALL_LIBS:lib/%=$(DESTDIR)$(libdir)/%) $(if $(SHARED_LIBS),$(DESTDIR)$(LDSO_PATHNAME),)
 
 install-headers: $(ALL_INCLUDES:include/%=$(DESTDIR)$(includedir)/%)

  parent reply	other threads:[~2013-04-04  4:00 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-01 23:31 Rich Felker
2013-04-02 11:11 ` Szabolcs Nagy
2013-04-04 23:37   ` Rich Felker
2013-04-05  0:41     ` Rich Felker
2013-04-05  0:48     ` Isaac Dunham
2013-04-02 11:42 ` Jens Gustedt
2013-04-02 18:09   ` Rich Felker
2013-04-02 19:40     ` Jens Gustedt
2013-04-03  1:37       ` Rich Felker
2013-04-04  4:00 ` Isaac Dunham [this message]
2013-04-04 23:04   ` Rich Felker
2013-04-09 17:19     ` Rich Felker
2013-04-04 23:28 ` Rich Felker
2013-04-06 21:28 ` Update (Re: [musl] Pending issues for next release) Rich Felker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130403210000.b60ba35a.idunham@lavabit.com \
    --to=idunham@lavabit.com \
    --cc=musl@lists.openwall.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/musl/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).