Changeset View
Standalone View
setup.py
Show First 20 Lines • Show All 48 Lines • ▼ Show 20 Lines | setup( | ||||
install_requires=parse_requirements() + parse_requirements("swh"), | install_requires=parse_requirements() + parse_requirements("swh"), | ||||
setup_requires=["setuptools-scm"], | setup_requires=["setuptools-scm"], | ||||
use_scm_version=True, | use_scm_version=True, | ||||
extras_require={"testing": parse_requirements("test")}, | extras_require={"testing": parse_requirements("test")}, | ||||
include_package_data=True, | include_package_data=True, | ||||
entry_points=""" | entry_points=""" | ||||
[swh.workers] | [swh.workers] | ||||
loader.mercurial=swh.loader.mercurial:register | loader.mercurial=swh.loader.mercurial:register | ||||
[console_scripts] | |||||
ardumont: @douardda ^ do we still need this?
I recall using the `swh_cli_group` is the way now, isn't it? | |||||
Done Inline Actionsas an example 5cc573d ardumont: as an example 5cc573d | |||||
Done Inline Actionsswh-identify is declared like this: [console_scripts] swh-identify=swh.model.cli:identify Since the mercurial version has the same purpose I used the same declaration. swh-identify is either not up to date or different on purpose? acezar: `swh-identify` is declared like this:
```
[console_scripts]
swh-identify=swh. | |||||
Done Inline ActionsActually it got both declaration [1] so we can choose to either use:
Note that the commit I linked actually drops the old declaration. Either way, that's not a blocker per say (we can always improve on this later). [1] https://forge.softwareheritage.org/source/swh-model/browse/master/setup.py$61-64 ardumont: Actually it got both declaration [1] so we can choose to either use:
- `swh identify` (and i… | |||||
Done Inline Actionsunless I'm mistaken, this is not supposed to be a subcommand of the main swh identify for now, so yes, we expect it to a separate command for now. For now, swh identify has no plugin-like architecture for addigin support for specific VCS, and since it's not a click group but a command, we cannot easily "inject" the hg-identify there. So until we have a better solution, let's keep it a dedicated command. douardda: unless I'm mistaken, this is not supposed to be a subcommand of the main `swh identify` for now… | |||||
Done Inline Actionsthanks for the clarification ;) ardumont: thanks for the clarification ;) | |||||
swh-hg-identify=swh.loader.mercurial.identify:main | |||||
""", | """, | ||||
classifiers=[ | classifiers=[ | ||||
"Programming Language :: Python :: 3", | "Programming Language :: Python :: 3", | ||||
"Intended Audience :: Developers", | "Intended Audience :: Developers", | ||||
"License :: OSI Approved :: GNU General Public License v3 (GPLv3)", | "License :: OSI Approved :: GNU General Public License v3 (GPLv3)", | ||||
"Operating System :: OS Independent", | "Operating System :: OS Independent", | ||||
"Development Status :: 4 - Beta", | "Development Status :: 4 - Beta", | ||||
], | ], | ||||
project_urls={ | project_urls={ | ||||
"Bug Reports": "https://forge.softwareheritage.org/maniphest", | "Bug Reports": "https://forge.softwareheritage.org/maniphest", | ||||
"Funding": "https://www.softwareheritage.org/donate", | "Funding": "https://www.softwareheritage.org/donate", | ||||
"Source": ("https://forge.softwareheritage.org/source/swh-loader-mercurial"), | "Source": ("https://forge.softwareheritage.org/source/swh-loader-mercurial"), | ||||
"Documentation": "https://docs.softwareheritage.org/devel/swh-loader-mercurial/", # NoQA: E501 | "Documentation": "https://docs.softwareheritage.org/devel/swh-loader-mercurial/", # NoQA: E501 | ||||
}, | }, | ||||
) | ) |
@douardda ^ do we still need this?
I recall using the swh_cli_group is the way now, isn't it?