Page MenuHomeSoftware Heritage

sword mapping: Add support for Atom author elements
ClosedPublic

Authored by vlorentz on Aug 12 2022, 11:47 AM.

Details

Summary

Atom documents from old deposits usually use these instead of codemeta's

Depends on D8235.

Diff Detail

Repository
rDCIDX Metadata indexer
Branch
sword-mapping
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 30767
Build 48108: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 48107: arc lint + arc unit

Event Timeline

Build has FAILED

Patch application report for D8237 (id=29715)

Could not rebase; Attempt merge onto bb9082a6e5...

Updating bb9082a..03d8e1c
Fast-forward
 docs/metadata-workflow.rst                         |   2 +-
 swh/indexer/codemeta.py                            |   5 +-
 swh/indexer/metadata_dictionary/__init__.py        |   1 +
 swh/indexer/metadata_dictionary/codemeta.py        |  94 ++++++++++-
 .../tests/metadata_dictionary/test_codemeta.py     | 176 +++++++++++++++++++++
 swh/indexer/tests/test_cli.py                      |  12 +-
 6 files changed, 283 insertions(+), 7 deletions(-)
Changes applied before test
commit 03d8e1cbc5f863133c837cfb0ff394b7a658ce4d
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 12 11:47:26 2022 +0200

    sword mapping: Add support for Atom author elements
    
    Atom documents from old deposits usually use these instead of codemeta's

commit 30d195ccd64ba1c79379d444dff108a2314bc3b0
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 11 11:57:52 2022 +0200

    metadata_dictionary: Add mapping for SWORD/Atom with Codemeta

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/416/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/416/console

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 12 2022, 11:56 AM
Harbormaster failed remote builds in B30767: Diff 29715!

Build has FAILED

Patch application report for D8237 (id=29715)

Could not rebase; Attempt merge onto bb9082a6e5...

Updating bb9082a..03d8e1c
Fast-forward
 docs/metadata-workflow.rst                         |   2 +-
 swh/indexer/codemeta.py                            |   5 +-
 swh/indexer/metadata_dictionary/__init__.py        |   1 +
 swh/indexer/metadata_dictionary/codemeta.py        |  94 ++++++++++-
 .../tests/metadata_dictionary/test_codemeta.py     | 176 +++++++++++++++++++++
 swh/indexer/tests/test_cli.py                      |  12 +-
 6 files changed, 283 insertions(+), 7 deletions(-)
Changes applied before test
commit 03d8e1cbc5f863133c837cfb0ff394b7a658ce4d
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 12 11:47:26 2022 +0200

    sword mapping: Add support for Atom author elements
    
    Atom documents from old deposits usually use these instead of codemeta's

commit 30d195ccd64ba1c79379d444dff108a2314bc3b0
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 11 11:57:52 2022 +0200

    metadata_dictionary: Add mapping for SWORD/Atom with Codemeta

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/419/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/419/console

I can't reproduce this build failure :/

it seems unrelated, anyway

This revision is now accepted and ready to land.Aug 23 2022, 10:01 AM
This revision is now accepted and ready to land.Aug 23 2022, 11:33 AM

Build is green

Patch application report for D8237 (id=29935)

Could not rebase; Attempt merge onto e8cafad61c...

Updating e8cafad..13e9bf1
Fast-forward
 docs/metadata-workflow.rst                         |   2 +-
 swh/indexer/codemeta.py                            |   5 +-
 swh/indexer/metadata_dictionary/__init__.py        |   1 +
 swh/indexer/metadata_dictionary/codemeta.py        |  94 ++++++++++-
 .../tests/metadata_dictionary/test_codemeta.py     | 176 +++++++++++++++++++++
 swh/indexer/tests/test_cli.py                      |  12 +-
 6 files changed, 283 insertions(+), 7 deletions(-)
Changes applied before test
commit 13e9bf18c2d6c138183ef42af6928cea473b75b0
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 12 11:47:26 2022 +0200

    sword mapping: Add support for Atom author elements
    
    Atom documents from old deposits usually use these instead of codemeta's

commit 90dad275308fd0d7e70cb3a50cd395912f0da733
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 11 11:57:52 2022 +0200

    metadata_dictionary: Add mapping for SWORD/Atom with Codemeta

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