From: Patrick McHardy <kaber@trash.net>
To: Jan Engelhardt <jengelh@medozas.de>
Cc: Netfilter Development Mailinglist
<netfilter-devel@vger.kernel.org>,
Netfilter Core Team <coreteam@netfilter.org>
Subject: Re: netfilter moving to git
Date: Tue, 13 May 2008 17:19:19 +0200 [thread overview]
Message-ID: <4829B177.9000501@trash.net> (raw)
In-Reply-To: <alpine.LNX.1.10.0805131620450.448@fbirervta.pbzchgretzou.qr>
Jan Engelhardt wrote:
> On Tuesday 2008-05-13 16:17, Patrick McHardy wrote:
>>>
>>> All of that I have long done in my copy of the git repository,
>>> announced already around mid-February. - git://dev.medozas.de/xtables
>>> (Never mind the repository's name, it's the official SVN-top-off-the-head)
>>
>> Feel free to send me instructions for repeating these steps,
>
> while (!done && !looks_good) {
> git-filter-branch
> }
>
> Seriously, there is no automated way to properly populate the Author,
> Committer, Date and Email fields, especially if the practice how it
> is effectively stored in the logs has changed over time.
> http://dev.medozas.de/files/iptables.aux/ is the base scripts
> (running do_rewrite), plus hand fixups (do_rewrite2) where the
> automated tools fail to catch every case.
I don't think its particulary important to fix up the commiter
names (and strip the certificate-mess), it you really need to
know, its easy to find out.
>> but I certainly won't replace our repository by an external one.
>
> This is the one from Yasuyuki, kept in-sync with the SVN.
>
> There is no difference in the blobs:
> diff -dpru -x .svn -x .git iptables.svncheckout iptables.git;
> (account for $Id$ expansion)
>
> or the directory structure:
> cd iptables.svncheckout/;
> for i in `find [a-z]* -type d`; do
> diff -dpru <(git ls-tree HEAD $i/) \
> <(cd ../xtables/; git ls-tree HEAD $i/);
> done | less -S
I took the last one from Yasuyuki and resynced it, might have
missed a more current version though.
> Only changes so far are logs to fix obvious spelling mistakes or
> metadata fixups (separation of author--committer), or
> perhaps just give a better message for the older commits like "argh".
>
> [something like
> for ((n = 0; n < 1244; ++n); do
> diff -dpru <(cd iptables.gitified; git log HEAD~"$n"!) \
> <(cd xtables; git log HEAD~"$n"!)
> done;
> should make it visible]
>
> Fact is there has been going a lot of work into keeping it in
> git-style shape. It's alright if you do not trust it, but then
> again, it would be a waste to run with a crippled-metadata repo
> or to do the conversion again.
True, but as a matter of principle, I can't use an untrusted repository.
And anyways, I also don't see anything wrong with simply starting with
a clean history by only importing the latest version, In that case I'd
have no problem using your fixed-up repository as historical reference.
next prev parent reply other threads:[~2008-05-13 15:19 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-13 13:24 netfilter moving to git Patrick McHardy
2008-05-13 14:16 ` Jan Engelhardt
2008-05-13 14:17 ` Patrick McHardy
2008-05-13 15:13 ` Jan Engelhardt
2008-05-13 15:19 ` Patrick McHardy [this message]
2008-05-13 16:32 ` Jan Engelhardt
2008-05-14 16:28 ` Patrick McHardy
2008-05-14 18:24 ` Jan Engelhardt
2008-05-15 14:33 ` Patrick McHardy
2008-05-17 7:59 ` Jan Engelhardt
2008-05-18 2:26 ` Patrick McHardy
2008-05-17 7:59 ` Jan Engelhardt
2008-05-18 2:25 ` Patrick McHardy
2008-05-20 22:39 ` Jan Engelhardt
2008-05-21 9:43 ` Patrick McHardy
2008-05-21 16:40 ` Jan Engelhardt
2008-05-21 16:40 ` Patrick McHardy
2008-05-21 16:43 ` Jan Engelhardt
2008-05-22 14:21 ` Patrick McHardy
2008-05-22 14:40 ` Jan Engelhardt
2008-05-22 14:46 ` Patrick McHardy
2008-05-22 14:47 ` Patrick McHardy
2008-05-22 15:32 ` Jan Engelhardt
2008-05-25 22:03 ` Jan Engelhardt
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=4829B177.9000501@trash.net \
--to=kaber@trash.net \
--cc=coreteam@netfilter.org \
--cc=jengelh@medozas.de \
--cc=netfilter-devel@vger.kernel.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.