git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Makefile: do not compile git with debugging symbols by default

From
Matthieu Moy <matthieu.moy@grenoble-inp.fr>
Date
Jan 22, 2015, 17:36 UTC
Message-ID
<vpqa91ahg46.fsf@anie.imag.fr>
In-Reply-To
<CANCZXo7ocgG27Y48NjYxurVMWOvHkvGqDrLuntkSTxHUK6hcNw@mail.gmail.com>
Alexander Kuleshov <kuleshovmail@gmail.com> writes:
> Hello Jeff,
>
> Yes, main point is size of executable.

The Git executable is a few megabytes, i.e. 0.001% the size of a really small hard disk. The benefit seems really negligible to me.

OTOH, debug information allow users to do better bug reports in case of crash (gdb, valgrind), which outweights by far the benefit of saving a handfull of megabytes IMHO.

On a side note, I find it very frustrating when a program I use crashes, opens a bug report wizard, and end up telling me "sorry, your distro removed the debug symbols, recompile everything if you want to report a bug".

I understand that for a few users, the size of executable matters. But this category of users should be able to find the "strip" target or something equivalent.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/
Previous: Alexander KuleshovNext: Jeff King
Message 6 of 9 in “Makefile: do not compile git with debugging symbols by default”
  1. Makefile: do not compile git with debugging symbols by defaultAlexander Kuleshov, Jan 22, 2015
  2. Jeff KingJan 22, 2015
  3. Mike HommeyJan 22, 2015
  4. Alexander KuleshovJan 22, 2015
  5. Alexander KuleshovJan 22, 2015
  6. Matthieu MoyJan 22, 2015
  7. Jeff KingJan 22, 2015
  8. Mike HommeyJan 22, 2015
  9. David AguilarJan 27, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.