Page MenuHomeSoftware Heritage

fix cvs pserver authentication error
ClosedPublic

Authored by stsp on Oct 1 2021, 2:45 PM.

Details

Summary

Testing against cvs.savannah.gnu.org revealed that the CVS module
name should not be included in the authentication request.

Diff Detail

Repository
rDLDCVS CVS Loader
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24154
Build 37696: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 37695: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D6388 (id=23237)

Rebasing onto 62b0f560ef...

Current branch diff-target is up to date.
Changes applied before test
commit 2b41f5205f62d0778cff55c326f256977d162b28
Author: Stefan Sperling <stsp@stsp.name>
Date:   Fri Oct 1 14:43:53 2021 +0200

    fix pserver authentication error
    
    Testing against cvs.savannah.gnu.org revealed that the CVS module
    name should not be included in the authentication request.

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

stsp requested review of this revision.Oct 1 2021, 2:47 PM
vlorentz added a subscriber: vlorentz.
vlorentz added inline comments.
swh/loader/cvs/cvsclient.py
92

forgot to remove this

This revision is now accepted and ready to land.Oct 1 2021, 3:12 PM
This revision was automatically updated to reflect the committed changes.