Page MenuHomeSoftware Heritage

nixguix: Add support for pseudo url with missing schema
ClosedPublic

Authored by ardumont on Oct 4 2022, 4:01 PM.

Diff Detail

Repository
rDLS Listers
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 D8612 (id=31106)

Could not rebase; Attempt merge onto d92474bbda...

Updating d92474b..94dafe1
Fast-forward
 swh/lister/nixguix/lister.py                         | 20 ++++++++++++--------
 swh/lister/nixguix/tests/data/guix-swh_sources.json  |  7 +++++++
 .../nixguix/tests/data/nixpkgs-swh_sources.json      |  7 +++++++
 swh/lister/nixguix/tests/test_lister.py              | 16 +++++++++++-----
 4 files changed, 37 insertions(+), 13 deletions(-)
Changes applied before test
commit 94dafe1d839106b76c65a098df11eec122af2f6b
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Oct 4 16:01:00 2022 +0200

    nixguix: Add support for pseudo url with missing schema
    
    Related to T3294
    Related to T3781

commit 0f8f293f968ee1efc8b269f4073bbac99a4f3615
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Oct 4 14:57:01 2022 +0200

    nixguix: Deal with connection error with server
    
    When that arises, we skip the origins.
    
    Related to T3781

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

Adapt according to suggestion

Build is green

Patch application report for D8612 (id=31109)

Rebasing onto 0f8f293f96...

Current branch diff-target is up to date.
Changes applied before test
commit 5daead68adcbde07762eb4824001637a15dda6b2
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Oct 4 16:01:00 2022 +0200

    nixguix: Add support for pseudo url with missing schema
    
    Related to T3294
    Related to T3781

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

This revision is now accepted and ready to land.Oct 4 2022, 4:43 PM