Daniel Urban urban.dani...@gmail.com added the comment:
It doesn't work with staticmethod:
import abc
class C(metaclass=abc.ABCMeta):
... @staticmethod
... @abc.abstractmethod
... def foo(x):
... raise NotImplementedError()
...
class D(C):
... @staticmethod
...
Darren Dale dsdal...@gmail.com added the comment:
On Sat, Jun 11, 2011 at 3:11 AM, Daniel Urban rep...@bugs.python.org wrote:
Daniel Urban urban.dani...@gmail.com added the comment:
It doesn't work with staticmethod:
import abc
class C(metaclass=abc.ABCMeta):
... @staticmethod
...
Charles-François Natali neolo...@free.fr added the comment:
I think it might be related to Issue #6721.
Using a mutex/condition variable after fork (from the child process) is unsafe:
it can lead to deadlocks, and on OS-X, it seems like it can lead to segfaults.
Normally, Queue's
Graham Wideman initcont...@grahamwideman.com added the comment:
Hi Eric, Thanks for starting to review this, and your responses are
encouraging. Some comments inline below.
FWIW, along the way I accumulated my own notes on this topic, on some pages
here:
grahamwideman.wikispaces.com
(Left
Darren Dale dsdal...@gmail.com added the comment:
[...]
Traceback (most recent call last):
File stdin, line 1, in module
TypeError: Can't instantiate abstract class D with abstract methods
foo.__func__
You still need to use @abc.abstractstaticmethod.
Thinking about this some more, I
Nick Coghlan ncogh...@gmail.com added the comment:
Thanks for getting started with such a detailed review on this Graham. We've
known the documentation in this area has been flawed for a long time, but
actually *fixing* seemed like such a big task that it has tended to get pushed
to the
Nick Coghlan ncogh...@gmail.com added the comment:
inspect.getattr_static has the necessary logic to search for descriptors
without invoking them.
However, it may be better to revert to the idea of pushing this functionality
back onto the individual descriptors and have the problematic
Darren Dale dsdal...@gmail.com added the comment:
On Sat, Jun 11, 2011 at 8:55 AM, Nick Coghlan rep...@bugs.python.org wrote:
Nick Coghlan ncogh...@gmail.com added the comment:
inspect.getattr_static has the necessary logic to search for descriptors
without invoking them.
Unfortunately,
Changes by Greg Słodkowicz jerg...@gmail.com:
--
nosy: +Greg.Slodkowicz
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11553
___
___
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment:
Patch is now in my public sandbox on hg.python.org.
--
hgrepos: +26
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12291
___
Graham Wideman initcont...@grahamwideman.com added the comment:
Hi Nick: Thanks for your additional points. Comments inline:
__all__ only affects import *, and may also affect documentation tools (e.g.
pydoc will respect __all__ when deciding what to display). It has no effect
on attribute
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment:
For some reason, Create Patch is failing with a
[Errno 2] No such file or directory:
'/home/roundup/trackers/tracker/cpython/Doc/Makefile'
I've logged an issue on the meta tracker:
http://psf.upfronthosting.co.za/roundup/meta/issue405
Antoine Pitrou pit...@free.fr added the comment:
Less disruptive approach:
old_process = _current_process
_current_process = self
try:
util._finalizer_registry.clear()
util._run_after_forkers()
finally:
del old_process
This
Charles-François Natali neolo...@free.fr added the comment:
No one seems to object, and since this approach has been suggested by
Martin and is consistent with the posix module's policy (i.e. a thin
wrapper around the underlying syscall), I guess you can go ahead.
--
Charles-François Natali neolo...@free.fr added the comment:
For oss_check_closed.diff, should I apply it to default only or to every branch?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12287
Antoine Pitrou pit...@free.fr added the comment:
For oss_check_closed.diff, should I apply it to default only or to
every branch?
Only default I'd say.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12287
Charles-François Natali neolo...@free.fr added the comment:
Perhaps by adding some new argument to the mmap constructor? (dup_fd = True)
I don't really like the idea of exposing the FD duplication to the
user, because:
- it's an implementation detail
- it doesn't reflect any argument of the
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +orsenthil
stage: - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12315
___
___
Charles-François Natali neolo...@free.fr added the comment:
Yes, I also tried this.
This fixed the test_multiprocessing failure, but I think it triggered a failure
in test_concurrent_futures (didin't look in more detail).
Would it be possible to try this on the buildbot to see if it fixes the
Darren Dale dsdal...@gmail.com added the comment:
[...]
This wouldn't allow for the prettier error messages, but it's much cleaner
than having ABCMeta trawling through class attribute dir() lists.
I think there is another reason to do it this way. Suppose I have a
custom descriptor
Roundup Robot devnull@devnull added the comment:
New changeset d0952a2fb7bd by Charles-François Natali in branch 'default':
Issue #12287: In ossaudiodev, check that the device isn't closed in several
http://hg.python.org/cpython/rev/d0952a2fb7bd
--
nosy: +python-dev
Yuri y...@tsoft.com added the comment:
I fixed all build problems on the current MinGW32. python.exe builds ok, but
build fails since python.exe can't find some modules after this. Not sure why.
--
keywords: +patch
nosy: +yurivict
Added file:
Roundup Robot devnull@devnull added the comment:
New changeset 303e3693d634 by Éric Araujo in branch 'default':
Move useful function to packaging.util.
http://hg.python.org/cpython/rev/303e3693d634
New changeset 06670bd0e59e by Éric Araujo in branch 'default':
Fix assorted bugs in
Roundup Robot devnull@devnull added the comment:
New changeset 3ba34c03f2fc by Éric Araujo in branch 'default':
Adjust logging in packaging.util.spawn (related to #11599)
http://hg.python.org/cpython/rev/3ba34c03f2fc
--
nosy: +python-dev
___
Python
Roundup Robot devnull@devnull added the comment:
New changeset 5f0cd4844061 by Éric Araujo in branch 'default':
Allow multiple setup hooks in packaging’s setup.cfg files (#12240).
http://hg.python.org/cpython/rev/5f0cd4844061
--
nosy: +python-dev
___
Éric Araujo mer...@netwok.org added the comment:
Also you may want to add some of the packaging test directories to
those excluded in the calls to compileall.py a little further down in
the libinstall target.
Ah, good one. I guess doing this could have prevented
Éric Araujo mer...@netwok.org added the comment:
I just recreated this patch against version 2.7, so I'm not sure it
can be applied to all the listed versions.
It’s okay, distutils in 2.7 and 3.x is very similar, I can port.
Note: there still are two pathes, one for sdist.py and another for
Éric Araujo mer...@netwok.org added the comment:
Thanks again!
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11595
Éric Araujo mer...@netwok.org added the comment:
I intend to ask on distutils-sig whether doing the change in distutils would
break code.
--
assignee: tarek - eric.araujo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11599
Éric Araujo mer...@netwok.org added the comment:
I forgot one thing: setuptools’ egg_info command does write a list of paths, so
we can look at how it solved the problem with the RECORD and RESOURCES files.
Higery: Michael is willing to work with you on this bug.
--
Éric Araujo mer...@netwok.org added the comment:
Done.
--
assignee: tarek - eric.araujo
resolution: - fixed
stage: - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12240
Éric Araujo mer...@netwok.org added the comment:
Hmm, in http://bugs.python.org/issue7511#msg106420 Tarek appeared to
be supportive of the patch.
I believe Martin has more knowledge about Windows.
--
___
Python tracker rep...@bugs.python.org
Changes by Giampaolo Rodola' g.rod...@gmail.com:
--
assignee: giampaolo.rodola
nosy: giampaolo.rodola
priority: normal
severity: normal
status: open
title: inspect.getabsfile() is not documented
versions: Python 2.6, Python 2.7, Python 3.1, Python 3.2, Python 3.3
John Edmonds pocketcooki...@gmail.com added the comment:
Here is the patch, re-written for the packaging module.
--
Added file: http://bugs.python.org/file22330/patch.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12169
Roundup Robot devnull@devnull added the comment:
New changeset 527c40add91d by Benjamin Peterson in branch '2.7':
allow fake filenames in findsource (closes #9284)
http://hg.python.org/cpython/rev/527c40add91d
New changeset 6cc4579dca02 by Benjamin Peterson in branch '3.2':
allow fake filenames
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment:
Attached is a simple patch clarifying that the level argument in the
constructor maps to the two LogRecord attributes, levelno and levelname,
--
keywords: +patch
Added file: http://bugs.python.org/file22331/patch12206.diff
New submission from Filip Gruszczyński grusz...@gmail.com:
You can do this:
[1] + (1,)
Traceback (most recent call last):
File stdin, line 1, in module
TypeError: can only concatenate list (not tuple) to list
But you can do this:
result = [1]
result += (1,)
result
[1, 1]
Is it the
Filip Gruszczyński grusz...@gmail.com added the comment:
Obviously first sentence should be You can't do this:.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12318
___
Roundup Robot devnull@devnull added the comment:
New changeset bb6fe43191c0 by Vinay Sajip in branch '3.2':
Issue #12206: documentation for LogRecord constructor updated re. the level
argument.
http://hg.python.org/cpython/rev/bb6fe43191c0
New changeset 596adf14914c by Vinay Sajip in branch
Changes by Vinay Sajip vinay_sa...@yahoo.co.uk:
--
assignee: docs@python - vinay.sajip
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12206
___
___
Raymond Hettinger raymond.hettin...@gmail.com added the comment:
Yes, this is the expected behavior and yes, it is inconsistent.
It's been that way for a long while and Guido said he wouldn't do it again
(it's in his list of regrets). However, we're not going to break code by
changing it
Eric Snow ericsnowcurren...@gmail.com added the comment:
Perhaps rather than changing ABCMeta, provide a base descriptor class that has
__isabstractmethod__ implemented to calculate the abstractness. Then property
could use that, as could any of the other relevant descriptors we have around.
STINNER Victor victor.stin...@haypocalc.com added the comment:
Would it be possible to try this on the buildbot to see if it fixes
the segfaults?
You can fork cpython, modify the code, and run a custom buildbot on your
clone.
--
___
Python tracker
Nick Coghlan ncogh...@gmail.com added the comment:
Public name is a term that describes a convention, not anything enforced by
the interpreter. Names starting with underscores typically aren't public either
(unless documented otherwise), but that has no effect on the ability to
retrieve them
Nick Coghlan ncogh...@gmail.com added the comment:
Remember the goal here is *not* to completely eliminate the need to test that
objects implement an ABC correctly. It's to make it easier to declare the
expected interface in a way that helps readers of the ABC definition to figure
out what is
Changes by Mark Mc Mahon mtnbikingm...@gmail.com:
--
keywords: +patch
Added file:
http://bugs.python.org/file22332/support_vt_empty_in_summary_getproperty.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12239
Graham Wideman initcont...@grahamwideman.com added the comment:
Public name is a term that describes a convention, not anything enforced by
the interpreter.
And I guess that's really the main point. In other languages Public means
accessible, and Private means not so. In Python, Public
higery shoulderhig...@gmail.com added the comment:
OK. I recreated a full version patch. I'll remove old patches.
--
Added file:
http://bugs.python.org/file22333/change_path_separator_fullversion.patch
___
Python tracker rep...@bugs.python.org
Changes by higery shoulderhig...@gmail.com:
Removed file:
http://bugs.python.org/file21713/change_path_separator_in_MANIFEST.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue828450
___
Changes by higery shoulderhig...@gmail.com:
Removed file: http://bugs.python.org/file22328/test_manifest_reading_sdist.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue828450
___
Eric Snow ericsnowcurren...@gmail.com added the comment:
Didn't mean to sidetrack. :) I really appreciate the effort Darren has put
into this.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11610
higery shoulderhig...@gmail.com added the comment:
Higery: Michael is willing to work with you on this bug.
OK. :)
--
Added file: http://bugs.python.org/file22334/unnamed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12279
Changes by higery shoulderhig...@gmail.com:
Removed file: http://bugs.python.org/file22334/unnamed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12279
___
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
I was looking at this - and see that (at least as far as GetFileAttributes is
concerned) that a mount and a linked directory are seen the same...
Here are some tests using ctypes
# mounted drive
Changes by Brian Curtin br...@python.org:
--
nosy: +brian.curtin
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9035
___
___
Python-bugs-list
55 matches
Mail list logo