STINNER Victor added the comment:
This issue has no activity for 5 years. regrtest output evolved a lot since
that one. I like "verbose" output on buildbots to be able to debug when
something goes wrong.
--
resolution: -> out of date
stage: patch review -> resolved
status: open ->
Roundup Robot added the comment:
New changeset 8c337b4a8811 by Berker Peksag in branch 'default':
Issue #17554: Print fetching url ... messages only in verbose mode.
https://hg.python.org/cpython/rev/8c337b4a8811
--
nosy: +python-dev
___
Python
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17554
___
Berker Peksag added the comment:
issue17554-urlfetch.diff LGTM.
--
stage: needs patch - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17554
___
Changes by Berker Peksag berker.pek...@gmail.com:
--
nosy: +berker.peksag
versions: +Python 3.5 -Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17554
___
Changes by Florent Xicluna florent.xicl...@gmail.com:
--
nosy: +flox
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17554
___
___
Python-bugs-list
Ezio Melotti added the comment:
Attached patch shows the Fetching url ... only in verbose mode.
--
Added file: http://bugs.python.org/file29670/issue17554-urlfetch.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17554
Ned Deily added the comment:
There definitely needs to be an option to continue to include the normal
progress list in the output. This is essential for comparing test runs. I'm
+0 on whether it be the default or not. Actually, what is really needed for
automated analysis of test runs is
Ezio Melotti added the comment:
I fixed all the tests that were producing extra output on my machine, but there
are probably some left on other machines. Using -uall also produces extra
output, and I haven't decided how to deal with that yet (maybe it should be
silenced as well).
Getting
R. David Murray added the comment:
-uall shouldn't produce more output by itself, it just runs more tests. Do you
mean that some *output* is actually conditionalized on whether a certain
resource is enabled?
--
___
Python tracker
Ezio Melotti added the comment:
The tests enabled by -uall (and probably -unetwork) use open_urlresource, and
open_urlresource has a print('\tfetching %s ...' % url,
file=get_original_stdout()). This should probably be printed only when -v is
passed.
--
Ezio Melotti added the comment:
I would prefer to have it by default with a flag to disable it, since IMHO the
full list of passed tests is just noise, but other devs don't seem to like the
idea.
Another option would be to save the flags in a config file or in an envvar, so
that it won't be
Antoine Pitrou added the comment:
One reason I think the full list is better is that it's easier for beginners to
understand and review what's happening (and also get a taste of the contents of
the test suite). Experienced contributors wouldn't care that much admittedly.
--
Ezio Melotti added the comment:
They will still see the test names and the counter that increases. I would
also say that having the whole output fitting in a screen is less intimidating
than a few screens filled with a wall of text. It also makes easier to notice
skipped and failing tests,
Brett Cannon added the comment:
So are you saying you want both the short-form output while running but then
the whole output upon completion, David? I can understand wanting the summaries
still in order to sort the skipped tests vs. failures, but not outputting every
test that succeeded
R. David Murray added the comment:
No, what I'd prefer is that the current print it as it runs behavior not
change, but that the list of skip reasons be displayed at the end. after all
the tests have completed. This is probably not a realistic request, so I'm
fine with just having Ezio's
Brett Cannon added the comment:
I say move forward and put it behind a flag (w/ quiet is fine, maybe some arg
to specify quietness or -qq much like -vv?).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17554
New submission from Ezio Melotti:
The attached patch is a proof of concept that changes the output of regrtest so
that the [xxx/yyy] test_zz is updated in place instead of printing each
test on a separate line. Failures, skips, and the final report are printed
normally, making it easier
Changes by Ezio Melotti ezio.melo...@gmail.com:
Added file: http://bugs.python.org/file29586/regroutput.txt
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17554
___
R. David Murray added the comment:
I would like both the normal (non-overwritten) progress list and the summary at
the end :)
I would not like the overwriting mode to be the default, but otherwise it looks
good.
--
___
Python tracker
R. David Murray added the comment:
Oh, by 'looks good' I was referring to the output, not the patch, which I
haven't looked at yet.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17554
21 matches
Mail list logo