Skip to content

Commit

Permalink
Revert ":fire: Delete MANIFEST file"
Browse files Browse the repository at this point in the history
This reverts commit 6b51403.

The commit never made sense. It deleted MANIFEST, but a couple years
later MANIFEST.in got added right back, which included the same files
MANIFEST did. And the commit added a data_files argument to setup.py
which was totally invalid -- it tried to install README.rst as
/usr/README.rst which was clearly wrong.

The original intention appears to have been to borrow the side effect of
data_files, and have the files appear in the sdist so setup.py could
read them, but it was the wrong way to do it.

Fixes: ikegami-yukino#28
  • Loading branch information
eli-schwartz committed Nov 29, 2024
1 parent dbc10ef commit 6a1c0fc
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@
'Programming Language :: Python :: 3.11',
'Programming Language :: Python :: 3.12', 'Topic :: Text Processing'
],
data_files=[('', ['README.rst', 'CHANGES.rst'])],
long_description='%s\n\n%s' %
(open('README.rst', encoding='utf8').read(),
open('CHANGES.rst', encoding='utf8').read()),
Expand Down

0 comments on commit 6a1c0fc

Please sign in to comment.