Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FileNotFoundError: [Errno 2] No such file or directory: 'requirements.txt' #55

Closed
rpanai opened this issue Sep 8, 2020 · 5 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@rpanai
Copy link

rpanai commented Sep 8, 2020

I'm trying to build a conda-forge package out of this but when I try to build the recipe with conda skeleton pypi opacus I have the following error

FileNotFoundError: [Errno 2] No such file or directory: 'requirements.txt'
$PYTHONPATH = /tmp/tmpm3lk0o9wconda_skeleton_opacus-0.9.0.tar.gz/opacus-0.9.0
@rpanai rpanai changed the title FileNotFoundError: [Errno 2] No such file or directory: 'requirements/main.txt' FileNotFoundError: [Errno 2] No such file or directory: 'requirements.txt' Sep 8, 2020
@Darktex Darktex self-assigned this Sep 10, 2020
@Darktex Darktex added the bug Something isn't working label Sep 10, 2020
@Darktex
Copy link
Contributor

Darktex commented Sep 10, 2020

First of all, THANK YOU for looking into this! :)

So, I went on a Google rabbit hole on this one and it looks like the issue is that we did not include all the relevant resources. This seems fixed via adding a MANIFEST.in file that contains the necessary includes: https://stackoverflow.com/questions/43748284/requirements-txt-file-for-my-package-missing-from-pypi-install-and-tarball

@ffuuugor can you add it and issue a new release this week?

@rpanai
Copy link
Author

rpanai commented Sep 10, 2020

Please let me know whenever the new release is out so I'll proceed.

P.S. I just edited the issue as it was basically a copy and paste on the one I made here

facebook-github-bot pushed a commit that referenced this issue Sep 14, 2020
Summary:
See #55

Out setup.py references `requirements.txt`, but the PyPi distribution doesn't include it by default (I actually checked https://pypi.org/project/opacus/)

Fixing by introducing MANIFEST.in

Reviewed By: karthikprasad

Differential Revision: D23649334

fbshipit-source-id: 879d3de3ca0f49225150e147931462885c671f66
@karthikprasad
Copy link
Contributor

The patch has landed, will be creating a new release this week.

@rpanai
Copy link
Author

rpanai commented Sep 21, 2020

@karthikprasad Any idea when the new release will be available?

@karthikprasad
Copy link
Contributor

@rpanai, the new release is out. Apologies for the delay; we wanted to get a few more changes in to this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants