Differential D6739

package.loader: Deduplicate target SWHIDs
ClosedPublic
Actions

Authored by ardumont on Dec 3 2021, 11:48 AM.

Details

Reviewers

vlorentz

Group Reviewers

Reviewers

Maniphest Tasks

T3763: Unstuck nixguix loading visits

Commits

rDLDBASEb3d7632d68f6: package.loader: Deduplicate target SWHIDs

Summary

So package loaders can actually finish their ingestion even when multiple releases
target the same directory.

Related to T3763

Test Plan

tox (should still be happy)

Diff Detail

Repository

rDLDBASE Generic VCS/Package Loader

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Dec 3 2021, 11:48 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptDec 3 2021, 11:48 AM

Build is green

Patch application report for D6739 (id=24476)

Rebasing onto 3e675d0d70...

Current branch diff-target is up to date.

Changes applied before test

commit b3d7632d68f69702f7b4172a5ed6900d48a4aa5d
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Dec 3 11:45:03 2021 +0100

    package.loader: Deduplicate target SWHIDs
    
    So package loaders can actually finish their ingestion even when multiple releases
    target the same directory.
    
    Related to T3763

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

Harbormaster completed remote builds in B25349: Diff 24476.Dec 3 2021, 11:50 AM

ardumont requested review of this revision.Dec 3 2021, 11:50 AM

vlorentz accepted this revision.Dec 3 2021, 12:00 PM

This revision is now accepted and ready to land.Dec 3 2021, 12:00 PM

Closed by commit rDLDBASEb3d7632d68f6: package.loader: Deduplicate target SWHIDs (authored by ardumont). · Explain WhyDec 3 2021, 12:02 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDLDBASEb3d7632d68f6: package.loader: Deduplicate target SWHIDs.