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

setup.py: use open instead of file #3

Open
wants to merge 6 commits into
base: multi-protocol
Choose a base branch
from

Conversation

groleo
Copy link

@groleo groleo commented Sep 4, 2024

When doing a pip3 install, the file from long_description is not defined.

Collecting https://github.com/enix223/modem/archive/refs/heads/multi-protocol.zip (from -r requirements.txt (line 5))
Downloading https://github.com/enix223/modem/archive/refs/heads/multi-protocol.zip
\ 184kB 2.4MB/s
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-2kc3v90y-build/setup.py", line 10, in
long_description = file('doc/source/about.rst').read(),
NameError: name 'file' is not defined

When doing a pip3 install, the `file` from long_description is not defined.

Collecting https://github.com/enix223/modem/archive/refs/heads/multi-protocol.zip (from -r requirements.txt (line 5))
  Downloading https://github.com/enix223/modem/archive/refs/heads/multi-protocol.zip
     \ 184kB 2.4MB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-2kc3v90y-build/setup.py", line 10, in <module>
        long_description = file('doc/source/about.rst').read(),
    NameError: name 'file' is not defined
@groleo groleo force-pushed the multi-protocol branch 9 times, most recently from b3bc07c to d3168f8 Compare September 4, 2024 12:25
Adrian Negreanu added 2 commits September 4, 2024 15:47
packages should include the subdirectories/sub-modules.
Otherwise, only the files in modem/ will be packaged (without subdirs).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant