Page MenuHomeSoftware Heritage

display file state in progress logging output
ClosedPublic

Authored by stsp on Oct 30 2021, 2:01 PM.

Diff Detail

Repository
rDLDCVS CVS Loader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D6591 (id=23953)

Could not rebase; Attempt merge onto beb7fc8a02...

Updating beb7fc8..32ff92e
Fast-forward
 swh/loader/cvs/cvsclient.py                     |  31 +++++---
 swh/loader/cvs/loader.py                        |   6 +-
 swh/loader/cvs/tests/data/greek-repository4.tgz | Bin 0 -> 12393 bytes
 swh/loader/cvs/tests/test_loader.py             |  92 +++++++++++++++++++++++-
 4 files changed, 116 insertions(+), 13 deletions(-)
 create mode 100644 swh/loader/cvs/tests/data/greek-repository4.tgz
Changes applied before test
commit 32ff92ed3e13f3fe63fff53d79e1d07881d64291
Author: Stefan Sperling <stsp@stsp.name>
Date:   Sat Oct 30 13:55:23 2021 +0200

    display file state in progress logging output

commit acc2601ae97ca729e3987de7cbd3548120e5d5a1
Author: Stefan Sperling <stsp@stsp.name>
Date:   Sat Oct 30 11:31:27 2021 +0200

    add support for RCS keyword expansion over pserver protocol
    
    We can simply ask the CVS server to expand keywords for us, instead
    of forcing binary file mode with the -kb option. The CVS repository
    contains per-file keyword expansion defaults the server will use.
    Files checked out by cvsclient.py should now match what a regular
    CVS client would check out by default.
    
    Add test cases which verify that we create the same snapshot ID
    for a repository which uses the Id keyword in a file, regardless
    of whether this repository is accessed via rsync or pserver.

commit 2563ac0917db60491fd3d3218a47d16c97083d13
Author: Stefan Sperling <stsp@stsp.name>
Date:   Thu Oct 28 19:04:05 2021 +0200

    in cvs loader tests, use f-strings to build repository URLs
    
    Suggested by ardumont in D6566

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

stsp requested review of this revision.Oct 30 2021, 2:03 PM
This revision is now accepted and ready to land.Nov 3 2021, 11:13 AM

Build is green

Patch application report for D6591 (id=24072)

Rebasing onto f52f0e4521...

Current branch diff-target is up to date.
Changes applied before test
commit ca23bc13c21bc60a0ad5b5f5736b5d1c2bd703d7
Author: Stefan Sperling <stsp@stsp.name>
Date:   Sat Oct 30 13:55:23 2021 +0200

    display file state in progress logging output

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