git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Linus Torvalds <torvalds@linux-foundation.org>
To: Junio C Hamano <gitster@pobox.com>,
	Git Mailing List <git@vger.kernel.org>
Subject: [Patch 0/5] make more commands built-in
Date: Fri, 22 Jan 2010 08:22:51 -0800 (PST)	[thread overview]
Message-ID: <alpine.LFD.2.00.1001220804550.13231@localhost.localdomain> (raw)


Ok, so since I was working on this yesterday, I decided to just continue 
until I was done with all the trivial files.

This series makes five more commands built-in: 'merge-index', 'mktag', 
'unpack-file', 'pack-redundant' and 'index-pack'. It further shrinks my 
git install footprint by about 20%:

	[torvalds@nehalem git]$ du -s /home/torvalds/libexec/git-core
	21424	/home/torvalds/libexec/git-core (before)
	16920	/home/torvalds/libexec/git-core (after)

and if we didn't default to having debug info enabled, it would look 
almost acceptable:

	[torvalds@nehalem git]$ du -s /home/torvalds/libexec/git-core
	5728	/home/torvalds/libexec/git-core (with "-g" removed)
	5108	/home/torvalds/libexec/git-core (with -Os and no debugging)

There still remains a few big git commands after this, but this takes care 
of most of the core ones. 

All of these patches are trivial, the only one that has _any_ changes 
apart from the obvious builtin conversion is 'pack-redundant' which 
required some of the pack creation interfaces to now take 'const' index 
pathnames etc.

But as you can see, it removes more lines than it adds, and considering 
that yesterday the 'du' reported ~40MB of disk space for the git install, 
the shrinking certainly matters (of course, I suspect most distros 
wouldn't ship with debug info, so for most people it's about a few MB 
rather than a few tens of MB, but still).

		Linus
---
Linus Torvalds (5):
  make "merge-index" a built-in
  make "mktag" a built-in
  make "git unpack-file" a built-in
  make "git pack-redundant" a built-in
  make "index-pack" a built-in

 Makefile                                     |   10 +++++-----
 index-pack.c => builtin-index-pack.c         |   16 +++++++---------
 merge-index.c => builtin-merge-index.c       |    7 ++-----
 mktag.c => builtin-mktag.c                   |    6 +-----
 builtin-pack-objects.c                       |    5 +++--
 pack-redundant.c => builtin-pack-redundant.c |    8 ++------
 unpack-file.c => builtin-unpack-file.c       |    5 +----
 builtin.h                                    |    5 +++++
 git.c                                        |    5 +++++
 pack-write.c                                 |    4 ++--
 pack.h                                       |    2 +-
 11 files changed, 34 insertions(+), 39 deletions(-)
 rename index-pack.c => builtin-index-pack.c (98%)
 rename merge-index.c => builtin-merge-index.c (94%)
 rename mktag.c => builtin-mktag.c (98%)
 rename pack-redundant.c => builtin-pack-redundant.c (99%)
 rename unpack-file.c => builtin-unpack-file.c (89%)

             reply	other threads:[~2010-01-22 16:23 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-22 16:22 Linus Torvalds [this message]
2010-01-22 16:25 ` [PATCH 1/5] make "merge-index" a built-in Linus Torvalds
2010-01-22 16:26   ` PATCH 2/5] make "mktag" " Linus Torvalds
2010-01-22 16:27     ` [PATCH 3/5] make "git unpack-file" " Linus Torvalds
2010-01-22 16:28       ` [PATCH 4/5] make "git pack-redundant" " Linus Torvalds
2010-01-22 16:31         ` [PATCH 5/5] make "index-pack" " Linus Torvalds
2010-01-22 18:04       ` [PATCH 3/5] make "git unpack-file" " Junio C Hamano
2010-01-22 18:26         ` Linus Torvalds
2010-01-22 16:33     ` PATCH 2/5] make "mktag" " 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=alpine.LFD.2.00.1001220804550.13231@localhost.localdomain \
    --to=torvalds@linux-foundation.org \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).