Details

Reviewers

Group Reviewers

Commits

rDLDBASE320bf5eb7ef8: loader.package: Remove dead code
rDLDBASEabc80f53ad78: package.pypi: Move pypi loader to its own namespace
rDLDBASEa45f3f5e758e: package.npm: Move npm loader to its own namespace
rDLDBASE3a6fc1225b49: package.debian: Move debian loader to its own namespace
rDLDBASE91c2df13b39e: package.deposit: Move deposit loader to its own namespace
rDLDBASE4320469725e0: package.archive: Move archive loader to its own namespace

Summary

This is pre-requisite work towards the unified cli.

(Other package loaders have been migrated the same way, i won't open similar diffs as this though, it's time consuming for everyone for little benefit)

Note:
This implies a scheduler's task_type migration on their backend name (P567)

Related to T2134

Test Plan

tox

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 4 2019, 1:47 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptDec 4 2019, 1:47 PM

Build is green
See https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/254/ for more details.

Harbormaster completed remote builds in B9506: Diff 8427.Dec 4 2019, 1:49 PM

Reuse the same conftest configuration between tests (instead of duplication)

Build is green
See https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/255/ for more details.

Harbormaster completed remote builds in B9512: Diff 8433.Dec 4 2019, 3:36 PM

ardumont edited the summary of this revision. (Show Details)Dec 4 2019, 4:07 PM

ardumont added projects: Core Loader, SWH command line interface.

Adapt setup.py accordingly

Build is green
See https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/256/ for more details.

Harbormaster completed remote builds in B9513: Diff 8434.Dec 4 2019, 4:17 PM

ardumont mentioned this in D2397: loader.cli: Reference new cli.Dec 4 2019, 7:45 PM

ardumont edited the summary of this revision. (Show Details)Dec 5 2019, 8:39 AM

ardumont edited the summary of this revision. (Show Details)Dec 5 2019, 9:13 AM

ardumont edited the summary of this revision. (Show Details)

LGTM

This revision is now accepted and ready to land.Dec 5 2019, 12:04 PM

package.npm: Move npm loader to its own namespace
package.pypi: Move pypi loader to its own namespace
loader.package: Remove dead code

Harbormaster failed remote builds in B9521: Diff 8442!Dec 5 2019, 12:30 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/259/
See console output for more information: https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/259/console

Diff against origin/master and plug to master branch

Build is green
See https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/260/ for more details.

Harbormaster completed remote builds in B9522: Diff 8443.Dec 5 2019, 12:33 PM

Closed by commit rDLDBASE4320469725e0: package.archive: Move archive loader to its own namespace (authored by ardumont). · Explain WhyDec 5 2019, 12:34 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDLDBASE4320469725e0: package.archive: Move archive loader to its own namespace.

ardumont added a commit: rDLDBASE3a6fc1225b49: package.debian: Move debian loader to its own namespace.

ardumont added a commit: rDLDBASE91c2df13b39e: package.deposit: Move deposit loader to its own namespace.

ardumont added a commit: rDLDBASEa45f3f5e758e: package.npm: Move npm loader to its own namespace.

ardumont added a commit: rDLDBASEabc80f53ad78: package.pypi: Move pypi loader to its own namespace.

ardumont added a commit: rDLDBASE320bf5eb7ef8: loader.package: Remove dead code.

ardumont mentioned this in rDENV94d82520c7ba: loader: Migrate the loader package tasks.Dec 6 2019, 9:13 AM

anlambert mentioned this in T2456: Source code view should span full browser width.Jun 17 2020, 12:19 PM

package.archive: Move archive loader to its own namespace
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 8444

MANIFEST.in

conftest.py

setup.py

swh/loader/package/archive.py

swh/loader/package/archive/init.py

swh/loader/package/archive/loader.py

swh/loader/package/archive/tasks.py

swh/loader/package/archive/tests/init.py

swh/loader/package/archive/tests/data/https_ftp.gnu.org/gnu_8sync_8sync-0.1.0.tar.gz

swh/loader/package/archive/tests/data/https_ftp.gnu.org/gnu_8sync_8sync-0.2.0.tar.gz

swh/loader/package/archive/tests/test_archive.py

swh/loader/package/archive/tests/test_tasks.py

swh/loader/package/tasks.py

swh/loader/package/tests/conftest.py

swh/loader/package/tests/data/https_ftp.gnu.org/gnu_8sync_8sync-0.1.0.tar.gz

swh/loader/package/tests/data/https_ftp.gnu.org/gnu_8sync_8sync-0.2.0.tar.gz

swh/loader/package/tests/test_archive.py

swh/loader/package/tests/test_tasks.py

package.archive: Move archive loader to its own namespaceClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 8444

MANIFEST.in

conftest.py

setup.py

swh/loader/package/archive.py

swh/loader/package/archive/__init__.py

swh/loader/package/archive/loader.py

swh/loader/package/archive/tasks.py

swh/loader/package/archive/tests/__init__.py

swh/loader/package/archive/tests/data/https_ftp.gnu.org/gnu_8sync_8sync-0.1.0.tar.gz

swh/loader/package/archive/tests/data/https_ftp.gnu.org/gnu_8sync_8sync-0.2.0.tar.gz

swh/loader/package/archive/tests/test_archive.py

swh/loader/package/archive/tests/test_tasks.py

swh/loader/package/tasks.py

swh/loader/package/tests/conftest.py

swh/loader/package/tests/data/https_ftp.gnu.org/gnu_8sync_8sync-0.1.0.tar.gz

swh/loader/package/tests/data/https_ftp.gnu.org/gnu_8sync_8sync-0.2.0.tar.gz

swh/loader/package/tests/test_archive.py

swh/loader/package/tests/test_tasks.py

package.archive: Move archive loader to its own namespace
ClosedPublic
Actions

Revision Contents
Changeset List

swh/loader/package/archive/init.py

swh/loader/package/archive/tests/init.py