git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Oakley <andrew@adoakley.name>
To: Luke Diamand <luke@diamand.org>
Cc: Git Users <git@vger.kernel.org>, Junio C Hamano <gitster@pobox.com>
Subject: Re: [PATCH v3] git-p4: recover from inconsistent perforce history
Date: Sun, 10 May 2020 15:13:54 +0100	[thread overview]
Message-ID: <20200510151354.16ce2ab1@ado-tr.home.arpa> (raw)
In-Reply-To: <CAE5ih793qKyOSE-hkOw7+nFmM3XTRxxrXv0FD2+WWXjGbVHkoQ@mail.gmail.com>

On Sun, 10 May 2020 13:03:11 +0100
Luke Diamand <luke@diamand.org> wrote:
> Perforce changed their server to reject this kind of thing in the
> 2017.1 version:
> 
>     Bugs fixed in 2017.1
>     #1489051 (Job #2170) **
>        Submitting a file with the same name as an existing depot
>        directory path (or vice versa) will now be rejected.
> 
> (Of course people will still have damaged repos even today).
> 
> I tried your test with both the 2015.1 and the 2020.1 versions, and it
> worked in both cases - shouldn't it be impossible to get into the
> state that git-p4 now recovers from with a newer p4d?

Yes, there is an option in perforce (submit.collision.check) that stops
new changelists from introducing this problem, and it is turned on by
default.  Unfortunately this option does *exactly* what the description
says, so you can't delete a directory and replace it with a file in the
same changelist - the delete has to happen first.  It's not clear which
behaviour is least bad.

The test case tries to turn the perforce option off so it works on both
old and new server versions.

Thanks

  reply	other threads:[~2020-05-10 14:14 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-10 10:16 [PATCH v3] git-p4: recover from inconsistent perforce history Andrew Oakley
2020-05-10 12:03 ` Luke Diamand
2020-05-10 14:13   ` Andrew Oakley [this message]
2020-05-10 17:01   ` Junio C Hamano

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=20200510151354.16ce2ab1@ado-tr.home.arpa \
    --to=andrew@adoakley.name \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=luke@diamand.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 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).