fedora-commops / fedora-happiness-packets

#130 Dependencies for Windows platform are different than those specified in `requirements/base.txt`
Closed: complete by phoenixabhishek. Opened by phoenixabhishek.

Closed: complete
Reopen Issue

Summary

Dependencies for Windows platform are different than those specified in requirements/base.txt

Background

When executing manage.py test -v 2 -p integration_test*.py --settings=happinesspackets.settings.tsting command, error occurs at dependency of pyreadline==2.1 in the file requirements/base.txt.

This dependency is specific to UNIX-based systems (Linux, Ubuntu and Mac OS)

Details

The equivalent for windows is gnureadline==6.3.3 dependency.
This dependency is checked for and it works.

Outcome

Executing manage.py test -v 2 -p integration_test*.py --settings=happinesspackets.settings.tsting command completes successfully

phoenixabhishek commented

@jflory7 I faced this issue on my Windows system and it was found that the dependency caused the error.

I would like this issue to be assigned to me.

Also, I was thinking it will be appropriate to add this part(of changing this dependency in the requirements/base.txt file manually) in the windows-specific project setup document.

Your thoughts..???
Edited by phoenixabhishek

Metadata Update from @jflory7:
- Issue priority set to: waiting on assignee (was: awaiting triage)
- Issue tagged with: PASSED, bug, needs info, type - summer coding

Metadata Update from @jflory7:
- Issue untagged with: PASSED

@phoenixabhishek To help me understand, are you using Docker for your local environment? To run the integration tests, you should exec into the container and run the command there. If you're running the commands in the Docker container, I'm not sure how this could happen.
phoenixabhishek commented

@jflory7 yes, I did exec in the docker container that was up.
I think this might be because in the Dockerfile in our project, we have a command RUN pip install -r /requirements/dev.txt.
The dev.txt file starts with -r base.txt, and this dependency is specified there (in base.txt).

Update

To check further on this issue I setup my environment afresh, and this time I observed that the gnureadline==6.3.3 dependency is already present in place of pyreadline==2.1.

I checked with the main repo too, and saw the same.

Was this file changed recently..??? 🤔🤔🤔🤔🤔

Can you please check this once.

If this is already done, we can close this ticket without having to change anything at all :smile: :smile: :smile: :smile: :smile:

To check further on this issue I setup my environment afresh, and this time I observed that the gnureadline==6.3.3 dependency is already present in place of pyreadline==2.1.

I checked with the main repo too, and saw the same.

Was this file changed recently..??? 🤔🤔🤔🤔🤔

I checked the files and don't see anything changed recently. Not sure what happened here. :mag: :question:

Can you please check this once.

Are you still able to reproduce this error? I'm not on a Windows environment and don't have a way to test this. If you cannot reproduce it, feel free to go ahead and close this. :smile:

phoenixabhishek commented

Are you still able to reproduce this error? I'm not on a Windows environment and don't have a way to test this. If you cannot reproduce it, feel free to go ahead and close this. 😄

No, it's no more happening on my system.
I'll go ahead and close this :smile: :smile: :smile: :smile: :smile:

Metadata Update from @phoenixabhishek:
- Issue close_status updated to: complete
- Issue status updated to: Closed (was: Open)
Metadata
None

bug needs info type - summer coding

None
None
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.