Catherine Devlin added the comment:
All that said, it's easy to imagine a PyPI package that would give this
functionality for those who want it. Maybe even one that could also alter the
behavior of Click and Typer. If you want to go that route, let me know
Catherine Devlin added the comment:
Your PR doesn't include any tests... but I like the idea enough to create the
tests for you if you prefer.
However, first I'd like to see whether the core devs like the idea. I could
see saying either "yes, why not make things easy" or "n
Catherine Devlin added the comment:
Thinking about this a little more...
I suggest that slowing down execution shouldn't be a worry in this case,
because trying to parse a garbage string into a date shouldn't be something
code is doing zillions of times, and if it is, being slow isn't
Catherine Devlin added the comment:
Can we close the ticket? It doesn't sound like we're going to go with anything
like this approach.
If there's a good place to record the general ambitions (better strptime tests,
more clear strptime errors), that would be good, but I don't think we use
Change by Catherine Devlin :
--
keywords: +patch
nosy: +Catherine.Devlin
nosy_count: 2.0 -> 3.0
pull_requests: +24832
stage: test needed -> patch review
pull_request: https://github.com/python/cpython/pull/26215
___
Python tracker
Catherine Devlin added the comment:
I think that it's possible to get the desired behavior by setting a filter to
"always".
```
> cat warnme.py
import warnings
for i in range(3):
warnings.warn("oh noes!")
> python warnme.py
warnme.
Catherine Devlin added the comment:
What about a __repr__ that includes the primary key value(s) (for tables where
that is defined)? That would be useful and informative, and also generally
short.
--
nosy: +Catherine.Devlin
___
Python tracker
Change by Catherine Devlin :
--
keywords: +patch
nosy: +Catherine.Devlin
nosy_count: 8.0 -> 9.0
pull_requests: +24815
stage: test needed -> patch review
pull_request: https://github.com/python/cpython/pull/26198
___
Python tracker
Catherine Devlin added the comment:
@ZackerySpytz - I made https://github.com/python/cpython/pull/26196 with a test
for the desired behavior; hopefully it helps. I could try to adapt Barry's old
patch myself, but it's probably better if somebody C-competent does so
Change by Catherine Devlin :
--
nosy: +Catherine.Devlin
nosy_count: 11.0 -> 12.0
pull_requests: +24813
stage: test needed -> patch review
pull_request: https://github.com/python/cpython/pull/26196
___
Python tracker
<https://bugs.p
Catherine Devlin <catherinedev...@users.sourceforge.net> added the comment:
Hi, everybody! I really appreciate the kind words about cmd2.
For several years now, Todd Leonhardt has been cmd2's primary maintainer, so I
may be speaking out of turn. But, before I wanted to let it get
Catherine Devlin added the comment:
My last commit to the PR includes a fix by delaying setting permission to all
files, not just to directories, in .extractall().
It might be better to catch the problem during .add instead, preventing tarring
multiple copies, but I found subtle difficulties
Catherine Devlin added the comment:
Okay, the problem is a little more specific than my last message suggested, but
also a little less specific than the original report.
A "PermissionError: [Errno 13] Permission denied" is thrown when expanding a
tarfile to which a file had been
Changes by Catherine Devlin <catherine.dev...@gmail.com>:
--
pull_requests: +1892
___
Python tracker <rep...@bugs.python.org>
<http://bugs.python
Catherine Devlin added the comment:
I apologize, I retract my earlier comment - I believe both Yaroslav and I were
confused about the nature of the problem. I think it's not related to
permissions at all, but to adding a file to the tarfile twice.
To see this, use `tarfilero.py` as provided
Changes by Catherine Devlin <catherine.dev...@gmail.com>:
Removed file: http://bugs.python.org/file46890/issue_30438_test.patch
___
Python tracker <rep...@bugs.python.org>
<http://bugs.python
Catherine Devlin added the comment:
I confirmed the error, and that doing the corresponding tar/untar cycle with
the command-line `tar` utility succeeds.
issue_30438_test.patch adds a unittest version of Yaroslav's demo file to
test_tarfile.py. (It's irrelevant if the PR is merged
Catherine Devlin added the comment:
Implementation for the enhancement. Includes tests.
--
keywords: +patch
Added file: http://bugs.python.org/file39823/json_callable_sort_24518.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org
New submission from Catherine Devlin:
Right now, json.dumps can be called with True or False, but it would be easy to
also support accepting a key function, which then could be used to control the
order of keys arbitrarily in the serialized JSON output.
--
components: Library (Lib
Catherine Devlin added the comment:
Attaching new patch incorporating Vajrasky's suggestion
--
Added file: http://bugs.python.org/file31316/test1666318.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1666318
Catherine Devlin added the comment:
Thanks, Antoine - I've signed and submitted the Contributor's Agreement.
--
Added file: http://bugs.python.org/file31104/Python Contributor Agreement Form
- signed.pdf
___
Python tracker rep...@bugs.python.org
Catherine Devlin added the comment:
Attaching a test that verifies that shutil.copytree is, in fact, preserving
permissions (as of Python 3.4.0a0 (default:66a3dc613627, Jul 27 2013, 21:23:10)
)
As far as I can tell this can be closed.
--
nosy: +catherinedevlin
Added file: http
Catherine Devlin added the comment:
Needed to update the patch slightly for Python 3; now that filter() returns an
iterator, ``do_help``'s call to
names = self.get_names()
followed by
names.sort()
was throwing an error, so I changed get_names to return a list.
--
nosy
Catherine Devlin added the comment:
Change to test_cmd.py to test for help displaying the name of the registered
subcommand (as well as a simple test for the basic operation of the registered
sub-CLI).
--
Added file: http://bugs.python.org/file26594/test_cmd.patch
Catherine Devlin added the comment:
It's very hard to tell what ought to be done here, since Lib/urllib/request.py
throws URLErrors with a great variety of order and number of arguments, and
it's not clear how URLError (in Lib/urllib/error.py) intends to handle them.
However, in this case
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Very simplistic patch to test_calendar.py that adds a test for this fix. This
fails on the unpatched trunk b/c the unpatched output begins with bb'?xm
instead of b?xml . Test needs a quick review... I'm not familiar
Catherine Devlin fredv8vi...@liquidid.net added the comment:
This blog post
http://justcramer.com/2009/04/02/problems-uploading-packages-with-setuptools-on-os-x/
describes working around the same problem by replacing [pypi] in .pypirc with
[server-login]. Is that the problem, a change
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Did my suggestion to alter pydoc output so it always contains a link to the
enclosing module's documentation not seem like a reasonable compromise?
I actually don't understand how that would help. The ``pydoc time`` output
doesn't
Catherine Devlin fredv8vi...@liquidid.net added the comment:
There is the non-zero cost of keeping two copies of that bit of
information in-sync with each other (and the code).
That's true, but how often do the strftime format codes change? I'd be happy
to personally volunteer to keep them
New submission from Catherine Devlin fredv8vi...@liquidid.net:
Is there any reason not to include the strftime formatting codes in the
docstrings of time.strftime and time.strptime?
print time.strftime.__doc__
strftime(format[, tuple]) - string
Convert a time tuple to a string according
Catherine Devlin fredv8vi...@liquidid.net added the comment:
This patch fixes it with a fourth approach: if unrecognized arguments are found
during subparser parsing, information about them is inserted into the namespace
(under ._unrecognized), and the decision about whether to exit
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Thanks for the correction, Steven. This unit test matches your description.
Use it instead of my earlier submission.
--
Added file: http://bugs.python.org/file18390/wrong_metavars_test_corrected.patch
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Unit test for Drake's patch
--
nosy: +catherine
Added file: http://bugs.python.org/file18392/test_mutually_exclusive.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org
Catherine Devlin fredv8vi...@liquidid.net added the comment:
copy of Drake's argparse.diff - same patch, just has less specific diff header
info (because ``patch -p0 argparse.diff`` failed on my machine)
--
Added file: http://bugs.python.org/file18393/mutually_exclusive_help.patch
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Attaching a test to verify parse failure on mismatched prefix (-abc or +abc).
Steven's patch makes it pass.
--
nosy: +catherine
versions: -Python 2.7, Python 3.2
Added file:
http://bugs.python.org/file18342
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Updated Steven's patch; no changes, but now it knows the new context so that
``patch -p0`` won't fail.
--
Added file:
http://bugs.python.org/file18343/multiple_short_same_prefix_new.patch
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Some basic unit tests for parse_known_args on a subparser.
--
keywords: +patch
nosy: +catherine
Added file:
http://bugs.python.org/file18344/test_subparser_parse_known_args.patch
Changes by Catherine Devlin fredv8vi...@liquidid.net:
Removed file:
http://bugs.python.org/file18344/test_subparser_parse_known_args.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9340
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Some simple unit tests for parse_known_args on a parser with a subparser. They
are indeed failing on the trunk.
--
Added file:
http://bugs.python.org/file18346/test_subparser_parse_known_args.patch
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Attached a unit test patch corresponding to Ted's patch.
http://bugs.python.org/file18320/argparse_test.patch
--
nosy: +catherine
___
Python tracker rep...@bugs.python.org
http
Catherine Devlin fredv8vi...@liquidid.net added the comment:
I'm attaching a unit test patch that does something vaguely like Steven
suggests. It definitely needs review.
Also, I'm assuming that it's OK to specify *fewer* metavars than nargs, just
not more... but I'm not sure about
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Here's a unit test for the simplest cases.
--
keywords: +patch
nosy: +catherine
Added file: http://bugs.python.org/file18328/test_pos_after_var_args.patch
___
Python tracker rep
Changes by Catherine Devlin fredv8vi...@liquidid.net:
--
nosy: -ted.turocy
Added file: http://bugs.python.org/file18320/argparse_test.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9444
Catherine Devlin fredv8vi...@liquidid.net added the comment:
Patch created live during PyOhio 2010 Teach Me Python Bugfixing session!
--
nosy: +catherine
Added file: http://bugs.python.org/file18294/cmd-noeol-test.fix.patch
___
Python tracker rep
Catherine Devlin fredv8vi...@liquidid.net added the comment:
same patch, stripped of trailing spaces
--
Added file: http://bugs.python.org/file18297/cmd-noeol-test.fix.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8620
45 matches
Mail list logo
