Skip to content
This repository has been archived by the owner on Oct 19, 2024. It is now read-only.

Dependencies in requirements.txt have module conflicts. #482

Open
unsatisfying opened this issue Jul 27, 2023 · 4 comments
Open

Dependencies in requirements.txt have module conflicts. #482

unsatisfying opened this issue Jul 27, 2023 · 4 comments

Comments

@unsatisfying
Copy link

Background

Dependencies in requirements.txt have module conflicts.

Description

There are two dependencies mentioned in the requirements.txt file: ffmpeg-python and ffmpeg. There is a module conflict between these two packages. The ffmpeg/__init__.py module in the ffmpeg package is an empty file, while the ffmpeg/__init__.py module in the ffmpeg-python package contains code. During the pip installation process, the latter module will overwrite the former ffmpeg/__init__.py.

Steps to Reproduce

pip install -r requirements.txt
This is the directory structure and file contents after installation, where it can be seen that the modules of the two packages are installed in the same folder, and init.py has been overwritten.
image

Desired Change

Indeed, it is not an ideal behavior for modules to be overwritten, even if they are not actively used or if the overwritten module is the one being called. It introduces uncertainty and can cause issues in the long run, especially if there are changes or updates to the overwritten modules in future development. It is generally recommended to avoid such conflicts and ensure that only the necessary and compatible dependencies are declared in the requirements to maintain a stable and predictable environment for the project.

@jantic
Copy link
Owner

jantic commented Aug 26, 2023

Thanks for digging into this and reporting it so thoroughly!

@SarthakNikhal
Copy link

@jantic Has this issue been resolved?

@jantic
Copy link
Owner

jantic commented Jan 14, 2024

@jantic Has this issue been resolved?

It hasn't. I don't think I'll be getting to this any time soon.

@snynisada
Copy link

snynisada commented Jan 14, 2024 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants