[issue12361] Memory Leak in File Logging
Changes by Vinay Sajip vinay_sa...@yahoo.co.uk: -- assignee: - vinay.sajip ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12361 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12278] Core mentorship mention in the devguide
Changes by Nick Coghlan ncogh...@gmail.com: -- assignee: - ncoghlan ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12278 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12291] file written using marshal in 3.2 can be read by 2.7, but not 3.2 or 3.3
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment:
The problem with calling fileno() and fdopen() is that you bypass the buffering
information held in BufferedIOReader. The first call works, but the FILE *
pointer is now positioned at 4K, rather than just past the end of the object
just read. The next call fails.
I verified that calling f.tell() after marshal.load(f) returns 4096, rather
than just the size of the object read by the load().
Just to be clear, here's what I did in marshal_load:
int is_file = 0;
int fd;
data = PyObject_CallMethod(f, fileno, );
if (data == NULL)
PyErr_Clear();
else {
fd = PyLong_AsLong(data);
Py_DECREF(data);
is_file = 1;
}
if (is_file) {
rf.readable = NULL;
rf.fp = fdopen(fd, rb);
}
else {
/* what I was doing before to set up rf */
}
/* and on to the read_object call */
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12291
___
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12362] General Windows stdout redirection not working
New submission from CrouZ alfred.theo...@gmail.com: Steps to repeat: * Create the script foo.py consisting of the line: print(foo) * Run: foo.py bar Behavior: 2.7.2: The file bar is created but empty. Prints the following message on exit: close failed in file object destructor: sys.excepthook is missing lost sys.stderr 3.1.4: The file bar is created but empty. No printouts indicating that something went wrong. Expected behavior: All: The file bar has the content foo. Additional information: sys.stdout, and probably also sys.stderr, do not appear to be set up properly. 2.7.2: stdout is set to a file but performing any operations on it throws the exception: IOError: [Errno 9] Bad file descriptor 3.1.4: stdout is set to None Workaround: All: python foo.py bar Used software: Windows 7 Ultimate SP1 x64 (6.1.7601) Python 2.7.2 and 3.1.4 installed with the msi-installer including Registering Extensions. -- components: Windows messages: 138626 nosy: CrouZ priority: normal severity: normal status: open title: General Windows stdout redirection not working type: crash versions: Python 2.7, Python 3.1 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12362 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12362] General Windows stdout redirection not working
Changes by CrouZ alfred.theo...@gmail.com: -- type: crash - behavior ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12362 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12361] Memory Leak in File Logging
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment: Thanks for the report, but more information from the failing system may be needed to find the problem. Although 2.6 does not contain full unit test coverage, the default branch does - and I tested there with resource leak checking turned on, with no leaks being detected. An eyeball inspection fails to show where a leak could be occurring: WatchedFileHandler just makes stat() calls and, if appropriate, calls flush/close on the existing stream and open on the new one. Test output: vinay@eta-jaunty:~/projects/python/default$ ./python Lib/test/regrtest.py -R :: test_logging [1/1] test_logging beginning 9 repetitions 123456789 . 1 test OK. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12361 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12358] validate server certificate when uploading packages to PyPI
Stefan Krah stefan-use...@bytereef.org added the comment: I agree with Éric: This is a duplicate. -- nosy: +skrah resolution: - duplicate status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12358 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12278] Core mentorship mention in the devguide
Roundup Robot devnull@devnull added the comment: New changeset 144b12d7bb28 by Nick Coghlan in branch 'default': ACKS update for devguide patch (closes #12278) http://hg.python.org/cpython/rev/144b12d7bb28 -- nosy: +python-dev resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12278 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12362] General Windows stdout redirection not working
R. David Murray rdmur...@bitdance.com added the comment: The file association for .py is pythonw, which does exactly as you say, intentionally, so as to avoid problems with windows when running a GUI application. My understanding is that this caters to the most common use case on Windows: double clicking a .py file launching a Python GUI application. When running a python script in the console, use 'python', as in your example (or manually change your file association). -- nosy: +r.david.murray resolution: - invalid stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12362 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12346] Python source code build fails with old mercurial
Benjamin Peterson benja...@python.org added the comment: So, it seems the problem is not actually that the build depends on mercurial, it's that it fails with ancient mercurials. -- title: Python source code build (release) depends on mercurial - Python source code build fails with old mercurial ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12346 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12362] General Windows stdout redirection not working
Changes by R. David Murray rdmur...@bitdance.com: -- dependencies: -Error in sys.excepthook on windows when redirecting output of the script superseder: - Error in sys.excepthook on windows when redirecting output of the script ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12362 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10454] Clarify compileall command-line options
R. David Murray rdmur...@bitdance.com added the comment: Looks fine except for your changes to the parenthesized defaults. Those should be '0' and 'False' for 2.7 and 3.x, respectively, since that's what they areally are. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10454 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12362] General Windows stdout redirection not working
Amaury Forgeot d'Arc amaur...@gmail.com added the comment: IMO the cause is actually the same as the one for issue9390, i.e. a bug in the Windows console. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12362 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12291] file written using marshal in 3.2 can be read by 2.7, but not 3.2 or 3.3
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment:
This seems a bit hacky, and I'm not sure how reliable it is. I added this after
the read_object call:
if (is_file) {
PyObject * newpos;
int cp, np;
cp = ftell(rf.fp);
newpos = PyObject_CallMethod(f, seek, ii, cp, SEEK_SET);
assert(newpos != NULL);
np = PyLong_AsLong(newpos);
Py_DECREF(newpos);
assert(cp == np);
}
When I run the code lots of times, I sometimes get assertion failures at the
assert(newpos != NULL)
line. It's not an EOF condition thing, necessarily: I do get expected behaviour
at least sometimes when seeking to the end of file.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12291
___
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11637] Add cwd to sys.path for hooks
Changes by Vinay Sajip vinay_sa...@yahoo.co.uk: -- nosy: +vinay.sajip ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11637 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11795] Better core dev guidelines for committing submitted patches
R. David Murray rdmur...@bitdance.com added the comment: Note that the older tradition was to *not* mention the contributor in NEWS (NEWS was just technical notes), but to mention them in the checkin message (and What's New, for things that make the What's New cut). However, since we can't edit checkin messages post-commit like we could with SVN, I suppose this new convention (that some people have been following since well before the switchover...) is more practical. -- nosy: +r.david.murray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11795 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12346] Python source code build fails with old mercurial
Éric Araujo mer...@netwok.org added the comment: Yes, it was committed to default. I think we can assume they used a new enough hg to check it out...and if they didn't, that *is* a bug in their setup they should fix. Agreed. -- nosy: +eric.araujo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12346 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11690] Devguide: Add communication FAQ
Nick Coghlan ncogh...@gmail.com added the comment: Comms FAQ: http://hg.python.org/devguide/rev/f1ebfb53437f Devguide note: http://hg.python.org/devguide/rev/5ab42baba771 -- resolution: - fixed stage: - committed/rejected ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11690 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11795] Better core dev guidelines for committing submitted patches
Changes by Nick Coghlan ncogh...@gmail.com: -- resolution: - fixed stage: needs patch - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11795 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12362] General Windows stdout redirection not working
R. David Murray rdmur...@bitdance.com added the comment: Ah, I should have known better than to rely on a memory instead of checking, since I don't use Windows much. My apologies. -- dependencies: +Error in sys.excepthook on windows when redirecting output of the script resolution: invalid - duplicate ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12362 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12291] file written using marshal in 3.2 can be read by 2.7, but not 3.2 or 3.3
Benjamin Peterson benja...@python.org added the comment:
2011/6/19 Vinay Sajip rep...@bugs.python.org:
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment:
This seems a bit hacky, and I'm not sure how reliable it is. I added this
after the read_object call:
if (is_file) {
PyObject * newpos;
int cp, np;
cp = ftell(rf.fp);
newpos = PyObject_CallMethod(f, seek, ii, cp, SEEK_SET);
Why not 0?
assert(newpos != NULL);
np = PyLong_AsLong(newpos);
Py_DECREF(newpos);
assert(cp == np);
}
When I run the code lots of times, I sometimes get assertion failures at the
assert(newpos != NULL)
That's because the call is failing. Why?
line. It's not an EOF condition thing, necessarily: I do get expected
behaviour at least sometimes when seeking to the end of file.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12291
___
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12291] file written using marshal in 3.2 can be read by 2.7, but not 3.2 or 3.3
Benjamin Peterson benja...@python.org added the comment: 2011/6/19 Vinay Sajip rep...@bugs.python.org: Vinay Sajip vinay_sa...@yahoo.co.uk added the comment: The problem with calling fileno() and fdopen() is that you bypass the buffering information held in BufferedIOReader. The first call works, but the FILE * pointer is now positioned at 4K, rather than just past the end of the object just read. The next call fails. But presumably once you have the fd, you can llseek(). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12291 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12346] Python source code build fails with old mercurial
R. David Murray rdmur...@bitdance.com added the comment: Only if Ralf's patch is applied to all branches. Otherwise the make step reports abort: repository . not found!, which most users will ignore but a few will report here. It looks from Ralf's quoted changeset like it was only applied to default, but frankly I'm not comfortable enough with mercurial yet to be able to tell. If Ralf's patch is applied, then the lack of a repository would cause the old hg to not get called. If someone *has* a repository in the build dir, I think we can assume they used a new enough hg to check it out...and if they didn't, that *is* a bug in their setup they should fix. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12346 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11690] Devguide: Add communication FAQ
Changes by Nick Coghlan ncogh...@gmail.com: -- status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11690 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11637] Add cwd to sys.path for hooks
Changes by Vinay Sajip vinay_sa...@yahoo.co.uk: -- hgrepos: +32 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11637 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11795] Better core dev guidelines for committing submitted patches
Nick Coghlan ncogh...@gmail.com added the comment: Enhanced committer guidelines: http://hg.python.org/devguide/rev/774fb024b152 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11795 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12362] General Windows stdout redirection not working
Amaury Forgeot d'Arc amaur...@gmail.com added the comment: The file association for .py is pythonw Really? http://docs.python.org/using/windows.html#executing-scripts -- nosy: +amaury.forgeotdarc ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12362 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12255] A few changes to .*ignore
Senthil Kumaran sent...@uthcode.com added the comment: +libpython*.so* is fine, but please don't remove the rej and orig. By convention and practice, those files are meant to be in .ignore files to prevent accidental checkins. Usually when the conflict occurs you are notified immediately and you fix them subsequently. Being notified of pending merge by seeing the.rej files in the hg status seems remote to me. -- nosy: +orsenthil ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12255] A few changes to .*ignore
R. David Murray rdmur...@bitdance.com added the comment: As I've said before, I would vote to not have .rej and .orig in .hgignore. You can always add them to your personal .hgignore, but I know of no way to tell mercurial I *don't* want to ignore these files that are in the repo .hgignore. If there is such a way I'll be happy with that. -- nosy: +r.david.murray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12255] A few changes to .*ignore
Éric Araujo mer...@netwok.org added the comment: By convention and practice, those files are meant to be in .ignore files to prevent accidental checkins. Maybe with other tools. Mercurial will tell you what new files are to be added in the next commit once to four times, depending on your setup, so I don’t understand how people accidentally commit .rej files (or .svn directories or Mac OS X crap). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12363] test_signal.test_without_siginterrupt() sporadic failures on FreeBSD 6.4
New submission from STINNER Victor victor.stin...@haypocalc.com: == FAIL: test_siginterrupt_on (test.test_signal.SiginterruptTest) -- Traceback (most recent call last): File /usr/home/db3l/buildarea/3.x.bolen-freebsd/build/Lib/test/test_signal.py, line 396, in test_siginterrupt_on self.assertTrue(i) AssertionError: False is not true http://www.python.org/dev/buildbot/all/builders/x86%20FreeBSD%206.4%203.x/builds/1586/steps/test/logs/stdio http://www.python.org/dev/buildbot/all/builders/x86%20FreeBSD%206.4%203.x/builds/1587/steps/test/logs/stdio -- messages: 138647 nosy: haypo, neologix priority: normal severity: normal status: open title: test_signal.test_without_siginterrupt() sporadic failures on FreeBSD 6.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12363 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12364] Timeout (1 hour) in test_concurrent_futures.tearDown() on sparc solaris10 gcc 3.x
New submission from STINNER Victor victor.stin...@haypocalc.com: [271/356/1] test_concurrent_futures Traceback (most recent call last): File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/multiprocessing/queues.py, line 268, in _feed send(obj) File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/multiprocessing/connection.py, line 229, in send self._send_bytes(memoryview(buf)) File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/multiprocessing/connection.py, line 423, in _send_bytes self._send(struct.pack(=i, len(buf))) File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/multiprocessing/connection.py, line 392, in _send n = write(self._handle, buf) OSError: [Errno 32] Broken pipe Timeout (1:00:00)! Thread 0x0954: File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/threading.py, line 237 in wait File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/multiprocessing/queues.py, line 252 in _feed File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/threading.py, line 690 in run File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/threading.py, line 737 in _bootstrap_inner File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/threading.py, line 710 in _bootstrap Thread 0x0953: File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/multiprocessing/forking.py, line 146 in poll File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/multiprocessing/forking.py, line 166 in wait File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/multiprocessing/process.py, line 150 in join File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/concurrent/futures/process.py, line 208 in shutdown_worker File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/concurrent/futures/process.py, line 264 in _queue_management_worker File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/threading.py, line 690 in run File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/threading.py, line 737 in _bootstrap_inner File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/threading.py, line 710 in _bootstrap Thread 0x0001: File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/threading.py, line 237 in wait File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/threading.py, line 851 in join File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/concurrent/futures/process.py, line 395 in shutdown File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/test/test_concurrent_futures.py, line 67 in tearDown File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/unittest/case.py, line 407 in _executeTestPart File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/unittest/case.py, line 463 in run File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/unittest/case.py, line 514 in __call__ File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/unittest/suite.py, line 105 in run File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/unittest/suite.py, line 67 in __call__ File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/unittest/suite.py, line 105 in run File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/unittest/suite.py, line 67 in __call__ File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/test/support.py, line 1166 in run File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/test/support.py, line 1254 in _run_suite File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/test/support.py, line 1280 in run_unittest File /home2/buildbot/slave/3.x.loewis-sun/build/Lib/test/test_concurrent_futures.py, line 628 in test_main File ./Lib/test/regrtest.py, line 1043 in runtest_inner File ./Lib/test/regrtest.py, line 841 in runtest File ./Lib/test/regrtest.py, line 668 in main File ./Lib/test/regrtest.py, line 1618 in module *** Error code 1 make: Fatal error: Command failed for target `buildbottest' program finished with exit code 1 See commit e6e7e42efdc2 of the issue #12310. -- components: Tests messages: 138648 nosy: haypo, neologix, pitrou priority: normal severity: normal status: open title: Timeout (1 hour) in test_concurrent_futures.tearDown() on sparc solaris10 gcc 3.x versions: Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12364 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12365] URLopener should support context manager protocol
New submission from Jeff McNeil j...@jmcneil.net: Per discussion within Issue10050, URLopener ought to support the context manager protocol. That allows more idiomatic usage and doesn't require calls to contextlib.closing for use with the 'with' statement. If agreed, I'll create a patch. -- components: Library (Lib) messages: 138649 nosy: mcjeff priority: normal severity: normal status: open title: URLopener should support context manager protocol versions: Python 3.1 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12365 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12365] URLopener should support context manager protocol
Changes by Jeff McNeil j...@jmcneil.net: -- type: - feature request ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12365 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12365] URLopener should support context manager protocol
Éric Araujo mer...@netwok.org added the comment: +1. -- nosy: +eric.araujo stage: - needs patch versions: +Python 3.3 -Python 3.1 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12365 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11637] Add cwd to sys.path for hooks
Éric Araujo mer...@netwok.org added the comment: Thanks, I made slight editions, now I’m improving the docs and will commit shortly. -- assignee: tarek - eric.araujo stage: needs patch - commit review versions: +Python 3.3 -3rd party ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11637 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12363] test_signal.test_without_siginterrupt() sporadic failures on FreeBSD 6.4
STINNER Victor victor.stin...@haypocalc.com added the comment: Seen also on OpenSolaris: test test_signal failed -- Traceback (most recent call last): File /export/home/buildbot/64bits/3.x.cea-indiana-amd64/build/Lib/test/test_signal.py, line 399, in test_siginterrupt_on self.assertTrue(i) AssertionError: False is not true http://www.python.org/dev/buildbot/all/builders/AMD64%20OpenIndiana%203.x/builds/1443/steps/test/logs/stdio -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12363 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12321] documentation of ElementTree.find
patrick vrijlandt patrick.vrijla...@gmail.com added the comment: [...] Same as getroot().find(match). [...] - [...] For a relative path, this is equivalent to getroot().find(match). Additionally, this form accepts an absolute path. [...] This is easy, but might not be a very good solution. Random thoughts/Points to consider: It does help the novice in debugging his expressions. A peculiarity of this implementation is documented. As others have stated, the whole elementpath documentation within the python docs is incomplete. Should we document the exception but not the rule? It makes no real sense to do a an absolute search from an element. However, it's not ambiguous. lxml does accept the absolute path search from an element. Actually, I think it's up to Fredrik to decide. 2011/6/18 Terry J. Reedy rep...@bugs.python.org Terry J. Reedy tjre...@udel.edu added the comment: Are you requesting that the doc be changed or the code? From the title, I would infer the doc (which is much easier ;-). If so, can you suggest an actual revised text? -- nosy: +terry.reedy stage: - needs patch versions: +Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12321 ___ -- Added file: http://bugs.python.org/file22406/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12321 ___span class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; [...] Same as getroot().find(match). [...] -gt;/spandivfont class=Apple-style-span face=arial, sans-serifspan class=Apple-style-span style=border-collapse: collapse;[...] For a relative path, this is equivalent to /span/fontspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; getroot().find(match). Additionally, this form accepts an absolute path. [...]/span/div divspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; br/span/divdivspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; This is easy, but might not be a very good solution. /span/div divspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; br/span/divdivspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; Random thoughts//spanspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; Points to consider:/span/div divspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; It does help the novice in debugging his expressions. /span/divdivspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; A peculiarity of this implementation is documented./span/div divspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; As others have stated, the whole elementpath documentation within the python docs is incomplete. Should we document the exception but not the rule?/span/div divspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; It makes no real sense to do a an absolute search from an element. However, it#39;s not ambiguous./span/div divspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; lxml does accept the absolute path search from an element./span/divdivfont class=Apple-style-span face=arial, sans-serifspan class=Apple-style-span style=border-collapse: collapse;br /span/font/divdivdivspan class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; Actually, I think it#39;s up to Fredrik to decide. /span/div/divdiv span class=Apple-style-span style=border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; br/span/divdivfont class=Apple-style-span face=arial, sans-serifspan class=Apple-style-span style=border-collapse: collapse;br /span/fontbrdiv class=gmail_quote2011/6/18 Terry J. Reedy span dir=ltrlt;a href=mailto:rep...@bugs.python.org;rep...@bugs.python.org/agt;/spanbrblockquote class=gmail_quote style=margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex; br Terry J. Reedy lt;a href=mailto:tjre...@udel.edu;tjre...@udel.edu/agt; added the comment:br br Are you requesting that the doc be changed or the code?br gt;From the title, I would infer the doc (which is much easier ;-).br If so, can you suggest an actual revised text?br br --br nosy: +terry.reedybr stage:  -gt; needs patchbr versions: +Python 3.3br divdiv/divdiv class=h5br
[issue11610] Improved support for abstract base classes with descriptors
Darren Dale dsdal...@gmail.com added the comment: Here is attempt #4. This patch extends the property, classmethod and staticmethod builtins with an __isabstractmethod__ descriptor. Docs and tests are updated as well. make test runs without failures. This is my first real attempt with the C-API, and I think I am finally over the learning curve, but comments would be greatly appreciated. -- Added file: http://bugs.python.org/file22407/abc_descriptor.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11610 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11610] Improved support for abstract base classes with descriptors
Changes by Darren Dale dsdal...@gmail.com: Removed file: http://bugs.python.org/file21307/issue11610.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11610 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11610] Improved support for abstract base classes with descriptors
Changes by Darren Dale dsdal...@gmail.com: Removed file: http://bugs.python.org/file21375/issue11610_v2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11610 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11610] Improved support for abstract base classes with descriptors
Changes by Darren Dale dsdal...@gmail.com: Removed file: http://bugs.python.org/file22323/abc_descriptors.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11610 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11637] Add cwd to sys.path for hooks
Roundup Robot devnull@devnull added the comment: New changeset b732b02bd0ba by Éric Araujo in branch 'default': packaging: Add the project directory to sys.path to support local setup hooks. http://hg.python.org/cpython/rev/b732b02bd0ba -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11637 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11637] Add cwd to sys.path for hooks
Éric Araujo mer...@netwok.org added the comment: Thanks! -- resolution: - fixed stage: commit review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11637 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2983] Ttk support for Tkinter
Éric Araujo mer...@netwok.org added the comment: The 2.7 docs are missing a versionadded directive. -- nosy: +eric.araujo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2983 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11637] Add cwd to sys.path for hooks
Changes by Martin v. Löwis mar...@v.loewis.de: -- keywords: +patch Added file: http://bugs.python.org/file22408/8b9da1557ad2.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11637 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11637] Add cwd to sys.path for hooks
Changes by Éric Araujo mer...@netwok.org: -- hgrepos: +33 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11637 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12366] packaging.pypi.dist should abstract download errors.
New submission from Michael Mulich michael.mul...@gmail.com: packaging.pypi.dist should abstract download errors, especially those from external sources. Download errors are currently reported from urllib. We should probably be using packaging.errors.PackagingPyPIError in this situation. Other suggestions? Example case: sake version 0.0.0 has a external download URL that throws a 404 (see also http://pypi.python.org/simple/sake/). When attempting to download this release, we receive a ValueError from urllib, which is not very detailed and could mean a number of things. Traceback (most recent call last): ... [dev project] ... File .../cpython/Lib/packaging/pypi/dist.py, line 167, in download .download(path=temp_path) File .../cpython/Lib/packaging/pypi/dist.py, line 302, in download path + / + archive_name) File .../cpython/Lib/urllib/request.py, line 150, in urlretrieve return _urlopener.retrieve(url, filename, reporthook, data) File .../cpython/Lib/urllib/request.py, line 1600, in retrieve block = fp.read(bs) ValueError: read of closed file -- assignee: tarek components: Distutils2 messages: 138658 nosy: alexis, eric.araujo, michael.mulich, tarek priority: normal severity: normal status: open title: packaging.pypi.dist should abstract download errors. type: behavior versions: Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12366 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12367] select.error has no errno attribute
New submission from STINNER Victor victor.stin...@haypocalc.com: It would be nice to have a errno attribute for select.error. I don't know if select.errno should inherit from OSError, WindowsError or nothing. See also the PEP 3151: http://www.python.org/dev/peps/pep-3151/#common-errnos-with-select-error -- messages: 138659 nosy: haypo, pitrou priority: normal severity: normal status: open title: select.error has no errno attribute versions: Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12367 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12366] packaging.pypi.dist should abstract download errors.
Éric Araujo mer...@netwok.org added the comment: I agree that we can wrap exceptions to provide more useful info where needed. It’s not necessary, however, to always catch exceptions and raise Packaging*Errors instead: distutils and packaging are documented to raise ValueError and other standard exceptions in some cases. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12366 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12291] file written using marshal in 3.2 can be read by 2.7, but not 3.2 or 3.3
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment: Benjamin Peterson benja...@python.org added the comment: assert(newpos != NULL) That's because the call is failing. Why? It's seemingly because the Python code did a seek (in Python) which was not communicated to the FILE object; after reading all objects from the file, a seek was done to the beginning to start reading again. On the previous call, the FILE would have been at EOF (at a C level). So it seems as if we have to call ftell on the Python object at the beginning of the read, and do an fseek to sync the positions. It's doable, of course, so it's the next thing I'll try, but what I'm worried about is the thing turning into a bit of a rabbit-hole. I haven't looked at the io implementation, but for this sort of synchronisation, it would be better if when based on a FILE object, the io implementation delegated all its operations to the FILE object - but it doesn't look like that's the case, and I'm uncomfortable that there may be some undesirable consequences of that. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12291 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12261] urllib.parse docs still refer to urlparse
Roundup Robot devnull@devnull added the comment: New changeset 4211ace1ff5d by Senthil Kumaran in branch '3.1': Fix closes issue12261 - Minor documention changes in the urllib.parse.rst http://hg.python.org/cpython/rev/4211ace1ff5d New changeset 18f3239b3d48 by Senthil Kumaran in branch '3.2': merge from 3.1 for issue issue12261. http://hg.python.org/cpython/rev/18f3239b3d48 -- nosy: +python-dev resolution: - fixed stage: commit review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12261 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12368] packaging.pypi.simple.Crawler assumes external download links are ok to follow
New submission from Michael Mulich michael.mul...@gmail.com: The packaging.pypi.simple.Crawler blindly follows external download URLs. The crawler should honor a list of allowed hosts (see also the hosts parameter) before attempting to download from an external source. Éric Araujo has also pointed out that established tools like easy_install and pip provide ways of allowing/restricting by host. -- assignee: tarek components: Distutils2 messages: 138663 nosy: alexis, eric.araujo, michael.mulich, tarek priority: normal severity: normal status: open title: packaging.pypi.simple.Crawler assumes external download links are ok to follow type: behavior versions: Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12368 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12363] test_signal.test_without_siginterrupt() sporadic failures on FreeBSD 6.4
STINNER Victor victor.stin...@haypocalc.com added the comment: I tried to patch the test to use a semaphore, but my patch was not reliable (don't remove completly the race condition). Here is a patch using a subprocess to: - have only one thread - have a timeout on the blocking read (select cannot be used in the test, select always fail with EINTR, the kernel doesn't restart it) - not touch signal handling of the parent process It is also based on time: it uses alarm() to raise a signal in one second, and use an hardcoded timeout of 3 seconds. But it doesn't need tricky synchronization between two processes. -- keywords: +patch nosy: +pitrou Added file: http://bugs.python.org/file22409/test_siginterrupt.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12363 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12291] file written using marshal in 3.2 can be read by 2.7, but not 3.2 or 3.3
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment: @Benjamin: I missed commenting on your Why not 0?, but here's the reasoning: one can't assume that the file only contains one object to be read, at the beginning of the file. It may be that some data is being written to file using marshal.dump, interspersed with other data, and then the stream is being read in at a later time, with marshal.load called to load a previously saved object. In that scenario, why would 0 be always the correct offset value to pass to fseek? I am synchronising the Python object with the FILE *, but not the other way around - in the failing case I mentioned, external Python code has positioned the io object to zero, but that of course will be overwritten to point the io object back to where the FILE object is positioned. Perhaps I just need to go the other way ... I'm not sure how reliable a solution will be which tries to work around there apparently being two buffering implementations - at the io level and at the FILE level, and trying to keep them synced in a seemingly ad hoc fashion. It's perfectly possible that I don't know what I'm talking about, so I'd welcome an improved patch using the ideas you've suggested. It's been a very long time since I wrote C stdio code regularly ;-) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12291 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12368] packaging.pypi.simple.Crawler assumes external download links are ok to follow
Éric Araujo mer...@netwok.org added the comment: Extract from IRC: pumazi hmm... I'm thinking Crawler's follow_externals flag isn't working as expected [...] pumazi I'm not sure, my assumption of [its] function could be off [...] merwok “hosts is a list of hosts allowed to be processed if follow_externals is true (default behavior is to follow all hosts), follow_externals enables or disables following external links (default is false, meaning disabled).” pumazi Well, I was assuming it would disable external downloads merwok I think “external links” are external links to be scraped, not download links merwok But I see your misunderstanding pumazi I see, but wouldn't we want the same restrictions on download links? [...] merwok IIUC, follow_externals can be disabled because it’s guesswork merwok The info obtained from XML-RPC or the simple interface is not guesswork merwok So I think you could want to disable guessing from external links, but I don’t see why you should care about the origin of the download pumazi trust issues I suppose merwok But the same person can upload a malicious file to PyPI as well as on their site merwok Without reading the code, I think this is the rationale. OTOH, if easy_install and pip can restrict downloads and your user expectations show that it can be needed to restrict downloads, let’s file a bug -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12368 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12255] A few changes to .*ignore
Senthil Kumaran sent...@uthcode.com added the comment: I strictly have not faced any problem with those rej and orig files being present in the .hgignore. I also thought keeping the .hgignore in the repo was for distribution and never knew that it can affect local views (If you remove .rej and .orig your local version, will it help?) Yes, mercurial presents us with a list of files are that being added/committed, so we can be careful in not add those spurious leftovers. But at a time when you want to be fast with doing everything via cli options you are at a risk of adding unwanted files (unless protected by .hgignore file). I see that .hgignore can ignore itself. So, once commited with itself to ignore, I believe people can have personal customization in there. This suggestion is with the understanding that repo/.hgignore is just to distribution and does not affect hg tracking behavior. Please correct me if I am wrong here. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12369] Revised core mentorship section of help.rst
New submission from Adam Woodbeck adam.woodb...@gmail.com: Here is the latest update to the devguide's help.rst for your consideration. It includes Nick's tweaks to what I originally submitted. -- components: Devguide files: help.rst.patch keywords: patch messages: 138668 nosy: adam.woodbeck, ncoghlan priority: normal severity: normal status: open title: Revised core mentorship section of help.rst versions: Python 2.6, Python 2.7, Python 3.1, Python 3.2, Python 3.3, Python 3.4 Added file: http://bugs.python.org/file22410/help.rst.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12369 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12291] file written using marshal in 3.2 can be read by 2.7, but not 3.2 or 3.3
Benjamin Peterson benja...@python.org added the comment: I think you're right about playing with the bare fd being too fragile. I think a simpler solution is to read say 1024 bytes at a time and buffer it internally. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12291 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue3067] setlocale error message is confusing
Changes by STINNER Victor victor.stin...@haypocalc.com: -- nosy: +haypo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3067 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12370] Use of super overwrites use of __class__ in class namespace
New submission from Michael Foord mich...@voidspace.org.uk: In Python 3 the following code prints False because the use of super() has caused the __class__ descriptor to be omitted from the class namespace. Remove the use of super and it prints True. class X(object): def __init__(self): super().__init__() @property def __class__(self): return int print (isinstance(X(), int)) -- messages: 138670 nosy: michael.foord priority: normal severity: normal status: open title: Use of super overwrites use of __class__ in class namespace type: behavior versions: Python 3.2, Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12370 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9436] test_sysconfig failure: build a 32-bit Python a 64-bit OS
Changes by STINNER Victor victor.stin...@haypocalc.com: -- title: test_sysconfig failure - test_sysconfig failure: build a 32-bit Python a 64-bit OS ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9436 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue7652] Merge C version of decimal into py3k.
Changes by STINNER Victor victor.stin...@haypocalc.com: -- nosy: +haypo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue7652 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12370] Use of super overwrites use of __class__ in class namespace
Changes by Alex Gaynor alex.gay...@gmail.com: -- nosy: +alex ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12370 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12191] Add shutil.chown to allow to use user and group name (and not only uid/gid)
STINNER Victor victor.stin...@haypocalc.com added the comment: shutil_chown-default-v3.patch: - os.chown() is only available on Unix, shutil.chown() should not be defined if os.chown() doesn't exist (require to add a @unittest.skipUnless decorator to the test) - tests: is it possible that shutil.chown() exists whereas pwd or grp module is missing? if yes, you should split the test into two parts. it looks like pwd and grp are avaiable on Unix. - os.chown(path, -1, -1) is accepted, why not accepting shutil.chown(path) (shutil.chown(path, None, None))? - style: i don't like _user name, i suggest uid and gid (or user_id and group_id) instead of _user and _group... or you can reuse user/group variables - style: else: \n if not isinstance(user, int): can be written elif not isinstance(user, int): to avoid an useless level of indentation - tests: you may only call os.getuid() and os.getgid() once -- nosy: +haypo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12191 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12291] file written using marshal in 3.2 can be read by 2.7, but not 3.2 or 3.3
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment: Benjamin Peterson benja...@python.org added the comment: I think you're right about playing with the bare fd being too fragile. I think a simpler solution is to read say 1024 bytes at a time and buffer it internally. Doesn't this suffer from a similar problem? Namely, external Python code expecting the stream pointer to always be just past the object just read. See my earlier comments pointing out that there's nowhere to store the buffer state between successive calls to marshal.load. And the synchronising can be a problem to achieve with non-seekable streams (including, but not limited to, sockets). I'm not trying to be difficult. No, really! :-) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12291 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12314] regrtest checks (os.environ, sys.path, etc.) are hard to use
STINNER Victor victor.stin...@haypocalc.com added the comment: I’d like regrtest to tell me what exactly was changed, and where. regrtests has many tests (you give some examples: os.environ, sys.path), run all tests after calling a single test function would make regrtest slower. We can add an option (e.g. --strict?) to run these sanity tests after calling a single test function. I would also be nice to have this option for the refleak checks: run refleak checks on each function, instead of doing the checks on the whole file. -- nosy: +haypo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12314 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12338] multiprocessing.util._eintr_retry doen't recalculate timeouts
Changes by STINNER Victor victor.stin...@haypocalc.com: -- nosy: +haypo, pitrou ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12338 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12338] multiprocessing.util._eintr_retry doen't recalculate timeouts
STINNER Victor victor.stin...@haypocalc.com added the comment: subprocess._communicate_with_select() retries select.select() on EINTR: it recomputes timeout before each call. while self._read_set or self._write_set: timeout = self._remaining_time(endtime) if timeout is not None and timeout 0: raise TimeoutExpired(self.args, orig_timeout) try: (rlist, wlist, xlist) = \ select.select(self._read_set, self._write_set, [], timeout) except select.error as e: if e.args[0] == errno.EINTR: continue raise ... It has a similar code for select.poll(). asyncore.poll() handles EINTR: it just returns. I think it would be better to just implement the retrying version of select directly. It would be nice to share more code between subprocess and multiprocessing, but I don't know where such code should be moved. Create a new module just for one function is stupid. Handling EINTR is a common problem when managing subprocesses. subprocess has a _eintr_retry_call() function. multiprocessing has a @_eintr_retry decorator handling supporting more error types (use except (EnvironmentError, select.error):). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12338 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12315] Improve http.client.HTTPResponse.read documentation
Roundup Robot devnull@devnull added the comment: New changeset d801b570b1dd by Senthil Kumaran in branch '3.2': Fix closes Issue12315 - Updates to http.client documentation. http://hg.python.org/cpython/rev/d801b570b1dd -- nosy: +python-dev resolution: - fixed stage: patch review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12315 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12370] Use of super overwrites use of __class__ in class namespace
Roundup Robot devnull@devnull added the comment: New changeset 2d62ee4e7d98 by Benjamin Peterson in branch 'default': use a invalid name for the __class__ closure for super() (closes #12370) http://hg.python.org/cpython/rev/2d62ee4e7d98 -- nosy: +python-dev resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12370 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12359] tutorial: Module search path description is incorrect
Roundup Robot devnull@devnull added the comment: New changeset 6c16a31e0442 by Senthil Kumaran in branch '2.7': Fix issue12359. Minor doc update on import module description. http://hg.python.org/cpython/rev/6c16a31e0442 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12359 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12359] tutorial: Module search path description is incorrect
Roundup Robot devnull@devnull added the comment: New changeset bf8b4c43fb94 by Senthil Kumaran in branch '3.2': Fix closes Issue12359 - Minor update to module import description. http://hg.python.org/cpython/rev/bf8b4c43fb94 New changeset 8754fd2ff64a by Senthil Kumaran in branch 'default': merge from 3.2. Issue 12359 http://hg.python.org/cpython/rev/8754fd2ff64a -- nosy: +python-dev resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12359 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12360] Doc Typo
Roundup Robot devnull@devnull added the comment: New changeset d310cc1c3a5d by Senthil Kumaran in branch '3.2': Fix closes issue 12360 - correcting parameter names in asyncore documentation. http://hg.python.org/cpython/rev/d310cc1c3a5d -- nosy: +python-dev resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12360 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12365] URLopener should support context manager protocol
Jeff McNeil j...@jmcneil.net added the comment: In looking at this again, I may have spoken too soon. It seems that addinfobase HTTPResponse already handle this. As this is what's returned by the opener, then what I was shooting for should already be handled. -- status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12365 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11376] Solaris/GCC/shared lib problem
Dave Abrahams d...@boostpro.com added the comment: I run: sudo pip install --upgrade twisted -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11376 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11376] Solaris/GCC/shared lib problem
Dave Abrahams d...@boostpro.com added the comment: Note: even after I install Sun CC, -KPIC is unrecognized. At least it's only a warning in this case: $ sudo pip install --upgrade twisted Downloading/unpacking twisted Running setup.py egg_info for package twisted Downloading/unpacking zope.interface (from twisted) Running setup.py egg_info for package zope.interface Downloading/unpacking setuptools (from zope.interface-twisted) Running setup.py egg_info for package setuptools Installing collected packages: setuptools, twisted, zope.interface Found existing installation: setuptools 0.6c12dev-r88846 Uninstalling setuptools: Successfully uninstalled setuptools Running setup.py install for setuptools Installing easy_install script to /usr/bin Installing easy_install-2.6 script to /usr/bin Found existing installation: Twisted 10.0.0 Uninstalling Twisted: Successfully uninstalled Twisted Running setup.py install for twisted /usr/lib/python2.6/pycc -DNDEBUG -KPIC -I/usr/include/python2.6 -c conftest.c -o conftest.o cc: unrecognized option `-KPIC' /usr/lib/python2.6/pycc -DNDEBUG -KPIC -I/usr/include/python2.6 -c conftest.c -o conftest.o cc: unrecognized option `-KPIC' conftest.c:1:23: sys/epoll.h: No such file or directory building 'twisted.runner.portmap' extension /usr/lib/python2.6/pycc -DNDEBUG -KPIC -I/usr/include/python2.6 -c twisted/runner/portmap.c -o build/temp.solaris-2.11-i86pc-2.6/twisted/runner/portmap.o cc: unrecognized option `-KPIC' /usr/lib/python2.6/pycc -G build/temp.solaris-2.11-i86pc-2.6/twisted/runner/portmap.o -L/usr/lib -lpython2.6 -o build/lib.solaris-2.11-i86pc-2.6/twisted/runner/portmap.so building 'twisted.test.raiser' extension /usr/lib/python2.6/pycc -DNDEBUG -KPIC -I/usr/include/python2.6 -c twisted/test/raiser.c -o build/temp.solaris-2.11-i86pc-2.6/twisted/test/raiser.o cc: unrecognized option `-KPIC' /usr/lib/python2.6/pycc -G build/temp.solaris-2.11-i86pc-2.6/twisted/test/raiser.o -L/usr/lib -lpython2.6 -o build/lib.solaris-2.11-i86pc-2.6/twisted/test/raiser.so building 'twisted.python._initgroups' extension /usr/lib/python2.6/pycc -DNDEBUG -KPIC -I/usr/include/python2.6 -c twisted/python/_initgroups.c -o build/temp.solaris-2.11-i86pc-2.6/twisted/python/_initgroups.o cc: unrecognized option `-KPIC' /usr/lib/python2.6/pycc -G build/temp.solaris-2.11-i86pc-2.6/twisted/python/_initgroups.o -L/usr/lib -lpython2.6 -o build/lib.solaris-2.11-i86pc-2.6/twisted/python/_initgroups.so building 'twisted.internet._sigchld' extension /usr/lib/python2.6/pycc -DNDEBUG -KPIC -I/usr/include/python2.6 -c twisted/internet/_sigchld.c -o build/temp.solaris-2.11-i86pc-2.6/twisted/internet/_sigchld.o cc: unrecognized option `-KPIC' In file included from /usr/include/python2.6/Python.h:8, from twisted/internet/_sigchld.c:9: /usr/include/python2.6/pyconfig.h:1004:1: warning: _FILE_OFFSET_BITS redefined In file included from /usr/include/signal.h:36, from twisted/internet/_sigchld.c:6: /usr/include/sys/feature_tests.h:209:1: warning: this is the location of the previous definition /usr/lib/python2.6/pycc -G build/temp.solaris-2.11-i86pc-2.6/twisted/internet/_sigchld.o -L/usr/lib -lpython2.6 -o build/lib.solaris-2.11-i86pc-2.6/twisted/internet/_sigchld.so changing mode of build/scripts-2.6/manhole from 644 to 755 changing mode of build/scripts-2.6/mktap from 644 to 755 changing mode of build/scripts-2.6/pyhtmlizer from 644 to 755 changing mode of build/scripts-2.6/twistd from 644 to 755 changing mode of build/scripts-2.6/trial from 644 to 755 changing mode of build/scripts-2.6/tap2rpm from 644 to 755 changing mode of build/scripts-2.6/tap2deb from 644 to 755 changing mode of build/scripts-2.6/tapconvert from 644 to 755 changing mode of build/scripts-2.6/mailmail from 644 to 755 changing mode of build/scripts-2.6/conch from 644 to 755 changing mode of build/scripts-2.6/cftp from 644 to 755 changing mode of build/scripts-2.6/tkconch from 644 to 755 changing mode of build/scripts-2.6/ckeygen from 644 to 755 changing mode of build/scripts-2.6/lore from 644 to 755 changing mode of /usr/bin/tap2deb to 755 changing mode of /usr/bin/ckeygen to 755 changing mode of /usr/bin/tap2rpm to 755 changing mode of /usr/bin/mktap to 755 changing mode of /usr/bin/pyhtmlizer to 755 changing mode of /usr/bin/twistd to 755 changing mode of /usr/bin/lore to 755 changing mode of /usr/bin/mailmail to 755 changing mode of /usr/bin/cftp to 755 changing mode of /usr/bin/tkconch to 755 changing mode of /usr/bin/conch to 755 changing mode of /usr/bin/trial to 755 changing mode of /usr/bin/tapconvert to 755 changing mode of /usr/bin/manhole to 755 Found existing installation: zope.interface 3.3.0 Uninstalling
[issue12255] A few changes to .*ignore
R. David Murray rdmur...@bitdance.com added the comment: I have no idea what you mean by for distrabution in this context. The .hgignore in the checkout causes hg to ignore the files/patterns listed in it when any hg commands are run against that checkout. And no I can't remove it in my local copy, since the whole point of the file is that is is under version control in the repo. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12255] A few changes to .*ignore
Senthil Kumaran sent...@uthcode.com added the comment: By for distribution, I meant that everyone who checks out ( clones) from hg.cpython.org repository share the same .hgignore file, instead of each person having to create their own file. I got an impression from the earlier message that hg tracks (or ignores tracking) based on .hgignore file in the server itself, which I wanted to clarify. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6734] Imap lib implicit conversion from bytes to string
R. David Murray rdmur...@bitdance.com added the comment: This was fixed in issue 4471 by Antoine when he added some tests that call login. He fixed it by changing _quote to work with strings. Per the discussion here I'm not sure this is the best fix, but until someone reports a bug with it it we may as well let it stand. -- nosy: +pitrou resolution: - fixed stage: test needed - committed/rejected status: open - closed superseder: - IMAP4 missing support for starttls ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6734 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12255] A few changes to .*ignore
R. David Murray rdmur...@bitdance.com added the comment: Well, since there's no central server for hg, it can't track based on the server. Perhaps I confused you by saying that the .hgignore file was in the repo. (It is in the repo in the sense that any tracked file is in the repo, but it's effect comes from its existence in the checkout tree from which you are running the hg commands.) So yes, the .hgignore you get when you clone is so that everybody has the same base list. That's why we are arguing about its appropriate contents :) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12255 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12291] file written using marshal in 3.2 can be read by 2.7, but not 3.2 or 3.3
Changes by R. David Murray rdmur...@bitdance.com: -- nosy: +pitrou ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12291 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue4470] smtplib SMTP_SSL not working.
Lorenzo M. Catucci lore...@sancho.ccd.uniroma2.it added the comment: I'd still prefer if smtplib_05_shutdown_socket_v2.patch could get in, ^^ since, this way the REMOTE socket close will be unconditionally correct, instead of being dependent on GC artifacts. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4470 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
