Open Boruch-Baum opened 1 year ago
Hi thanks for the PR. Changes/improvements to setup.py are welcome, but please leave it in it's current location.
The changes won't work unless the file is moved. That's a central point of the PR: the file is placed counter to the design, instruction, and expectations of python 'setuptools'. It's also inconsistent with the pyrproject.toml file, which IS in the correct location per 'setuptools'.
On2023-01-23 16:34, Campbell Barton wrote:
Hi thanks for the PR. Changes/improvements to setup.py are welcome, but please leave it in it's current location.
— Reply to this email directly, [1]view it on GitHub, or [2]unsubscribe. You are receiving this because you authored the thread. Message ID: @.***>
References
-- hkp://keys.gnupg.net CA45 09B5 5351 7C11 A9D1 7286 0036 9E45 1595 8BC0
In this case I rather leave it as is. If there are mistakes or problems with setup.py
not working, that should be addressed, last I tested it seemed to work OK. But it should not be necessary to have this in the projects root directory.
Hi! I'm writing this pull request using nerd dictation. Last week, I had surgery and have not been able to type with both hands so I looked for an application like this and installed it. In the course of installing I observed a few things that I could contribute and hence this pull request. Thanks very much for putting this together; I hope this contribution will be helpful; I have a few other ideas that I can also potentially contribute.
My operating environment is Debian, using python 3.10. The following is the default boilerplate that magit produced for me:
file setup.py had been placing the following files and directories in improper places as follows:
installed directory doc to ~/.local/lib/python3.10/site-packages/
installed executable nerd-dictation to ~/.local/lib/python3.10/site-packages/
installed directory nerd-dictation to ~/.local/lib/
installed executable and readme.rst to ~/.local/nerd-dictation/
file setup.py had NOT been performing the following:
installing directory doc to ~/.local/share/doc/nerd-dictation
installing documentation beyond readme.rst
installing example nerd-dictation.py files
installing executable nerd-dictation to ~/.local/bin
file setup.py suffered from needless complexity due to being placed in a sub folder instead of the based directory as is standard and expected practice
file pyproject.toml did not contain the minimum content as prescribed by package setuptools' documentation
a config file nerd-dictation.py is described as default, but had not been installed by default, so it wasn't really the default.