Differential D1729 Diff 5806 swh/loader/gnu/loader.py

Changeset View

Standalone View

swh/loader/gnu/loader.py

This file was added.

				# Copyright (C) 2019 The Software Heritage developers
				# See the AUTHORS file at the top-level directory of this distribution
				# License: GNU General Public License version 3, or any later version
				# See top-level LICENSE file for more information


				from swh.loader.base.loader import BaseLoader
				from swh.loader.base.dowload import compare_field


				class GNULoader(BaseLoader, compare_field):
				loader_name = 'gnu'
				class_name = 'GNULoader'
				compare_field = 'date'

				anlambertUnsubmitted Not Done Inline Actions Here you should add these: CONFIG_BASE_FILENAME = 'loader/gnu' ADDITIONAL_CONFIG = { 'temp_directory': ('str', '/tmp/swh.loader.gnu/'), 'debug': ('bool', False) } in order to declare the config filename and some default config entries for the loader. anlambert: Here you should add these: ```lang=python CONFIG_BASE_FILENAME = 'loader/gnu' ADDITIONAL_CONFIG…
				nahimilegaUnsubmitted Not Done Inline Actions This was present in base loader class itself, it would assign config filename and some default config entries on the bases of loader name. but after a discussion with @ardumont in IRC and via these comments(https://forge.softwareheritage.org/D1694?id=5884#inline-11570 ) it was removed. There are three things what we can do add it again in the base loader add them separately for each loader Discard these config filenames and some default config entries completely and use environment variable stanza(as discussed in comment) What should I do? nahimilega: This was present in base loader class itself, it would assign config filename and some default…
				def convert_to_standard_format(**kwargs):
				return {
				'name': kwargs['name'],
				'origin_url': kwargs['origin_url'],
				'tarballs': kwargs['tarballs'],
				anlambertUnsubmitted Not Done Inline Actions cf https://forge.softwareheritage.org/D1694#inline-11749 anlambert: cf https://forge.softwareheritage.org/D1694#inline-11749
				}