Page MenuHomeSoftware Heritage

package/debian: Handle missing md5sum in source package metadata
ClosedPublic

Authored by anlambert on Sep 13 2021, 2:04 PM.

Details

Summary

Some debian source package metadata are missing md5 sums for their
files, for instance those from the buster-proposed-updates suite.

So turn the md5sum field from DebianFileMetadata model optional
in order to avoid loading errors.

Related to T3547

Diff Detail

Repository
rDLDBASE Generic VCS/Package Loader
Branch
debian-loader-missing-md5sum
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 23536
Build 36723: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 36722: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D6244 (id=22599)

Rebasing onto 50b062adc7...

Current branch diff-target is up to date.
Changes applied before test
commit fd70af2758c107c6cab2f2979676f761b107bd96
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Mon Sep 13 13:59:41 2021 +0200

    package/debian: Handle missing md5sum in source package metadata
    
    Some debian source package metadata are missing md5 sums for their
    files, for instance those from the buster-proposed-updates suite.
    
    So turn the md5sum field from DebianFileMetadata model optional
    in order to avoid loading errors.
    
    Related to T3547

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

ardumont added a subscriber: ardumont.

Thanks.

One typo inline, nothing blocking.

swh/loader/package/debian/tests/test_debian.py
314–315
This revision is now accepted and ready to land.Sep 13 2021, 2:11 PM
swh/loader/package/debian/tests/test_debian.py
314–315

thanks !

Build is green

Patch application report for D6244 (id=22602)

Rebasing onto 50b062adc7...

Current branch diff-target is up to date.
Changes applied before test
commit 8e00be225c7cf5d7609da1f4fbb85afd44c2314e
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Mon Sep 13 13:59:41 2021 +0200

    package/debian: Handle missing md5sum in source package metadata
    
    Some debian source package metadata are missing md5 sums for their
    files, for instance those from the buster-proposed-updates suite.
    
    So turn the md5sum field from DebianFileMetadata model optional
    in order to avoid loading errors.
    
    Related to T3547

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