This continues to happen. For example:

• http://qa.in-portal.org/ - rTSIM16064 (Untitled Commit) on home page ( http://qa.in-portal.org/rSIM16064 )
• the actual number from revision belongs to different repository - http://qa.in-portal.org/rTDEF16064

Here is TSIM repository configuration:

Here is TDEF repository configuration:

As you can see they both share same SVN repository on disk, but each scans only sub-path from it.

So the question stays: why same SVN commit gets imported under 2 repositories.

The http://qa.in-portal.org/rSIM16064 should be http://qa.in-portal.org/rTSIM16064 in previous comment.

Some more info I was able to find out. The commits:

• have importStatus 15 (in contrast to 1039 for all other commits)
• have "foreign-svn-stub":true in repository_commitdata table in the commitDetails field

Here are 2 options I see:

1. delete these stub commit if they are not needed
2. don't show sub commits in any commit lists

What can be the best way to proceed here?

It appears that during commit discovery on a repository (when daemon is looking for new commits) each commit wants to set it's own parent commit. And that parent commit is calculated as commitIdentifier -1 for SVN repositories. This is incorrect when we're dealing with SVN repositories, where only sub-path is imported.

This parent commit lookup only happens when a file was deleted.

This appears to be the problematic line: https://secure.phabricator.com/diffusion/P/browse/master/src/applications/repository/worker/commitchangeparser/PhabricatorRepositorySvnCommitChangeParserWorker.php;12f3f6d3a9ef9c7731051815846810cb3c4cd248$102

In the related T6537 task same issue was solved for cases, when Skip Past This Commit link is used on such commit (where parent commit identifier isn't just -1 of current commit).

@epriestley, could you please confirm that a described behavior is a bug?

Another case, when this can be happening is during file/folder coping. Even through I can copy within the repository svn-subpath the revision reported by svn log as copyfrom-rev is last at the moment of copy per whole SVN repository and not just sub-path: https://secure.phabricator.com/diffusion/P/browse/master/src/applications/repository/worker/commitchangeparser/PhabricatorRepositorySvnCommitChangeParserWorker.php;fa77fdd168d4f4f0a306e3b9867b5f09481fb6d1$50

So getting previous SVN revision in given repository sub-path (if possible) would be best thing to do.