openedx / openedx/xqueue-watcher

Can't import xqueue-watcher to other project as a package

Open
#53 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
18
Forks
44
PR merge metrics
No merged PRs in 30d

Description

Hi all.

I'd like to use xqueue-watcher as a package to create my own watcher as child class of Grader, as described here.
In order to do that I created new project and tried to import the watcher there via a command:
pip install -r requirements.txt
there a content of the requirements file is
-e git+https://github.com/edx/xqueue-watcher#egg=xqueue-watcher

The process failed with error
'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Invalid requirement, parse error at "'-e git+h'"

After I tried to clone this project and build it via a command
python setup.py install
and got the same error.

Looks like an issue is install_requires=open('requirements/production.txt', 'rb').readlines() string in setup.py and git codejail dependecy in requirements files.

Is it really the issue or do I make something wrong?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the failure with the requirements.txt command and inspect setup.py, requirements/production.txt, and the referenced requirements files. Check how install_requires is built and how the codejail dependency is specified; done means the project installs without the invalid requirement parse error and can be imported as a package.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
build-system
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.