From: Andreas Ericsson <ae@op5.se>
To: Linus Torvalds <torvalds@osdl.org>
Cc: Junio C Hamano <junkio@cox.net>, Git Mailing List <git@vger.kernel.org>
Subject: Re: git binary size...
Date: Wed, 11 Jan 2006 20:14:03 +0100 [thread overview]
Message-ID: <43C558FB.3030102@op5.se> (raw)
In-Reply-To: <Pine.LNX.4.64.0601111021450.5073@g5.osdl.org>
Linus Torvalds wrote:
> Guess what the difference is here?
>
> [torvalds@g5 ~]$ du -sh bin/
> 14M bin/
> [torvalds@g5 ~]$ du -sh bin/
> 5.8M bin/
>
> Give up?
>
> In one case, "git" was compiled with the default options in the git
> Makefile. In the other one, the "-g" was removed.
>
> Now, maybe this is extra visible with PowerPC (32-bit) binaries, and it's
> not as bad on x86, but it's still a bit distressing.
>
> That "-g" doesn't buy users much of anything, and I doubt most developers
> care that deeply most of the time either (and can easily add it when they
> do care). It's left-over from long ago when it was much more useful.
>
> So I'd suggest just removing it.
>
I'd suggest adding
strip:
strip $(PROGRAMS)
install: strip
to Makefile instead. That way people working on various git-tools won't
have to remember to override the CFLAGS when debugging new stuff.
--
Andreas Ericsson andreas.ericsson@op5.se
OP5 AB www.op5.se
Tel: +46 8-230225 Fax: +46 8-230231
next prev parent reply other threads:[~2006-01-11 19:14 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-01-11 18:26 git binary size Linus Torvalds
2006-01-11 19:14 ` Andreas Ericsson [this message]
2006-01-11 19:39 ` H. Peter Anvin
2006-01-11 19:54 ` Andreas Ericsson
2006-01-11 19:56 ` H. Peter Anvin
2006-01-11 20:00 ` Andreas Ericsson
2006-01-11 19:40 ` Junio C Hamano
2006-01-11 19:44 ` Linus Torvalds
2006-01-11 19:58 ` H. Peter Anvin
2006-01-12 10:15 ` Coywolf Qi Hunt
2006-01-12 13:49 ` Andreas Ericsson
2006-01-12 17:37 ` H. Peter Anvin
2006-01-12 18:13 ` Linus Torvalds
2006-01-12 18:32 ` Johannes Schindelin
2006-01-12 18:46 ` Linus Torvalds
2006-01-12 19:16 ` H. Peter Anvin
2006-01-20 18:00 ` Jon Loeliger
2006-01-12 18:12 ` Linus Torvalds
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=43C558FB.3030102@op5.se \
--to=ae@op5.se \
--cc=git@vger.kernel.org \
--cc=junkio@cox.net \
--cc=torvalds@osdl.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.