Re: [PATCH 0/3] fast-export: allow dumping anonymization mappings
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Jun 19, 2020, 13:51 UTC
- Message-ID
- <nycvar.QRO.7.76.6.2006191550260.54@tvgsbejvaqbjf.bet>
- In-Reply-To
- <20200619132304.GA2540657@coredump.intra.peff.net>
Hi Peff,
On Fri, 19 Jun 2020, Jeff King wrote:
Show 14 quoted lines
> The "master" branch is special in fast-export in that we don't anonymize > it. And the reason is that it helps to have some known reference point > between the original and anonymized repo, so you can find the same > commits. > > This series gives an alternate way to achieve the same effect, but much > better in that it works for _any_ ref (so if you are trying to reproduce > the effect of "rev-list origin/foo..bar" in the anonymized repo, you can > easily do so). Ditto for paths, so that "rev-list -- foo.c" can be > reproduced in the anonymized repo. > > And then we can drop the specialness of "master", which in turn is one > less thing to worry about in Dscho's series to make the default branch > configurable.
Thank you for working on this!
I don't have any suggestions on top of Eric's and Junio's.
I'll drop the `fast-export` patches from the next round of my patch series.
Ciao, Dscho
Show 12 quoted lines
> > [1/3]: fast-export: allow dumping the refname mapping > [2/3]: fast-export: anonymize "master" refname > [3/3]: fast-export: allow dumping the path mapping > > Documentation/git-fast-export.txt | 32 +++++++++++++++++ > builtin/fast-export.c | 58 +++++++++++++++++++++++++++---- > t/t9351-fast-export-anonymize.sh | 30 ++++++++++++---- > 3 files changed, 106 insertions(+), 14 deletions(-) > > -Peff >