[issue7202] "python setup.py cmd --verbose" does not set verbosity

[Steve Dower]

Steve Dower  added the comment:

Distutils is now deprecated (see PEP 632) and all tagged issues are being 
closed. From now until removal, only release blocking issues will be considered 
for distutils.

If this issue does not relate to distutils, please remove the component and 
reopen it. If you believe it still requires a fix, most likely the issue should 
be re-reported at https://github.com/pypa/setuptools

--
nosy: +steve.dower
resolution:  -> out of date
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue7202] "python setup.py cmd --verbose" does not set verbosity

[Giovanni Cappellotto]

Giovanni Cappellotto  added the comment:

I took a stab at this, see attached PR.

I was able encode the first two cases described by @tarek, but not the first 
one because `_parse_command_opts` doesn't have visibility to the list of global 
options that have already been parsed.

Note:

I had to move `log.set_verbosity(self.verbose)` after the call to 
`_parse_command_opts` in order to correctly apply the verbosity level after all 
command options are parsed. I think there should be a better way to handle this 
case, maybe by creating a `verbose` setter that re-runs `log.set_verbosity` 
every time `verbose` value is updated.

--
nosy: +potomak

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue7202] "python setup.py cmd --verbose" does not set verbosity

[Giovanni Cappellotto]

Change by Giovanni Cappellotto :


--
keywords: +patch
pull_requests: +14585
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/14787

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue7202] "python setup.py cmd --verbose" does not set verbosity

[Milan Oberkirch]

Milan Oberkirch  added the comment:

The help states:

> usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]

so I would argue that setup.py ignoring global_opts after cmd1 is not a bug.

But I do think that the fact that "python setup.py build --dry-run" does not 
prevent the build from running is not ideal.

Maybe throwing an error for misplaced global options is the best solution?

--
nosy: +zvyn

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue7202] "python setup.py cmd --verbose" does not set verbosity

[Avinash Sajjanshetty]

Avinash Sajjanshetty  added the comment:

I would like to take this up task and propose a patch. However, I am not able 
to reproduce this issue on my machine (3.7.1). Does the issue seem to be fixed?

Here is what I tried: 

1. 

python setup.py --verbose build

I got the verbose build, as expected. 

2. 

python setup.py build --verbose 

In this case, it seems to have ignored the verbose option, as I got a normal 
(non-verbose) output. 

3. since the issue is with `verbose`, I tried quiet. 

python setup.py --quiet build

I got a quiet build. 

4. 

python setup.py build --quiet 


it ignored the quiet option, as I got a normal output.


5. I also tried a non-existent option. In the following both cases, I got an 
error saying 

error: option --qqqp not recognized


for 


python setup.py --qqqp build


and 


python setup.py build --qqqp

--
nosy: +avi

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue7202] "python setup.py cmd --verbose" does not set verbosity

[Mark Lawrence]

Change by Mark Lawrence :


--
nosy:  -BreamoreBoy

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue7202] python setup.py cmd --verbose does not set verbosity

[Mark Lawrence]

Mark Lawrence added the comment:

Assuming that this is still an issue would someone like to propose a patch?

--
components:  -Distutils2
nosy: +BreamoreBoy, dstufft
versions: +Python 3.4, Python 3.5 -3rd party, Python 3.1, Python 3.2

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7202
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue7202] python setup.py cmd --verbose does not set verbosity

[Éric Araujo]

Changes by Éric Araujo mer...@netwok.org:


--
components: +Distutils2
keywords: +easy
resolution: accepted - 
title: python setup.py MYCOMMAND --verbose does not yield an unrecognized 
option error but also does not set the verbosity - python setup.py cmd 
--verbose does not set verbosity
type:  - behavior
versions: +3rd party, Python 2.7, Python 3.1, Python 3.2

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7202
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com