From: ludo@gnu.org (Ludovic Courtès)
To: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Cc: Guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH] gnu: gcj: remove broken symlink and stump binaries.
Date: Thu, 30 Apr 2015 18:05:45 +0200 [thread overview]
Message-ID: <87pp6ly52e.fsf@gnu.org> (raw)
In-Reply-To: <idjfv7heifn.fsf@bimsb-sys02.mdc-berlin.net> (Ricardo Wurmus's message of "Thu, 30 Apr 2015 17:37:16 +0200")
Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> skribis:
> This patch adds a build phase to GCJ removing a broken symlink and a
> couple of non-functional binaries, such as "gcc" or "g++".
>
> In a GCJ build these binaries are useless as GCJ is a GCC built only
> with Java support. When GCJ is installed alongside gcc-toolchain, for
> example, there would be conflicts with GCJ's "gcc" stump binary
> shadowing the usable "gcc" binary from gcc-toolchain.
>
> Removing these binaries should be harmless as users of GCJ are unlikely
> to use "gcc" but "gcj" directly.
Sounds good.
> From 99b56cf4b17b1dbed7977dbce6e3c81fa4115aa7 Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
> Date: Thu, 30 Apr 2015 17:11:39 +0200
> Subject: [PATCH] gnu: gcj: Remove broken symlink and conflicting files.
>
> * gnu/packages/gcc.scm (gcj)[arguments]: Add a build phase to remove a broken
> symlink and generic stump binaries.
[...]
> + (for-each delete-file
> + (map (cut string-append out <>)
> + '("/lib/jvm/jre/lib/amd64/libjawt.so"
> + "/bin/c++"
> + "/bin/cpp"
> + "/bin/g++"
> + "/bin/gcc"
> + "/bin/gcc-ar"
> + "/bin/gcc-nm"
> + "/bin/gcc-ranlib"
> + "/bin/x86_64-unknown-linux-gnu-c++"
> + "/bin/x86_64-unknown-linux-gnu-g++"
> + "/bin/x86_64-unknown-linux-gnu-gcc"
> + "/bin/x86_64-unknown-linux-gnu-gcc-4.8.4"
> + "/bin/x86_64-unknown-linux-gnu-gcc-ar"
> + "/bin/x86_64-unknown-linux-gnu-gcc-nm"
> + "/bin/x86_64-unknown-linux-gnu-gcc-ranlib"))))
This is specific to x86_64-linux-gnu though. Wouldn’t it be better to
use a white list instead, or maybe keep it a black list but use
find-files and regexps?
Thanks,
Ludo’.
next prev parent reply other threads:[~2015-04-30 16:05 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-30 15:37 [PATCH] gnu: gcj: remove broken symlink and stump binaries Ricardo Wurmus
2015-04-30 16:05 ` Ludovic Courtès [this message]
2015-04-30 16:29 ` Ricardo Wurmus
2015-05-02 16:14 ` Ricardo Wurmus
2015-05-02 18:26 ` Ludovic Courtès
2015-05-02 19:09 ` Ricardo Wurmus
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=87pp6ly52e.fsf@gnu.org \
--to=ludo@gnu.org \
--cc=guix-devel@gnu.org \
--cc=ricardo.wurmus@mdc-berlin.de \
/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 external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.