Changes by Serhiy Storchaka storch...@gmail.com:
Removed file: http://bugs.python.org/file27826/sched_unblock_2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16165
___
Serhiy Storchaka added the comment:
Oh, I submitted wrong patch (it contained unrelated changes). Here is a right
one.
--
Added file: http://bugs.python.org/file28254/sched_unblock_3.patch
___
Python tracker rep...@bugs.python.org
New submission from Serhiy Storchaka:
In additional to issue16165 I think that sched.scheduler runs more code under a
lock than needed. The proposed patch moves this code outside a lock. This
unlikely a bug, may be only small enhancement and cleanup.
--
components: Library (Lib)
Antoine Pitrou added the comment:
Can't you propose a PEP for this? I'd like to see the DSL officially discussed
(or at least validated).
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16612
Roundup Robot added the comment:
New changeset 1996171cfb96 by Antoine Pitrou in branch '3.2':
Issue #16628: Fix a memory leak in ctypes.resize().
http://hg.python.org/cpython/rev/1996171cfb96
New changeset cae5f3266b81 by Antoine Pitrou in branch '3.3':
Issue #16628: Fix a memory leak in
Antoine Pitrou added the comment:
Should be fixed now.
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16628
New submission from Serhiy Storchaka:
Example:
import sched
s=sched.scheduler()
s.enter(10, 1, print)
Event(time=7452.676787873, priority=1, action=built-in function print,
argument=[], kwargs={})
next(s.queue).argument.append(spam)
s.enter(5, 1, print)
Event(time=7473.326011725,
Changes by Serhiy Storchaka storch...@gmail.com:
--
title: sched.scheduler.enter arguments should be modifiable -
sched.scheduler.enter arguments should not be modifiable
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16641
Antoine Pitrou added the comment:
Minor bikeshedding/spelling nit: should
_Py_AllocedBlocks
be changed to
_Py_AllocatedBlocks
(and s/_Py_GetAllocedBlocks/_Py_GetAllocatedBlocks/)?
Right indeed. I'll do the change.
--
___
Python tracker
Changes by Serhiy Storchaka storch...@gmail.com:
Removed file: http://bugs.python.org/file28256/sched_nonmodifiable_args.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16641
___
Serhiy Storchaka added the comment:
Sorry, wrong patch.
--
Added file: http://bugs.python.org/file28257/sched_nonmodifiable_args.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16641
___
Changes by Nick Coghlan ncogh...@gmail.com:
--
nosy: +ncoghlan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4395
___
___
Python-bugs-list mailing
Changes by Nick Coghlan ncogh...@gmail.com:
--
nosy: +antocuni
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4395
___
___
Python-bugs-list mailing
New submission from Serhiy Storchaka:
Here is a patch which adds missed mentions of kwargs named tuple parameter.
--
assignee: docs@python
components: Documentation
files: sched_docs_kwargs.patch
keywords: patch
messages: 177154
nosy: docs@python, giampaolo.rodola, serhiy.storchaka
New submission from Serhiy Storchaka:
sched.scheduler() prefers to use time.monotonic as a default value for timefunc
parameter, and time.time used only as a fallback. But time.time documented as a
default value.
--
assignee: docs@python
components: Documentation
messages: 177155
Changes by Chris Jerdonek chris.jerdo...@gmail.com:
--
nosy: +chris.jerdonek
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4395
___
___
Chris Jerdonek added the comment:
It seems the clinic.txt DSL document should be proofread for proper/consistent
use of argument/parameter (e.g. as described in the recently added
http://docs.python.org/dev/glossary.html#term-parameter ). To choose a couple
random examples--
+Argument
Changes by Chris Jerdonek chris.jerdo...@gmail.com:
--
nosy: +ezio.melotti
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16612
___
___
Roundup Robot added the comment:
New changeset 8ba3c975775b by Nick Coghlan in branch '3.3':
Issue #15209: Clarify exception chaining description
http://hg.python.org/cpython/rev/8ba3c975775b
New changeset 5854101552c2 by Nick Coghlan in branch 'default':
Merge from 3.3 (Issue #15209)
Nick Coghlan added the comment:
I rewrote the relevant section of the module docs (since they were a bit murky
in other ways as well).
Since I didn't answer the question earlier, the main reason a bare raise is
permitted is because it's designed to be used to a bare except clause (e.g.
when
Roundup Robot added the comment:
New changeset 3345afd6dc61 by Nick Coghlan in branch '3.3':
Close issue #16267: better docs for @abstractmethod composition
http://hg.python.org/cpython/rev/3345afd6dc61
New changeset be7202c38089 by Nick Coghlan in branch 'default':
Merge from 3.3 (issue
Nick Coghlan added the comment:
OK, after a long detour that delved deep into codeop and the vagaries of
PyCF_DONT_IMPLY_DEDENT (due to a problem that turned out to be due to a missing
\n in a test case I added), my main conclusion is:
Coupling the single vs multiple statement decision to the
Andrew Svetlov added the comment:
Committed, except whatsnew files.
That ones should be untouched from my perspective — the issue is not bugfix but
improvement and I see no reason why we need to backedit whatsnew for old
outdated versions.
Changesetes fed7306f26ce, 268ead8ae46b, 9e4b003a4d7a,
Stefan Krah added the comment:
I second Antoine's request for a PEP. I'm not only concerned about the DSL:
This approach may work for modules like posixmodule.c, where each function
is largely a self-contained unit wrapping some API function.
On the other hand, I don't want to imagine how
Ethan Furman added the comment:
There is one typo and one error in the first paragraph of the patch:
When raising a new exception (rather than
using to bare ``raise`` to re-raise the
^ should be an 'a'
exception currently being handled), the
implicit exception chain can be made
New submission from Serhiy Storchaka:
Test ContextManagerTests.test_invalid_args() in file
Lib/test/test_subprocess.py contains code which has no sense. When Popen() fail
then this code will not reachable. When Popen() success then c.exception will
not assigned.
Here is a patch with right
New submission from Serhiy Storchaka:
Test test_extract_hardlink() in Lib/test/test_tarfile.py ran 4 times. First
time (uncompressed tarfile) it extract symbolic and hard links successfully,
but in other 3 times (gzipped, bzipped and lzma'ed tarfile) extracting fails
becouse this links
Larry Hastings added the comment:
Antoine, Stefan: There doesn't appear to be a bright line separating this
should get a PEP from this doesn't need a PEP. That said, changes to the C
API for CPython don't seem to merit PEPs very often, the recent Stable ABI
being the lone exception to the
Antoine Pitrou added the comment:
Antoine, Stefan: There doesn't appear to be a bright line separating
this should get a PEP from this doesn't need a PEP. That said,
changes to the C API for CPython don't seem to merit PEPs very often,
the recent Stable ABI being the lone exception to the
Alex Gaynor added the comment:
For what it's worth, I'm not as concerned with the process of the PEP, as
having a single document we can review and discuss.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16612
Antoine Pitrou added the comment:
(and, oh, I agree there isn't a bright line separating this should get
a PEP from this doesn't need a PEP)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16612
Daniel Shahaf added the comment:
@Eli, thanks. In the meantime, attaching a patch addressing Ezio's review; the
caveats from msg 177134 still apply.
--
Added file: http://bugs.python.org/file28261/i16076-v4-combined.diff
___
Python tracker
New submission from Serhiy Storchaka:
In FTP.makeport() a socket error catched and saved but then raised a new error
with saved error as argument.
Here is a patch which reraises an original error.
--
components: Library (Lib)
files: ftplib_makeport_raise_err.patch
keywords: patch
Changes by Serhiy Storchaka storch...@gmail.com:
Removed file: http://bugs.python.org/file28262/ftplib_makeport_raise_err.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16646
___
Changes by Serhiy Storchaka storch...@gmail.com:
Added file: http://bugs.python.org/file28263/ftplib_makeport_raise_err.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16646
___
New submission from Serhiy Storchaka:
In LMTP.connect() a socket error catched and saved but then raised a new error
with saved error as argument.
Here is a patch which reraises an original error.
--
components: Library (Lib)
files: smtplib_LMTP_connect_raise_err.patch
keywords: patch
Changes by Carsten Grohmann carstengrohm...@gmx.de:
--
nosy: +cgrohmann
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11159
___
___
Antoine Pitrou added the comment:
The test comes from e02021fbfe22 (Fix #10554. Added context manager support to
Popen objects.).
--
nosy: +brian.curtin
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16644
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +brian.curtin, tim.golden
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16632
___
___
New submission from Andrew Svetlov:
http://www.python.org/dev/peps/pep-3151 define exception classes like
ConnectionError and BrokenPipeError
We need to refactor stdlib to use that shiny new exceptions instead of checking
errno values if possible.
Also maybe we need to add some absent
Serhiy Storchaka added the comment:
Right now I'm working on a patch. ;) A pack of issues were produced in process.
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16648
Andrew Svetlov added the comment:
Please, feel free to add me in nosylist for any issue related to this one.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16648
___
Roundup Robot added the comment:
New changeset 0748c22b37e5 by Antoine Pitrou in branch '3.2':
Issue #16602: When a weakref's target was part of a long deallocation chain,
the object could remain reachable through its weakref even though its refcount
had dropped to zero.
Roundup Robot added the comment:
New changeset 7e771f0363e2 by Antoine Pitrou in branch '2.7':
Issue #16602: When a weakref's target was part of a long deallocation chain,
the object could remain reachable through its weakref even though its refcount
had dropped to zero.
Antoine Pitrou added the comment:
Thank you very much for reporting this issue. I believe it is now fixed.
--
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
Changes by Daniel Urban urban.dani...@gmail.com:
--
type: crash - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11159
___
___
Eugene Toder added the comment:
Thank you, Antoine.
This change has one effect that's worth highlighting in NEWS at least -- the
PyWeakref_GET_OBJECT() macro now evaluates its argument twice. This can break
existing code where the argument has side-effects, e.g. o =
PyWeakref_GET_OBJECT(p++).
Antoine Pitrou added the comment:
I don't think there's a way to write PyWeakref_GET_OBJECT() in standard
C90 without double evaluation.
Agreed. We generally don't document whether our macros are compatible with side
effects in parameters, and I think we'd like to keep it that way. People
Eugene Toder added the comment:
People should simply avoid doing this kind of thing, as it's knowingly
fragile, and trivial to avoid anyway.
Is this documented in the C API guide, or somewhere else?
In any case, notifying people so they can quickly check their code seems much
nicer than
Antoine Pitrou added the comment:
People should simply avoid doing this kind of thing, as it's
knowingly fragile, and trivial to avoid anyway.
Is this documented in the C API guide, or somewhere else?
I don't think so, but it's common C wisdom that you shouldn't pass
arguments which have
Antoine Pitrou added the comment:
In any case, notifying people so they can quickly check their code
seems much nicer than having them figure this out the hard way.
I see it as a double-edged sword: if we start adding a warning for this
macro, people will expect us to do it for every
Aaron Iles added the comment:
Should a new issue be created to decouple print after every call from the
single vs multiple statement condition that is a blocker for this issue? Or
can it be resolved here?
On Sunday, 9 December 2012, Nick Coghlan wrote:
Nick Coghlan added the comment:
OK,
Changes by Bradley Froehle brad.froe...@gmail.com:
--
nosy: +bfroehle
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16612
___
___
Python-bugs-list
Éric Araujo added the comment:
On a Ubuntu system you don’t need to install readline from PyPI: CPython has a
readline module, which gets compiled if the headers are available.
That said, the error you report is still strange. Can you attach the full
console output as a text file?
Éric Araujo added the comment:
Vinay, do you think dereferencing sys.executable could lead to trouble with
venvs?
--
nosy: +vinay.sajip
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16326
Éric Araujo added the comment:
Thanks Dwayne. I think that most tools (e.g. setuptools) generate things like
1.0.2+gitab25c6fe95ee92fac3187dcd90e0560ccacb084a i.e. real version set by
maintainer + hash, not just the hash, so I am still not convinced a change is
needed.
--
stage:
Éric Araujo added the comment:
I’m reluctant to break links. I think the issues you report could be fixed or
mitigated with changes to text, clarifications, interlinks. What do you think?
--
___
Python tracker rep...@bugs.python.org
Chris Jerdonek added the comment:
I've been meaning to ask Georg or anyone else knowledgeable about what options
we have for redirects, etc. -- specifically for this issue. Maybe something
similar can even be done using Sphinx.
If we can't redirect, I was thinking even a stub page with a
Roundup Robot added the comment:
New changeset 692be1f9fa1d by Éric Araujo in branch '2.7':
Fix setup.py register failure with invalid rst in description (#13614).
http://hg.python.org/cpython/rev/692be1f9fa1d
--
___
Python tracker
Roundup Robot added the comment:
New changeset d978828bfd0e by Éric Araujo in branch '3.2':
Ignore .nfs* files in distutils (#7719).
http://hg.python.org/cpython/rev/d978828bfd0e
New changeset 10ab746f55fb by Éric Araujo in branch '3.3':
Merge fixes for #13614, #13512 and #7719 from 3.2
Roundup Robot added the comment:
New changeset 99c80e8a721e by Éric Araujo in branch '3.2':
Fix setup.py register failure with invalid rst in description (#13614).
http://hg.python.org/cpython/rev/99c80e8a721e
New changeset 10ab746f55fb by Éric Araujo in branch '3.3':
Merge fixes for #13614,
Roundup Robot added the comment:
New changeset 4a2814f24a10 by Éric Araujo in branch '3.2':
Create ~/.pypirc securely (#13512).
http://hg.python.org/cpython/rev/4a2814f24a10
New changeset 10ab746f55fb by Éric Araujo in branch '3.3':
Merge fixes for #13614, #13512 and #7719 from 3.2
Éric Araujo added the comment:
Thanks all!
--
resolution: - fixed
stage: commit review - committed/rejected
status: open - closed
versions: +Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7719
Changes by Éric Araujo mer...@netwok.org:
--
resolution: - fixed
stage: commit review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13614
___
Changes by Éric Araujo mer...@netwok.org:
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
versions: +Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13512
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/issue16267
___
Roundup Robot added the comment:
New changeset 3b67247f0bbb by Nick Coghlan in branch '3.3':
Issue #15209: Fix typo and some additional wording tweaks
http://hg.python.org/cpython/rev/3b67247f0bbb
New changeset 04eb89e078b5 by Nick Coghlan in branch 'default':
Merge from 3.3 (issue #15209)
Nick Coghlan added the comment:
On Sun, Dec 9, 2012 at 2:59 AM, Ethan Furman rep...@bugs.python.org wrote:
Ethan Furman added the comment:
There is one typo and one error in the first paragraph of the patch:
When raising a new exception (rather than
using to bare ``raise`` to re-raise
Sean Ochoa added the comment:
Update based on Taggnostr's (Ezio on IRC, if I recall correctly) feedback from
11/12/2012 around 11:57 PST.
* Added check for index result in positive tests.
* Added assertIn check for remove result in positive tests.
* Removed extra whitespace.
* Formatted
New submission from Nick Coghlan:
code.InteractiveConsole.push attempts to support multi-line input when using an
input method that doesn't reliably split on newlines.
This support fails as soon as the multi-line input contains multiple statements
(including when the last statement is
Changes by Nick Coghlan ncogh...@gmail.com:
--
nosy: +benjamin.peterson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16649
___
___
Changes by Nick Coghlan ncogh...@gmail.com:
--
title: Add a PyCF_PRINT_EXPRESSION_RESULTS flag - Add a
PyCF_DISPLAY_EXPRESSION_RESULTS flag
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16649
Changes by Nick Coghlan ncogh...@gmail.com:
--
stage: - needs patch
type: - enhancement
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16649
___
Nick Coghlan added the comment:
More implicit magic seems like a bad idea, so I've split out a proposal for an
explicit PyCF_DISPLAY_EXPRESSION_RESULTS flag as #16649.
The behaviour would then be selectable regardless of the compilation mode, but
would remain the default for single.
New submission from Serhiy Storchaka:
As noted in Popen._internal_poll() docstring this method cannot reference
anything outside of the local scope. However it references errno.ECHILD. The
proposed patch fixes this.
Is it good that Popen._handle_exitstatus() references building
Mathieu Dutour Sikiric added the comment:
Well, that was the points, the headers were not available.
Mathieu
On Sunday, December 9, 2012, Éric Araujo wrote:
Éric Araujo added the comment:
On a Ubuntu system you don’t need to install readline from PyPI: CPython
has a readline module,
Sean Ochoa added the comment:
* Fixed issue with test name in Lib/test/test_tuple.py
* Fixed issue with test_remove in Lib/test/test_list.py to assertNotIn instead
of assertIn for positive case.
Confirmed with Ezio that issue #7330 will need to be fixed/approved before this
issue can be
Changes by Sean Ochoa sean.m.oc...@gmail.com:
--
nosy: +Sean.Ochoa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7330
___
___
Python-bugs-list
Georg Brandl added the comment:
Redirects are no problem in principle, but it should not get out of hand. A
simple this was here but now isn't page might be better.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16406
79 matches
Mail list logo