search

phac-nml / mob-suite

MOB-suite: Software tools for clustering, reconstruction and typing of plasmids from draft assemblies
Apache License 2.0
124 stars 33 forks source link

Add missing comma to entry_points #147

Closed mencian closed 1 year ago

mencian commented 1 year ago

setup.py is missing a comma on line 61 after the 'Sequenoscope=Sequenoscope:main' entry_point and causes parsing errors in the setup command. I am currently attempting to update this software's Bioconda recipe: bioconda/bioconda-recipes#42131 and ran into this during the process.

This PR adds a comma on line 61 in setup.py, which fixes the parse error.

kimTina321 commented 1 year ago

I still met the error message below. Are there anymore problems with the setup.py?

Collecting mob_suite Downloading mob_suite-3.1.5.tar.gz (210 kB) ━━━━━━━━━━━━━━━━━━━━━ 210.6/210.6 kB 14.0 MB/s eta 0:00:00 Preparing metadata (setup.py) ... error error: subprocess-exited-with-error

× python setup.py egg_info did not run successfully. │ exit code: 1 ╰─> [2 lines of output] error in mob_suite setup command: Problems to parse EntryPoint(name='Sequenoscope', value='Sequenoscope:mainmob_init=mob_suite.mob_init:main', group='console_scripts'). Please ensure entry-point follows the spec: https://packaging.python.org/en/latest/specifications/entry-points/ [end of output]

jrober84 commented 1 year ago

The included new entry points are in error, thank you fo bringing them to my attention. I will remove it.