Irit Katriel added the comment:
As per Camilla's comment, this is no longer an issue.
--
nosy: +iritkatriel
resolution: accepted -> wont fix
stage: needs patch -> resolved
status: open -> closed
___
Python tracker
Camilla Montonen added the comment:
Apologies, I should have clarified:
I can still replicate the bug in the original post, but I no longer believe
this is an issue, because the wording in the documentation has been changed
for Python 2.X
Camilla Montonen added the comment:
This is still an issue in Python 3.4.3, but I believe the relevant
documentation has been changed already to alert users to the fact that the
class name and the command name should be the same.
Quoting from:
Changes by Éric Araujo mer...@netwok.org:
--
Removed message: http://bugs.python.org/msg102824
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6860
___
Éric Araujo mer...@netwok.org added the comment:
This is a murky area, I’m not sure we have to refactor so much just to fix this
bug. Maybe it’s a doc bug: People have to define command_name if their command
has a name that is different from the class name. Maybe it’s a simple fix in
the
Changes by Éric Araujo mer...@netwok.org:
--
stage: - needs patch
versions: +Python 2.5, Python 2.6, Python 3.1
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6860
___
Éric Araujo mer...@netwok.org added the comment:
Hello
Distutils being frozen, I’m reassigning to Distutils2. Not sure if I should
make versions blank, 3.3 or third-party, so leaving it alone.
Regards
--
components: +Distutils2 -Distutils
___
Changes by Éric Araujo mer...@netwok.org:
--
nosy: +merwok
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6860
___
___
Python-bugs-list mailing
Tarek Ziadé ziade.ta...@gmail.com added the comment:
It's a bit more complicated.
the option you are desrcibing it just one way to register commands in fact.
Distutils also has a discovery function that will load commands from
packages, and that's where the biggest issue is.
--
Tarek Ziadé ziade.ta...@gmail.com added the comment:
To be able to do this fix, I also need to change the way commands are
registered in Distutils.
Right now, Distutils scans packages that were provided as command
packages and just adds all commands from the namespace, using the class
name.
Patrick Näf herzb...@herzbube.ch added the comment:
To be able to do this fix, I also need to change the way commands are
registered in Distutils.
Hm, I thought commands were registered in the setup() function with the
cmdclass dict. Like this:
setup(
# test is the name that should be
Tarek Ziadé ziade.ta...@gmail.com added the comment:
Right, thanks for noticing this.
Here's the change I am going to make:
The code will use command.get_command_name() *everywhere* for the help
display.
This method implemented in Command does the following:
- if the attribute command_name
New submission from Patrick Näf herzb...@herzbube.ch:
The attached setup.py file defines a custom command named test, which
is implemented in a class named TestClass. Try to run both of the
following:
1) ./setup.py test -h
2) ./setup.py --help-commands
In case 1, Distutils will use the class
13 matches
Mail list logo