Differential D7027
replay: Handle recursive external definition Authored by anlambert on Jan 24 2022, 3:38 PM.
Details
It is possible that an external definition lead to a recursive subversion As that recursive export will crash the loader and make the loading failed, We will then ignore externals when exporting a revision to check for divergence Related to T3870#77491
Diff Detail
Event TimelineComment Actions Build is green Patch application report for D7027 (id=25464)Rebasing onto 3e4d5d7cc9... Current branch diff-target is up to date. Changes applied before testcommit bb606ed569a3f07d792f0550625560dcb47a6cd2
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date: Mon Jan 24 15:32:02 2022 +0100
replay: Handle recursive external definition
It is possible that an external definition lead to a recursive subversion
export operation (https://issues.apache.org/jira/browse/SVN-1703).
As that recursive export will crash the loader and make the loading failed,
the only solution for that edge case is to stop processing externals while
a recursive external is detected in a revision.
We will then ignore externals when exporting a revision to check for divergence
with the recontructed filesystem.
Related to T3870See https://jenkins.softwareheritage.org/job/DLDSVN/job/tests-on-diff/267/ for more details. Comment Actions Build is green Patch application report for D7027 (id=25471)Rebasing onto 3e4d5d7cc9... Current branch diff-target is up to date. Changes applied before testcommit 51caee0663dfcb8645a6c440084a69225dc0c4a0
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date: Mon Jan 24 15:32:02 2022 +0100
replay: Handle recursive external definition
It is possible that an external definition lead to a recursive subversion
export operation (https://issues.apache.org/jira/browse/SVN-1703).
As that recursive export will crash the loader and make the loading failed,
the only solution for that edge case is to stop processing externals while
a recursive external is detected in a revision.
We will then ignore externals when exporting a revision to check for divergence
with the recontructed filesystem.
Related to T3870See https://jenkins.softwareheritage.org/job/DLDSVN/job/tests-on-diff/268/ for more details. Comment Actions ok, one suggestion inline
Comment Actions Build is green Patch application report for D7027 (id=25476)Rebasing onto 3e4d5d7cc9... Current branch diff-target is up to date. Changes applied before testcommit afdee6dbee9be55f2eaf192dfb23733952977a2e
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date: Mon Jan 24 15:32:02 2022 +0100
replay: Handle recursive external definition
It is possible that an external definition lead to a recursive subversion
export operation (https://issues.apache.org/jira/browse/SVN-1703).
As that recursive export will crash the loader and make the loading failed,
the only solution for that edge case is to stop processing externals while
a recursive external is detected in a revision.
We will then ignore externals when exporting a revision to check for divergence
with the recontructed filesystem.
Related to T3870See https://jenkins.softwareheritage.org/job/DLDSVN/job/tests-on-diff/269/ for more details. | ||||||||||||||||||||