replay: Do not ignore externals in copyfrom operations
ClosedPublic
Actions

Authored by anlambert on Nov 24 2022, 11:56 AM.

Details

Reviewers

vlorentz
ardumont

Group Reviewers

Reviewers

Commits

rDLDSVN9f3a5800d208: replay: Do not ignore externals in copyfrom operations

Summary

When copying a directory from an ancestor revision, do not ignore
externals as properties are also copied by subversion so external
paths must also be exported.

Diff Detail

Repository

rDLDSVN Subversion (SVN) loader

Branch

copyfrom-do-not-ignore-externals

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 32948
Build 51642: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 51641: arc lint + arc unit

Event Timeline

anlambert created this revision.Nov 24 2022, 11:56 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptNov 24 2022, 11:56 AM

Build is green

Patch application report for D8882 (id=32015)

Rebasing onto bcd1e753c6...

Current branch diff-target is up to date.

Changes applied before test

commit 2730021d2f4b6c516eae8f5f75e484757b48acf0
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Thu Nov 24 11:54:03 2022 +0100

    replay: Do not ignore externals in copyfrom operations
    
    When copying a directory from an ancestor revision, do not ignore
    externals as properties are also copied by subversion so external
    paths must also be exported.

See https://jenkins.softwareheritage.org/job/DLDSVN/job/tests-on-diff/357/ for more details.

Harbormaster completed remote builds in B32948: Diff 32015.Nov 24 2022, 11:59 AM

anlambert requested review of this revision.Nov 24 2022, 11:59 AM

ardumont added a subscriber: ardumont.Nov 29 2022, 2:27 PM

ardumont added inline comments.

swh/loader/svn/tests/test_externals.py
1698	what happened in this test initially (without the runtime code change)?

Rebase

Build is green

Patch application report for D8882 (id=32217)

Rebasing onto 2b80484b8b...

Current branch diff-target is up to date.

Changes applied before test

commit 9f3a5800d208f060949b8fb0f3e9788ab275a384
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Thu Nov 24 11:54:03 2022 +0100

    replay: Do not ignore externals in copyfrom operations
    
    When copying a directory from an ancestor revision, do not ignore
    externals as properties are also copied by subversion so external
    paths must also be exported.

See https://jenkins.softwareheritage.org/job/DLDSVN/job/tests-on-diff/361/ for more details.

Harbormaster completed remote builds in B33172: Diff 32217.Dec 7 2022, 5:27 PM

anlambert added a child revision: D8941: replay: Simplify FileEditor implementation.Dec 7 2022, 5:34 PM

anlambert added inline comments.Dec 8 2022, 11:15 AM