git-archive and submodules

(Please note that I am not subscribed to the mailing list, so if you
want me to read your reply, it must be sent to
woody.gilk@kohanaphp.com)

I would find it hugely valuable if git-archive would support
submodules, rather than leaving empty files in the place of
submodules. Supporting submodules is critical to packaging releases
(combined with signed tags). With git-archive not support submodules,
it means that I have to manually package and sign each release, rather
than using git-tag.

This seems like it should be a fairly trivial thing to support,
possibly with the addition of a --with-submodules parameter for
git-archive to avoid backwards compatibility issues.

Thank you,
-Woody Gilk
Lead Developer of Kohana PHP Framework

On Mon, Jul 20, 2009 at 12:22 PM, Woody Gilk[off-list ref] wrote:

I would find it hugely valuable if git-archive would support
submodules, rather than leaving empty files in the place of
submodules. Supporting submodules is critical to packaging releases
(combined with signed tags). With git-archive not support submodules,
it means that I have to manually package and sign each release, rather
than using git-tag.

<shameless-plug> This works already today if you use git-subtree[1]
instead of submodules. </shameless-plug>

Have fun,

Avery

[1] http://github.com/apenwarr/git-subtree

Woody Gilk wrote:

I would find it hugely valuable if git-archive would support
submodules, rather than leaving empty files in the place of
submodules.

Such support was already in the pipeline before 1.6.2, see this
thread:

  http://thread.gmane.org/gmane.comp.version-control.git/107030

Lars' series is still available from the repo mentioned in one of the
messages

  git://hjemli.net/pub/git/git lh/traverse-gitlinks

but does not merge into master cleanly any more, because an earlier
version of the first commit (of now three) made it into 1.6.2.  For
your testing convenience, I rebased the other two commits on 'master'
and pushed them to

  git://repo.or.cz/git/trast.git lh/traverse-gitlinks-on-master

All conflicts were trivial, but I made a minor change: there's already
a t5001 from 8aece7f (archive test: attributes, 2009-04-18), so I
renamed it to t5002 in the tip commit.

-- 
Thomas Rast
trast@{inf,student}.ethz.ch