git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Martin Langhoff <martin@catalyst.net.nz>
Cc: git@vger.kernel.org, jmacd@cs.berkeley.edu
Subject: Re: [offtopic?] xdelta patch format wrapper
Date: Tue, 12 Feb 2008 19:32:26 -0800	[thread overview]
Message-ID: <7vy79py1it.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <47B24D8A.5090703@catalyst.net.nz> (Martin Langhoff's message of "Wed, 13 Feb 2008 14:53:14 +1300")

Martin Langhoff <martin@catalyst.net.nz> writes:

> So my question is what is a good format for the header? My thinking sofar:
>
>  - have a prefix to scan for, such as "xdelta" at the beginning of
>    the file, or after a newline/whitespace
>
>  - keep the <fromsha1> <tosha1> line
>
>  - \0 delimited filenames
>
>  - filenames as ambiguous bag'o'bytes or utf-8?
>    (should we have another flamewar on this? ;-) )
>
>  - keep file modes and perhaps support copy/move headers
>
>  - keep a/ b/ prefixes?
>
>  - last line in the header is length: <length-in-bytes>, followed by
>    a newline and the xdelta itself
>
>  - one or more newlines follow the end of the xdelta if there is another
>    header coming

I am lost as to your objective because you seem to be keeping a
whole LOT more than I would have imagined for a specialized
purpose file format.

If you want to reuse that much of git, maybe our binary patch
format is good enough for you?  We always produce two xdelta so
that we can apply in reverse, but it is Ok to add a one-way
option.

  reply	other threads:[~2008-02-13  3:33 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-13  1:53 [offtopic?] xdelta patch format wrapper Martin Langhoff
2008-02-13  3:32 ` Junio C Hamano [this message]
2008-02-13  3:46   ` Martin Langhoff
2008-02-13  3:56     ` Junio C Hamano
2008-02-13 11:33     ` Johannes Schindelin
2008-02-13 17:53       ` Martin Langhoff
2008-02-13  4:13   ` Martin Langhoff

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=7vy79py1it.fsf@gitster.siamese.dyndns.org \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=jmacd@cs.berkeley.edu \
    --cc=martin@catalyst.net.nz \
    /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).