[gentoo-dev] RFC: remove annoying "You should enable -g (or higher) for debugging!" message

[gentoo-dev] RFC: remove annoying "You should enable -g (or higher) for debugging!" message

["Paweł Hajdan, Jr."]

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

This is mostly a nit-like RFC. The developer profile adds a
profile.bashrc, which prints the "You should enable -g (or higher) for
debugging!" message when "-g" is not in CFLAGS.

I wonder if we can remove that file. It has been added 3 years ago:

<http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/profiles/targets/developer/profile.bashrc?view=markup>

It is not obvious from looking at emerge output what prints this
message, it doesn't fit well with the rest of the output, and may be
easy to miss:

 * USE:        elibc_glibc gtk kernel_linux test userland_GNU x86
 * FEATURES:   sandbox splitdebug test userpriv usersandbox
You should enable -g (or higher) for debugging!
>>> Unpacking source...
>>> Unpacking mtr-0.80.tar.gz to /var/tmp/portage/net-analyzer/mtr-0.80/work

I think it is not necessary for every developer to add "-g" to CFLAGS -
the main point is to have any symbols at all, not just
to-the-file-and-line-precise debugging info. If the latter is needed,
the developer will obviously know what to do.

What do you think about removing
gentoo-x86/profiles/targets/developer/profile.bashrc file?


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 194 bytes --]

Re: [gentoo-dev] RFC: remove annoying "You should enable -g (or higher) for debugging!" message

[Tomáš Chvátal]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Dne 7.5.2011 16:20, "Paweł Hajdan, Jr." napsal(a):
> This is mostly a nit-like RFC. The developer profile adds a
> profile.bashrc, which prints the "You should enable -g (or higher) for
> debugging!" message when "-g" is not in CFLAGS.
> 
> I wonder if we can remove that file. It has been added 3 years ago:
> 
> <http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/profiles/targets/developer/profile.bashrc?view=markup>
> 
> It is not obvious from looking at emerge output what prints this
> message, it doesn't fit well with the rest of the output, and may be
> easy to miss:
> 
>  * USE:        elibc_glibc gtk kernel_linux test userland_GNU x86
>  * FEATURES:   sandbox splitdebug test userpriv usersandbox
> You should enable -g (or higher) for debugging!
>>>> Unpacking source...
>>>> Unpacking mtr-0.80.tar.gz to /var/tmp/portage/net-analyzer/mtr-0.80/work
> 
> I think it is not necessary for every developer to add "-g" to CFLAGS -
> the main point is to have any symbols at all, not just
> to-the-file-and-line-precise debugging info. If the latter is needed,
> the developer will obviously know what to do.
> 
> What do you think about removing
> gentoo-x86/profiles/targets/developer/profile.bashrc file?
> 
I would preffer to see there -ggdb and FEATURES=splitdebug to be honest.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk3FX4sACgkQHB6c3gNBRYdxGACglTvsumBWP2k3EHqTsUn35HRP
jXUAoKSPfXIqFcPDeQ5XtnAHutggf9bH
=jABx
-----END PGP SIGNATURE-----

Re: [gentoo-dev] RFC: remove annoying "You should enable -g (or higher) for debugging!" message

["Paweł Hajdan, Jr."]

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

On 5/7/11 5:04 PM, Tomáš Chvátal wrote:
> Dne 7.5.2011 16:20, "PaweB Hajdan, Jr." napsal(a):
>> This is mostly a nit-like RFC. The developer profile adds a
>> profile.bashrc, which prints the "You should enable -g (or higher) for
>> debugging!" message when "-g" is not in CFLAGS.
> 
>> I wonder if we can remove that file. It has been added 3 years ago:
> 
>> <http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/profiles/targets/developer/profile.bashrc?view=markup>
> 
>> It is not obvious from looking at emerge output what prints this
>> message, it doesn't fit well with the rest of the output, and may be
>> easy to miss:
> 
>>  * USE:        elibc_glibc gtk kernel_linux test userland_GNU x86
>>  * FEATURES:   sandbox splitdebug test userpriv usersandbox
>> You should enable -g (or higher) for debugging!
>>>>> Unpacking source...
>>>>> Unpacking mtr-0.80.tar.gz to /var/tmp/portage/net-analyzer/mtr-0.80/work
> 
>> I think it is not necessary for every developer to add "-g" to CFLAGS -
>> the main point is to have any symbols at all, not just
>> to-the-file-and-line-precise debugging info. If the latter is needed,
>> the developer will obviously know what to do.
> 
>> What do you think about removing
>> gentoo-x86/profiles/targets/developer/profile.bashrc file?
> 
> I would preffer to see there -ggdb and FEATURES=splitdebug to be honest.

FEATURES=splitdebug is already a part of the developer profile. And then
that bashrc file was added there to avoid adding -ggdb to CFLAGS
unconditionally (it increases compile/link time a *lot* for packages
like xulrunner, webkit, or chromium).

This was the only response to my e-mail, so I guess most people don't
care about the bashrc file in question. I'd still like to remove it -
was your reply a "no way" or just "I'd prefer something else"?


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 194 bytes --]

Re: [gentoo-dev] RFC: remove annoying "You should enable -g (or higher) for debugging!" message

["Paweł Hajdan, Jr."]

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

On 5/7/11 4:20 PM, "Paweł Hajdan, Jr." wrote:
> This is mostly a nit-like RFC. The developer profile adds a
> profile.bashrc, which prints the "You should enable -g (or higher) for
> debugging!" message when "-g" is not in CFLAGS.
> 
> I wonder if we can remove that file. It has been added 3 years ago:
> 
> <http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/profiles/targets/developer/profile.bashrc?view=markup>
> 
> It is not obvious from looking at emerge output what prints this
> message, it doesn't fit well with the rest of the output, and may be
> easy to miss:
> 
>  * USE:        elibc_glibc gtk kernel_linux test userland_GNU x86
>  * FEATURES:   sandbox splitdebug test userpriv usersandbox
> You should enable -g (or higher) for debugging!
>>>> Unpacking source...
>>>> Unpacking mtr-0.80.tar.gz to /var/tmp/portage/net-analyzer/mtr-0.80/work
> 
> I think it is not necessary for every developer to add "-g" to CFLAGS -
> the main point is to have any symbols at all, not just
> to-the-file-and-line-precise debugging info. If the latter is needed,
> the developer will obviously know what to do.
> 
> What do you think about removing
> gentoo-x86/profiles/targets/developer/profile.bashrc file?
> 

FYI: I went ahead and committed the change. Enjoy a slightly simpler
developer profile!


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 194 bytes --]