Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-10-27 Thread John Baldwin
On Tuesday, October 27, 2015 06:09:37 PM Ed Maste wrote: > On 11 February 2015 at 21:39, Glen Barber wrote: > > Hi, > > > > Within the next 24 hours, I will merge the release-install-debug branch > > into head, which will enable building and installing stripped debugging > > files by default. > >

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-10-27 Thread Ed Maste
On 11 February 2015 at 21:39, Glen Barber wrote: > Hi, > > Within the next 24 hours, I will merge the release-install-debug branch > into head, which will enable building and installing stripped debugging > files by default. > > In general, this should have no significant impact, but any fallout w

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Ed Maste
On 12 February 2015 at 10:29, Steve Kargl wrote: > > Does this feature take into account the optimization level > that one may set via CFLAGS? Conventional wisdom used to > claim that debuggers did not fair too well with code that > was compiled with -O2 (or -Os and/or other well-known gcc > opti

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Glen Barber
On Thu, Feb 12, 2015 at 04:30:58PM +, Glen Barber wrote: > On Wed, Feb 11, 2015 at 08:56:00PM -0700, Ian Lepore wrote: > > How much time will this add to a build? > > > > I did not keep the build log files from the initial universe build with > DEBUG_FILES enabled, so cannot yet provide a gra

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Glen Barber
On Wed, Feb 11, 2015 at 08:56:00PM -0700, Ian Lepore wrote: > How much time will this add to a build? > I did not keep the build log files from the initial universe build with DEBUG_FILES enabled, so cannot yet provide a granular overview of the build times at the moment unfortunately. I will pr

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Glen Barber
On Thu, Feb 12, 2015 at 02:39:12AM +, Glen Barber wrote: > Within the next 24 hours, I will merge the release-install-debug branch > into head, which will enable building and installing stripped debugging > files by default. > Given the feedback received, I will not be merging this branch to

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Steve Kargl
On Thu, Feb 12, 2015 at 02:39:12AM +, Glen Barber wrote: > Hi, > > Within the next 24 hours, I will merge the release-install-debug branch > into head, which will enable building and installing stripped debugging > files by default. > > In general, this should have no significant impact, but

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Ed Maste
On 11 February 2015 at 23:23, Ian Lepore wrote: > > I can't imagine that anybody is going to be happy with an installed > system size increase from 520 to 2520 MB no matter how much it helps > debugging, especially considering the the typical installation media is > in the 2-8 GB range (with lots

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Glen Barber
On Thu, Feb 12, 2015 at 09:27:53AM -0500, Ed Maste wrote: > On 11 February 2015 at 23:43, Alfred Perlstein wrote: > > > > Agreed, SMALL platforms should turn off at decision of those platform > > maintainers. Won't fight to keep debug default on for arm, that's for sure. > > Actually, it's even

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Ed Maste
On 11 February 2015 at 23:43, Alfred Perlstein wrote: > > Agreed, SMALL platforms should turn off at decision of those platform > maintainers. Won't fight to keep debug default on for arm, that's for sure. Actually, it's even more important that we build and store debug files for ARM releases. C

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Ed Maste
On 12 February 2015 at 08:21, Glen Barber wrote: > On Thu, Feb 12, 2015 at 11:12:19AM +0200, Konstantin Belousov wrote: >> >> IMO having debugging information for all binaries is just an overkill. >> Nobody debugs ls on regular basis, while debug information for C runtime >> is needed often. >> >

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Glen Barber
On Thu, Feb 12, 2015 at 11:12:19AM +0200, Konstantin Belousov wrote: > On Thu, Feb 12, 2015 at 02:39:12AM +, Glen Barber wrote: > > Hi, > > > > Within the next 24 hours, I will merge the release-install-debug branch > > into head, which will enable building and installing stripped debugging >

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread David Chisnall
On 12 Feb 2015, at 04:49, Glen Barber wrote: > > I would like to see it enabled by default for 11.0-RELEASE. For everyone concerned about size, remember that the goal for 11.0-RELEASE is to use pkg for the base system, which makes it much easier to: - Install some debug info later, on demand.

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Andreas Nilsson
On Thu, Feb 12, 2015 at 10:12 AM, Konstantin Belousov wrote: > On Thu, Feb 12, 2015 at 02:39:12AM +, Glen Barber wrote: > > Hi, > > > > Within the next 24 hours, I will merge the release-install-debug branch > > into head, which will enable building and installing stripped debugging > > files

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-12 Thread Konstantin Belousov
On Thu, Feb 12, 2015 at 02:39:12AM +, Glen Barber wrote: > Hi, > > Within the next 24 hours, I will merge the release-install-debug branch > into head, which will enable building and installing stripped debugging > files by default. > > In general, this should have no significant impact, but

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-11 Thread Glen Barber
On Thu, Feb 12, 2015 at 12:46:18PM +0800, Jia-Shiun Li wrote: > On Thu, Feb 12, 2015 at 12:11 PM, Glen Barber wrote: > > > The major benefit is that all debugging data that we need to properly > > debug application crashes in the base system will be available > > out-of-box. > > > > There is a tr

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-11 Thread Jia-Shiun Li
On Thu, Feb 12, 2015 at 12:11 PM, Glen Barber wrote: > The major benefit is that all debugging data that we need to properly > debug application crashes in the base system will be available > out-of-box. > > There is a trade-off here, in both directions. For arm, for example, > the trade-off is

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-11 Thread Alfred Perlstein
On 2/11/15 8:28 PM, Glen Barber wrote: On Wed, Feb 11, 2015 at 09:23:01PM -0700, Ian Lepore wrote: On Thu, 2015-02-12 at 04:11 +, Glen Barber wrote: On Wed, Feb 11, 2015 at 08:56:00PM -0700, Ian Lepore wrote: On Wed, 2015-02-11 at 22:21 -0500, Ed Maste wrote: On 11 February 2015 at 21:3

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-11 Thread Glen Barber
On Wed, Feb 11, 2015 at 09:23:01PM -0700, Ian Lepore wrote: > On Thu, 2015-02-12 at 04:11 +, Glen Barber wrote: > > On Wed, Feb 11, 2015 at 08:56:00PM -0700, Ian Lepore wrote: > > > On Wed, 2015-02-11 at 22:21 -0500, Ed Maste wrote: > > > > On 11 February 2015 at 21:39, Glen Barber wrote: > >

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-11 Thread Glen Barber
On Thu, Feb 12, 2015 at 04:11:58AM +, Glen Barber wrote: > On Wed, Feb 11, 2015 at 08:56:00PM -0700, Ian Lepore wrote: > > On Wed, 2015-02-11 at 22:21 -0500, Ed Maste wrote: > > > On 11 February 2015 at 21:39, Glen Barber wrote: > > > > > > > > Within the next 24 hours, I will merge the releas

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-11 Thread Ian Lepore
On Thu, 2015-02-12 at 04:11 +, Glen Barber wrote: > On Wed, Feb 11, 2015 at 08:56:00PM -0700, Ian Lepore wrote: > > On Wed, 2015-02-11 at 22:21 -0500, Ed Maste wrote: > > > On 11 February 2015 at 21:39, Glen Barber wrote: > > > > > > > > Within the next 24 hours, I will merge the release-insta

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-11 Thread Glen Barber
On Wed, Feb 11, 2015 at 08:56:00PM -0700, Ian Lepore wrote: > On Wed, 2015-02-11 at 22:21 -0500, Ed Maste wrote: > > On 11 February 2015 at 21:39, Glen Barber wrote: > > > > > > Within the next 24 hours, I will merge the release-install-debug branch > > > into head, which will enable building and

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-11 Thread Ian Lepore
On Wed, 2015-02-11 at 22:21 -0500, Ed Maste wrote: > On 11 February 2015 at 21:39, Glen Barber wrote: > > > > Within the next 24 hours, I will merge the release-install-debug branch > > into head, which will enable building and installing stripped debugging > > files by default. > > > > In general

Re: HEADS-UP: Enabling WITH_DEBUG_FILES by default

2015-02-11 Thread Ed Maste
On 11 February 2015 at 21:39, Glen Barber wrote: > > Within the next 24 hours, I will merge the release-install-debug branch > into head, which will enable building and installing stripped debugging > files by default. > > In general, this should have no significant impact, but any fallout will >