All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH 05/12] no need to set LD nowadays
Date: Tue, 9 Feb 2010 09:50:19 +0100	[thread overview]
Message-ID: <20100209085019.GA1663@mx.loc> (raw)
In-Reply-To: <1265666186.6761.3.camel@lenovo.internal.reciva.com>

On Mon, Feb 08, 2010 at 09:56:26PM +0000, Phil Blundell wrote:
>On Mon, 2010-02-08 at 22:14 +0100, Bernhard Reutner-Fischer wrote:
>>  # Use gcc for linking so LDFLAGS actually makes sense
>> -LD = "${CC} -nostdlib"
>> +#LD = "${CC} -nostdlib"
>>  
>
>Thanks for the patch.  When you say "nowadays", do you know which actual
>versions of busybox this is true for?  Is this definitely correct for

busybox >= 1.4.0 switched to kbuild, i.e.
make CROSS_COMPILE=alpha-linux- EXTRA_CFLAGS=-fwhatever \
EXTRA_LDFLAGS=-Wl,--warn-multiple-gp

For the usual case you'd either set CONFIG_CROSS_COMPILER_PREFIX in the
.config or pass CC down to make.

If you happen to have an odd host compiler then you want those too:
HOSTCC=ccc HOSTCFLAGS="-ieee" make ....

>all the versions that we have in the .dev tree?

It seems that the .dev tree has 1.2.1, 1.2.2, 1.3.1 still in there but
i, personally would strongly advise against using those (same for the
standalone tinylogin, but that's a different thing and not relevant
right now).
>
>Also, if that line is genuinely no longer required, you might as well
>just delete it.  There is no point in leaving commented-out cruft in
>these recipes indefinitely.

The ancient cruft versions made me leave in that comment.



  reply	other threads:[~2010-02-09  8:51 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-08 21:14 [PATCH 0/12] random misc trivia Bernhard Reutner-Fischer
2010-02-08 21:14 ` [PATCH 01/12] trim superfluous trailing whitespace Bernhard Reutner-Fischer
2010-02-08 21:14   ` [PATCH 02/12] some more !getopt_long fixes Bernhard Reutner-Fischer
2010-02-08 21:14     ` [PATCH 03/12] osso-application-installer: nonworking Bernhard Reutner-Fischer
2010-02-08 21:14       ` [PATCH 04/12] matchbox-panel-2: fix DEPENDS Bernhard Reutner-Fischer
2010-02-08 21:14         ` [PATCH 05/12] no need to set LD nowadays Bernhard Reutner-Fischer
2010-02-08 21:14           ` [PATCH 06/12] quilt: use builtin date routines Bernhard Reutner-Fischer
2010-02-08 21:14             ` [PATCH 07/12] Document proxy setup in local.conf.sample Bernhard Reutner-Fischer
2010-02-08 21:14               ` [PATCH 08/12] commentary typo fix Bernhard Reutner-Fischer
2010-02-08 21:14                 ` [PATCH 09/12] flex: remove version specifics from generic inc Bernhard Reutner-Fischer
2010-02-08 21:14                   ` [PATCH 10/12] git: " Bernhard Reutner-Fischer
2010-02-08 21:14                     ` [PATCH 11/12] fix binutils patch to apply to 2.20 Bernhard Reutner-Fischer
2010-02-08 21:14                       ` [PATCH 12/12] task-base.bb: use uniform test for set Bernhard Reutner-Fischer
2010-02-11 13:04                         ` Phil Blundell
2010-02-11 21:57                           ` Khem Raj
2010-02-09 11:49                   ` [PATCH 09/12] flex: remove version specifics from generic inc Phil Blundell
2010-02-11  9:04               ` [PATCH 07/12] Document proxy setup in local.conf.sample Denys Dmytriyenko
2010-02-13 12:08                 ` Bernhard Reutner-Fischer
2010-02-15 20:14                   ` Denys Dmytriyenko
2010-02-09 11:06             ` [PATCH 06/12] quilt: use builtin date routines Phil Blundell
2010-02-08 21:56           ` [PATCH 05/12] no need to set LD nowadays Phil Blundell
2010-02-09  8:50             ` Bernhard Reutner-Fischer [this message]
2010-02-09  8:56               ` Phil Blundell
2010-02-10 16:48                 ` Bernhard Reutner-Fischer
2010-02-09  8:10           ` Koen Kooi
2010-02-09 11:58         ` [PATCH 04/12] matchbox-panel-2: fix DEPENDS Phil Blundell
2010-02-10 16:51           ` Bernhard Reutner-Fischer
2010-02-10 21:11       ` [PATCH 03/12] osso-application-installer: nonworking Bernhard Reutner-Fischer
2010-02-09  9:38     ` [PATCH 02/12] some more !getopt_long fixes Jens Seidel
2010-02-10  8:54       ` Bernhard Reutner-Fischer
2010-02-09 12:05   ` [PATCH 01/12] trim superfluous trailing whitespace Phil Blundell
2010-02-10 22:48   ` [PATCH 00/12] misc trivia Bernhard Reutner-Fischer

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=20100209085019.GA1663@mx.loc \
    --to=rep.dot.nop@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /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.
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.