All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 6/6] remove superfluous OPTIMIZE and DEBUG config knobs
Date: Tue, 6 Oct 2009 22:35:58 +0200	[thread overview]
Message-ID: <20091006223558.73d0678f@surf> (raw)
In-Reply-To: <20091006184939.GD10261@mx.loc>

Le Tue, 6 Oct 2009 20:49:39 +0200,
Bernhard Reutner-Fischer <rep.dot.nop@gmail.com> a ?crit :

> The help-text of BR2_TARGET_OPTIMIZATION reads:
> ---8<---
> Optimizations to use when building for the target host.
> ---8<---
> 
> If you want to build with -O0 -g3 -ggdb3 , then just set it to that.
> Adding knobs for each -Ox and -gx is not userfriendly but odd, given
> that there always was BR2_TARGET_OPTIMIZATION and that this is (or
> should be anyway) used _everywhere_.

Hum, you're right. But I think I would still like to keep a "enable
debug" option that would :

 * Compile with -g and keep the unstripped binary in the staging/
   directory

 * Strip the binaries in target/

This is the normal setup needed to do remote debugging: stripped
binaries on the target, unstripped binaries with debugging symbols to
feed gdb on the host.

I don't have a very strong opinion on this, but I think having an
option for debugging/non-debugging is useful (and debugging + stripping
should probably be the default). But if we decide to get rid of the
option, some documentation would probably be enough.

Sincerly,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers and embedded Linux development,
consulting, training and support.
http://free-electrons.com

  reply	other threads:[~2009-10-06 20:35 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-10-06 18:26 [Buildroot] [PATCH 0/6] misc config-cleanup Bernhard Reutner-Fischer
2009-10-06 18:26 ` [Buildroot] [PATCH 1/6] atmel mirrors are only needed for atmel targets Bernhard Reutner-Fischer
2009-10-06 18:26   ` [Buildroot] [PATCH 2/6] always append FPU_SUFFIX Bernhard Reutner-Fischer
2009-10-06 18:26     ` [Buildroot] [PATCH 3/6] default to EABI on arm Bernhard Reutner-Fischer
2009-10-06 18:26       ` [Buildroot] [PATCH 4/6] improve arch selection and gcc setup Bernhard Reutner-Fischer
2009-10-06 18:26         ` [Buildroot] [PATCH 5/6] mark odd strip invocations Bernhard Reutner-Fischer
2009-10-06 18:26           ` [Buildroot] [PATCH 6/6] remove superfluous OPTIMIZE and DEBUG config knobs Bernhard Reutner-Fischer
2009-10-06 18:41             ` Peter Korsgaard
2009-10-06 18:49               ` Bernhard Reutner-Fischer
2009-10-06 20:35                 ` Thomas Petazzoni [this message]
2009-10-07  6:47                   ` Bernhard Reutner-Fischer
2009-10-07  7:03                     ` Peter Korsgaard
2009-10-07  7:14                       ` Bernhard Reutner-Fischer
2009-10-06 18:34           ` [Buildroot] [PATCH 5/6] mark odd strip invocations Peter Korsgaard
2009-10-06 18:42             ` Bernhard Reutner-Fischer
2009-10-06 18:51               ` Peter Korsgaard
2009-10-06 20:01                 ` Olaf Rempel
2009-10-07  6:41                   ` Bernhard Reutner-Fischer
2009-10-07  7:51                   ` Peter Korsgaard
2009-10-06 18:36         ` [Buildroot] [PATCH 4/6] improve arch selection and gcc setup Peter Korsgaard
2009-10-06 18:44           ` Bernhard Reutner-Fischer
2009-10-06 18:51             ` Peter Korsgaard
2009-10-07 18:26               ` Bernhard Reutner-Fischer
2009-10-06 18:35       ` [Buildroot] [PATCH 3/6] default to EABI on arm Peter Korsgaard
2009-10-06 21:07       ` Peter Korsgaard
2009-10-06 18:34     ` [Buildroot] [PATCH 2/6] always append FPU_SUFFIX Peter Korsgaard
2009-10-06 18:40       ` Bernhard Reutner-Fischer
2009-10-06 18:47         ` Peter Korsgaard
2009-10-06 20:31           ` Thomas Petazzoni
2009-10-06 20:30       ` Thomas Petazzoni
2009-10-06 18:32   ` [Buildroot] [PATCH 1/6] atmel mirrors are only needed for atmel targets Peter Korsgaard
2009-10-06 21:07   ` Peter Korsgaard
2009-10-06 21:13     ` [Buildroot] [PATCH 1/6] atmel mirrors are only needed for atmeltargets H Hartley Sweeten
2009-10-06 21:28       ` Peter Korsgaard
2009-10-06 18:32 ` [Buildroot] [PATCH 0/6] misc config-cleanup Peter Korsgaard
2009-10-06 20:31   ` Thomas Petazzoni

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=20091006223558.73d0678f@surf \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /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.