Serhiy Storchaka added the comment:
How the test can be simpler? It is already simple.
You ca use pathlib_resolve_test.py but replace `os.symlink('.', 'testdir/0')`
by `os.symlink(os.path.abspath('testdir'), 'testdir/0')`. Or use following
shell commands:
mkdir testdir
for i in $(seq 100);
Changes by Serhiy Storchaka storch...@gmail.com:
--
title: Path.resolve() ENAMETOOLONG on pathologic symlinks - Path.resolve()
fails on complex symlinks
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19887
Roundup Robot added the comment:
New changeset bd71352e950f by Alexandre Vassalotti in branch 'default':
Issue #6784: Strings from Python 2 can now be unpickled as bytes objects.
http://hg.python.org/cpython/rev/bd71352e950f
--
nosy: +python-dev
___
Alexandre Vassalotti added the comment:
I fixed up the last few review comments and submitted the patch. Thank you for
the help!
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker
Changes by Alexandre Vassalotti alexan...@peadrop.com:
--
nosy: -alexandre.vassalotti
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6673
___
___
Changes by Alexandre Vassalotti alexan...@peadrop.com:
--
assignee: - docs@python
components: +Documentation -Library (Lib)
nosy: +docs@python
stage: - patch review
versions: +Python 3.4 -Python 2.6, Python 2.7, Python 3.1, Python 3.2, Python
3.3
Changes by Alexandre Vassalotti alexan...@peadrop.com:
--
versions: +Python 2.7, Python 3.3
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12290
___
Alexandre Vassalotti added the comment:
Adding a special type is not a bad idea. We have to keep the code for loading
BINSTRING opcodes anyway, so we might as well use it. It could be helpful for
unit-testing our Python 2 compatibility support for pickle.
We should still fix array in 2.7 to
New submission from Matej Cepl:
looking at
http://hg.python.org/cpython/file/543c76769c14/Lib/http/client.py#l847
(logging in HTTPConnection.send method; but this code has been same since like
forever) I see that the HTTP request is NOT pretty printed:
if self.debuglevel 0:
Stefan Krah added the comment:
If performance is the reason for the feature: My impression is that
the goal of the struct module is not necessarily top performance.
E.g. in memoryview the custom unpackers for comparisons are 30-60 times
faster than using the struct module.
--
nosy:
New submission from Serhiy Storchaka:
import pathlib
pathlib.PureWindowsPath('C:/Foo/Bar').relative_to('C:/Foo')
PureWindowsPath('Bar')
pathlib.PureWindowsPath('C:/Foo/Bar').relative_to('C:/foo')
Traceback (most recent call last):
File stdin, line 1, in module
File
anon added the comment:
I like the i.bits_at(pos, width=1) suggestion. Unless slicing is chosen instead
this seems the most future-proof idea.
I think slicing semantically seems wrong but it might be more elegant. It
might also make catching errors harder (in the case where an int is sent to
Serhiy Storchaka added the comment:
Here is a patch which fixes first bug.
--
keywords: +patch
stage: needs patch - patch review
Added file: http://bugs.python.org/file33022/pathlib_relative_to.patch
___
Python tracker rep...@bugs.python.org
Jon Clements added the comment:
Was looking up epoll.modify and noticed in the docs it's listed as
Modify a register file descriptor. - I believe that should be Modify a
registered file descriptor...
--
nosy: +joncle
___
Python tracker
Serhiy Storchaka added the comment:
Actually relative_to() returns invalid path object.
import pathlib
p = pathlib.PureWindowsPath('C:/Foo/Bar/Baz').relative_to('C:')
p
PureWindowsPath('//Foo/Bar/Baz')
str(p)
'Foo\\Bar\\Baz'
p.drive
''
p.root
''
p.parts
('\\', 'Foo', 'Bar', 'Baz')
Mark Dickinson added the comment:
I'd rather see `i.bits_at(pos, width=1)`
Me too. Extracting sign, exponent and significand fields from the binary
representation of a float is at least one thing I'd use this for.
--
___
Python tracker
Changes by STINNER Victor victor.stin...@gmail.com:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19915
___
___
Python-bugs-list
STINNER Victor added the comment:
If you want to avoid the encoding errors, you can also use
PYTHONIOENCODING=:replace or PYTHONIOENCODING=:backslashreplace in Python 3.4
to use the locale encoding, but use an error handler different than strict.
--
Antoine Pitrou added the comment:
How the test can be simpler? It is already simple.
I mean, don't use loops but a simple test setup as in test_resolve_dot.
I am not interested in the pathologic 100 symlinks case, just the case where
the symlink is absolute.
--
Vajrasky Kok added the comment:
And don't forget to use self.dirlink instead of os.symlink(src, dst,
target_is_directory=True).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19887
___
R. David Murray added the comment:
This isn't a bug in Python, it is a bug in your particular *installation* of
Python (or OpenSSL). Your best resource for getting help resolving this would
be to post to the python-list mailing list, or to a support forum for your
particular Linux
Serhiy Storchaka added the comment:
Here is a patch with unrolled loops.
--
Added file: http://bugs.python.org/file33024/pathlib_resolve_3.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19887
Changes by Mark Dickinson dicki...@gmail.com:
--
nosy: +mark.dickinson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5996
___
___
Python-bugs-list
Changes by Mark Dickinson dicki...@gmail.com:
--
versions: +Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5996
___
___
Python-bugs-list
Sworddragon added the comment:
Using an environment variable is not the holy grail for this. On writing a
non-single-user application you can't expect the user to set extra environment
variables.
If compatibility is the only reason in my opinion it would be much better to
include something
anon added the comment:
I didn't really consider floats. bit_length() is only provided to ints for
example.
I think a better solution to pick apart floats would be a function similar to
math.frexp, if it isn't already sufficient. float.bits_at(pos, width) seems a
worse solution because the
Christian Heimes added the comment:
I found the offenders. distlib's wrapper scripts are detected as malicious
programs by some anti virus programs.
pip/_vendor/distlib/t32.exe
Antoine Pitrou added the comment:
Using an environment variable is not the holy grail for this. On
writing a non-single-user application you can't expect the user to set
extra environment variables.
I am not understanding why the user would have to set anything at all.
What is the use case
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +ncoghlan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19846
___
___
Python-bugs-list mailing
Vinay Sajip added the comment:
Hmmm. I use mpress (http://www.matcode.com/mpress.htm) to compress the
executables. These AV results seem to be false positives, given that the files
are green-lit by Symantec, Sophos, McAfee, Kaspersky, F-Prot, AVG, Avast and a
bunch of other reputable AV
Christian Heimes added the comment:
How are you creating these files anyway? I can't find any documentation or
source files in distlib.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19913
Nick Coghlan added the comment:
Antoine's suggestion of being a little more aggressive in choosing utf-8 over
ascii as the OS API encoding sounds reasonable to me.
I think we're getting to a point where a system claiming ASCII as the encoding
to use is almost certainly a misconfiguration
Vinay Sajip added the comment:
It's in the docs at e.g.
http://distlib.readthedocs.org/en/latest/reference.html?highlight=launcher#distlib.scripts.ScriptMaker.__init__
and in the code at e.g.
New submission from Christian Heimes:
On Windows the test_connect_ex_error sometimes fails with EWOULDBLOCK instead
of ECONNREFUSED. Valhallasw sometimes gets the same error with an Ubuntu VM on
Windows. This might be a Windows socket issue.
Guido van Rossum added the comment:
Sorry, here's another version. It keeps the original _fileobj_to_fd function
and wraps it with a method that does the exhaustive search.
--
Added file: http://bugs.python.org/file33025/unregister6.diff
___
Python
Merlijn van Deen added the comment:
My error is slightly different:
$ ./python -i -c from test.test_ssl import *;
support.run_unittest(NetworkedTests)
(...)
==
FAIL: test_connect_ex_error (test.test_ssl.NetworkedTests)
Christian Heimes added the comment:
EAGAIN Resource temporarily unavailable (may be the same value as
EWOULDBLOCK) (POSIX.1)
Can you please check of EAGAIN and EWOULDBLOCK are the same value on your Linux
box? They are the same on my box:
import errno
errno.EAGAIN,
Merlijn van Deen added the comment:
Yes, they are.
errno.EWOULDBLOCK
11
EAGAIN and EWOULDBLOCK are the only two with that errno:
[(k,v) for (k,v) in errno.__dict__.items() if v==11]
[('EWOULDBLOCK', 11), ('EAGAIN', 11)]
111 is just ECONNREFUSED:
[(k,v) for (k,v) in errno.__dict__.items()
Antoine Pitrou added the comment:
Here is a patch.
$ LANG=C ./python -c import os, sys, locale;
print(sys.getfilesystemencoding(), sys.stdin.encoding, os.device_encoding(0),
locale.getpreferredencoding())
- Without the patch:
ascii ANSI_X3.4-1968 ANSI_X3.4-1968 ANSI_X3.4-1968
- With the
Changes by Serhiy Storchaka storch...@gmail.com:
--
nosy: +lemburg, loewis
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19846
___
___
Mark Dickinson added the comment:
Sorry, I wasn't clear. I'm not interested in applying this to floats
themselves; I'm interested in applying it to the bit pattern of a float when
that bit pattern is treated as an integer.
But I have no major objection against it being extended to floats.
Antoine Pitrou added the comment:
The explanation for EAGAIN is in the test just above (test_timeout_connect_ex).
Read it.
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19919
New submission from Serhiy Storchaka:
TarFile.list() fails on some files. In particular on Lib/test/testtar.tar.
import tarfile
tarfile.open('Lib/test/testtar.tar').list()
?rw-r--r-- tarfile/tarfile 7011 2003-01-06 01:19:43 ustar/conttype
?rw-r--r-- tarfile/tarfile 7011
Merlijn van Deen added the comment:
OK. I did some network sniffing; inside the VM, this is what I see:
$ sudo tshark host 82.94.164.164
tshark: Lua: Error during loading:
[string /usr/share/wireshark/init.lua]:45: dofile has been disabled
Running as user root and group root. This could be
New submission from Serhiy Storchaka:
Path.mkdir() can't create a directory with cleared write or list permission
bits for owner when parent directories aren't created. This is because for
parent directories same mode is used as for final directory.
To support this use case we should
Guido van Rossum added the comment:
I think I got the closing sorted out now, and through reordering the dup2()
calls are actually needed.
--
Added file: http://bugs.python.org/file33028/unregister7.diff
___
Python tracker rep...@bugs.python.org
Gregory P. Smith added the comment:
I fixed this in 3.3 and 3.4 several weeks ago.
http://hg.python.org/cpython/rev/9791c5d55f52
http://hg.python.org/cpython/rev/00766fa3366b
--
assignee: - gregory.p.smith
nosy: +gregory.p.smith
resolution: - fixed
stage: - committed/rejected
R. David Murray added the comment:
Wouldn't it be better to throw an error rather than create a (parent) directory
with possibly wrong permission bits? It seems like this would require an
unusual use case in any event.
--
nosy: +r.david.murray
___
Roundup Robot added the comment:
New changeset fbbb591dea09 by Charles-François Natali in branch 'default':
Issue #19857: Make sure that test_imaplib reaps server threads even in face of
http://hg.python.org/cpython/rev/fbbb591dea09
New changeset 6c81df506739 by Charles-François Natali in
Gregory P. Smith added the comment:
I do not think this is ready to live in the standard library.
I've left some comments on your patch review. Consider those for incorporation
into the psutils project.
wait_procs() implementation should live in the psutils module on PyPI with more
Guido van Rossum added the comment:
OK, here's another try. I ran what you suggested for all three tests I added
and they are all clean. I realized that every single call to socketpair() is
followed by two addCleanup calls, so I added a make_socketpair() helper method
that does this.
Serhiy Storchaka added the comment:
The mkdir utility creates parent directories with mode 0o777 ~umask.
$ mkdir -p -m 0 t1/t2/t3
$ ls -l -d t1 t1/t2 t1/t2/t3
drwxrwxr-x 3 serhiy serhiy 4096 Dec 7 22:30 t1/
drwxrwxr-x 3 serhiy serhiy 4096 Dec 7 22:30 t1/t2/
d- 2 serhiy serhiy 4096
R. David Murray added the comment:
OK, emulating mkdir -p seems like the sensible thing to do. That's the least
likely to be surprising.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19921
New submission from Christian Heimes:
On HP-UX mbstate_t for mbrtowc() is only available when
_INCLUDE__STDC_A1_SOURCE is defined:
http://buildbot.python.org/all/builders/PA-RISC%20HP-UX%2011iv2%20%5BSB%5D%203.x/builds/2543/steps/compile/logs/stdio
cc -Ae -c -O -O -I. -IInclude
Serhiy Storchaka added the comment:
Updated patch emulates the mkdir utility.
--
Added file: http://bugs.python.org/file33030/pathlib_mkdir_mode_2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19921
New submission from Antoine Pitrou:
This rather weird error occurred on a buildbot:
http://buildbot.python.org/all/builders/x86%20Ubuntu%20Shared%203.x/builds/9297
[346/387] test_multiprocessing_forkserver
Process Process-497:
Traceback (most recent call last):
File
Serhiy Storchaka added the comment:
Updated patch addresses Antoine's comments.
--
Added file: http://bugs.python.org/file33031/pathlib_mkdir_mode_3.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19921
Terry J. Reedy added the comment:
I am not an expert in web stuff, but off the top of my head, it seems that
requests should be handled the same way as responses.
--
nosy: +orsenthil, r.david.murray, terry.reedy
stage: - test needed
type: - enhancement
versions: +Python 3.5
Changes by Christian Heimes li...@cheimes.de:
--
keywords: +patch
Added file: http://bugs.python.org/file33032/issue19922.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19922
___
Antoine Pitrou added the comment:
Patch looks good to me. Do you think the documentation should be clarified a
bit?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19921
___
Serhiy Storchaka added the comment:
Yes, please clarify the documentation.
Perhaps we should add new argument parents_mode?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19921
___
Antoine Pitrou added the comment:
A new argument sounds overkill to me.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19921
___
___
Changes by Martin Panter vadmium...@gmail.com:
--
nosy: +vadmium
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13736
___
___
Python-bugs-list
Christian Heimes added the comment:
http://www.mail-archive.com/autoconf-patches@gnu.org/msg04244.html
http://modman.unixdev.net/?sektion=3page=wcsftimemanpath=HP-UX-11.11
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19922
Roundup Robot added the comment:
New changeset d8cfc7106f41 by Christian Heimes in branch 'default':
Issue #19922: define _INCLUDE__STDC_A1_SOURCE in HP-UX to include mbstate_t
http://hg.python.org/cpython/rev/d8cfc7106f41
--
nosy: +python-dev
___
Charles-François Natali added the comment:
LGTM!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19876
___
___
Python-bugs-list mailing list
Changes by Charles-François Natali cf.nat...@gmail.com:
--
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19857
STINNER Victor added the comment:
There was a previous try to use a file encoding different than the locale
encoding and it introduces too many issues:
https://mail.python.org/pipermail/python-dev/2010-October/104509.html
Inconsistencies if locale and filesystem encodings are different
Python
Antoine Pitrou added the comment:
Python uses the fact that the filesystem encoding is the locale
encoding in various places.
The patch doesn't change that.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19846
Roundup Robot added the comment:
New changeset 39e7995f9ad1 by Guido van Rossum in branch 'default':
Silently ignore unregistering closed files. Fixes issue 19876. With docs and
slight test refactor.
http://hg.python.org/cpython/rev/39e7995f9ad1
--
nosy: +python-dev
Guido van Rossum added the comment:
Is this worthy of a Misc/NEWS entry?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19876
___
___
Roundup Robot added the comment:
New changeset f334dd2471e7 by Guido van Rossum in branch 'default':
News item for issue 19876.
http://hg.python.org/cpython/rev/f334dd2471e7
--
___
Python tracker rep...@bugs.python.org
Guido van Rossum added the comment:
Done.
--
assignee: neologix - gvanrossum
resolution: - fixed
stage: - committed/rejected
status: open - closed
type: - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19876
New submission from Christian Heimes:
The test fails when Python is compiled --without-threads. Two 3rd party modules
import threading unconditionally:
pip/_vendor/distlib/util.py
import threading
pip/_vendor/requests/packages/urllib3/_collections.py
from threading import RLock
Changes by Christian Heimes li...@cheimes.de:
--
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19922
___
Gregory P. Smith added the comment:
zipimport.c makes no attempt to support zip files larger than 2GiB or zip64
files.
--
nosy: +gregory.p.smith
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19883
Nick Coghlan added the comment:
Note that the *only* change Antoine's patch makes is that:
- *if* the locale encoding is ASCII (or an alias for ASCII)
- *then* Python sets the filesystem encoding to UTF-8 instead
If the locale encoding is anything *other* than ASCII, then that will still be
Changes by Nick Coghlan ncogh...@gmail.com:
--
resolution: - duplicate
status: open - closed
superseder: - test_venv: test_with_pip() failed on AMD64 Fedora without
threads 3.x buildbot: urllib3 dependency requires the threading module
___
Python
Nick Coghlan added the comment:
Issue 19924 suggests we may need a new distlib as well
--
nosy: +christian.heimes, vinay.sajip
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19766
___
Roundup Robot added the comment:
New changeset 94593dcb195a by Eric Snow in branch 'default':
Issue #19758: silence PendingDeprecationWarnings in test_importlib.
http://hg.python.org/cpython/rev/94593dcb195a
--
___
Python tracker
Roundup Robot added the comment:
New changeset 44948f5bdc12 by Gregory P. Smith in branch '3.3':
Fixes issue #19506: Use a memoryview to avoid a data copy when piping data
http://hg.python.org/cpython/rev/44948f5bdc12
New changeset 5379bba2fb21 by Gregory P. Smith in branch 'default':
Fixes
Changes by Gregory P. Smith g...@krypto.org:
--
resolution: - fixed
stage: - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19506
___
Gregory P. Smith added the comment:
fwiw, this patch made sense. using test_sub.py on my dual-core amd64 box with
an opt build is saw the times drop from ~0.90-0.97 to ~0.75-0.8. more speedup
than i was really anticipating for this use case. it probably depends upon
what the value of
Eric Snow added the comment:
This is actually a problem with importlib.reload() (which imp.reload() simply
wraps). The attached patch provides a test that reproduces the error. I'll
work on a fix ASAP.
Interestingly, the kind of failure depends on frozen vs. source importlib:
Eric Snow added the comment:
Actually, they're both getting the same error:
AttributeError: 'NoneType' object has no attribute 'name'
I forgot to clear the submodule from sys.modules first.
--
___
Python tracker rep...@bugs.python.org
Donald Stufft added the comment:
Requests was released and pip updated it, I can release a new pip but it
appears that perhaps distlib needs fixed before the without threads case is
taken care of?
--
___
Python tracker rep...@bugs.python.org
New submission from Vajrasky Kok:
So we may have buildbot with root account after all.
https://mail.python.org/pipermail/python-dev/2013-December/130708.html
So here is the unit test for spwd module that requires root account.
--
components: Tests
files: unittest_for_spwd.patch
Changes by Vajrasky Kok sky@speaklikeaking.com:
Removed file: http://bugs.python.org/file33034/unittest_for_spwd.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19925
___
Changes by Vajrasky Kok sky@speaklikeaking.com:
Added file: http://bugs.python.org/file33035/unittest_for_spwd.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19925
___
New submission from Vajrasky Kok:
There are superfluous lines and unused import in
Lib/test/test_abstract_numbers.py.
Attached the patch to remove superfluous lines and unused import.
--
components: Tests
files: refactor_test_abstract_number.patch
keywords: patch
messages: 205514
New submission from Eric Snow:
ModuleSpec.__eq__() does a comparision of its various attributes, one of them
being the loader. However, the __eq__() of the path-based loaders is just the
stock one that compares object identity. So most
--
messages: 205515
nosy: brett.cannon,
Eric Snow added the comment:
(my browser farted the half finished report into existence :P )
The __eq__() implementation of the path-based loaders in importlib is just the
stock one that compares object identity. So two that are effectively the same
compare unequal. This has a material
Changes by Eric Snow ericsnowcurren...@gmail.com:
--
Removed message: http://bugs.python.org/msg205515
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19927
___
Eric Snow added the comment:
The problem was that importlib.reload() was not passing the parent's __path__
to importlib._bootstrap._find_spec(). This was a consequence of us restoring
the pre-3.3 reload semantics.
Patch attached. (Note to self: add Misc/NEWS entry)
--
stage: needs
Changes by Eric Snow ericsnowcurren...@gmail.com:
Removed file: http://bugs.python.org/file33033/issue19851-test.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19851
___
Eric Snow added the comment:
Here's a patch.
--
keywords: +patch
stage: test needed - patch review
Added file: http://bugs.python.org/file33038/issue19927-loader-eq.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19927
Nick Coghlan added the comment:
Patch looks reasonable to me.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19851
___
___
Python-bugs-list
Eric Snow added the comment:
Good catch. My preference would be for (c) simply add a setter for the
property that sets the underlying private variable to True or False.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18864
Nick Coghlan added the comment:
There can be some interesting backwards compatibility consequences when adding
an __eq__ implementation to a class that was previously using the default ID
based __eq__:
- it becomes unhashable (unless you also add a suitable __hash__ definition)
- subclasses
Roundup Robot added the comment:
New changeset 3283fb24106d by Zachary Ware in branch '3.3':
Issue 19572: More silently skipped tests explicitly skipped.
http://hg.python.org/cpython/rev/3283fb24106d
New changeset 03afd2d7d395 by Zachary Ware in branch 'default':
Issue 19572: More silently
1 - 100 of 105 matches
Mail list logo