Ronald Oussoren ronaldousso...@mac.com added the comment:
On my system with the attached patch (os-getgroups.patch):
$ sudo ./python.exe
Python 2.7a3+ (trunk:78371M, Feb 23 2010, 09:19:44)
[GCC 4.2.1 (Apple Inc. build 5659)] on darwin
Type help, copyright, credits or license for more
Mark Dickinson dicki...@gmail.com added the comment:
BTW, should the changes to the generated configure be excluded
from such patches, or should they be included? (to what extent do
they contain meaningful information during review?)
Not sure. I think it's fine to leave the configure
Francesco Del Degan f.delde...@ngi.it added the comment:
I attached a patch that implements timegm according to two constraints:
1. If HAVE_TIMEGM is defined, use it
or
2. If HAVE_MKTIME and HAVE_WORKING_TZSET use a portable way, using mktime
(taken from timegm(3) man)
Attached patches are
Alexander Belopolsky alexander.belopol...@gmail.com added the comment:
On Tue, Feb 23, 2010 at 2:20 AM, Ronald Oussoren rep...@bugs.python.org wrote:
..
Alexander: Â What makes you think r63955 introduced the problem?
That revision introduced _DARWIN_C_SOURCE which, as I explained, has
two
Alexander Belopolsky alexander.belopol...@gmail.com added the comment:
Ronald,
Have you tested your patch with issue7900-tests.diff?
On my system:
$ ./python.exe Lib/test/test_posix.py
testNoArgFunctions (__main__.PosixTester) ... Segmentation fault
and
$ sudo ./python.exe
Changes by R. David Murray rdmur...@bitdance.com:
--
priority: - normal
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7900
___
___
Guido van Rossum gu...@python.org added the comment:
On Mon, Feb 22, 2010 at 6:51 PM, Jeremy Hylton rep...@bugs.python.org wrote:
There's no reason we couldn't revise the language spec to explain that
except clauses and comprehensions are block statements, i.e.
statements that introduce a new
Ezio Melotti ezio.melo...@gmail.com added the comment:
Here's a new patch that uses PyUnicode_FromStringAndSize and
PyUnicode_AS_UNICODE(s)[0]. I also added some comments and tested it and it
seems to work fine.
--
Added file: http://bugs.python.org/file16339/issue7649v3.diff
Ronald Oussoren ronaldousso...@mac.com added the comment:
My patch is very lightly tested and I didn't try to use the additional
testcases.
I get test failures, but no crashes, when I do run the additional tests. I
haven't had time to review the testcases though.
What I did notice though
Eric Smith e...@trueblade.com added the comment:
The patch looks good to me, in that it implements your desired functionality
well.
I haven't been following the issue closely enough to know whether or not this
new functionality is the right way to go. (I'm not saying it's not, just that I
Guido van Rossum gu...@python.org added the comment:
I don't think so. It's very marginal.
--Guido (on Android)
On Feb 23, 2010 8:52 AM, Amaury Forgeot dapos;Arc rep...@bugs.python.org
wrote:
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
The above patch adds a new opcode
Alexander Belopolsky alexander.belopol...@gmail.com added the comment:
On Tue, Feb 23, 2010 at 8:46 AM, Ronald Oussoren rep...@bugs.python.org wrote:
..
What I did notice though is that the tests don't actually test the issue at
hand:
having more than 16 secondairy groups on OSX.
This is
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
Interesting. What is the content of sys.path when you start ./python from the
build tree?
--
nosy: +amaury.forgeotdarc
___
Python tracker rep...@bugs.python.org
Éric Araujo mer...@netwok.org added the comment:
I believe the OP’s request is partly motivated by the fact that there is a need
to access a private function in warnings. See
http://mail.python.org/pipermail/python-ideas/2010-February/006870.html where
Nick Coghlan proposes that “the idea of
Marc-Andre Lemburg m...@egenix.com added the comment:
Brian Curtin wrote:
Brian Curtin cur...@acm.org added the comment:
Here is the patch which checks whether the new info is available and uses it,
otherwise it falls back to the original way.
Thanks. Could you apply the same logic to
New submission from Ronald Oussoren ronaldousso...@mac.com:
The stat implementation on osx 10.5 or later can provide additional fields
beyond the tradional unix ones (such as st_birthtimespec). Those are by
default only available when using 'stat64' instead of the regular stat, but
that can
Alexander Belopolsky alexander.belopol...@gmail.com added the comment:
On Tue, Feb 23, 2010 at 8:46 AM, Ronald Oussoren rep...@bugs.python.org wrote:
..
I get test failures, but no crashes, when I do run the additional tests.
The crash that I see is apparently due to underallocated memory:
Ronald Oussoren ronaldousso...@mac.com added the comment:
To be more precise: see issue8001.
The OSX 10.5 introduces a new version of stat 'stat64' that exposes additional
information to posix-style programs, amongst which is the file creation date.
This field is not present in earlier
Alexander Belopolsky alexander.belopol...@gmail.com added the comment:
E-mail to tracker handling still seems to remove '' quoted lines. I am
reposting a couple of comments that got clobbered:
--
* allocate an array of groupcount gid_t's and call getgroups(groupcount)
This is more or less
R. David Murray rdmur...@bitdance.com added the comment:
I see. I agree with Nick, a public API function in Warnings is better than
something specific to optparse or argparse. If the op (or anyone else) wants
to propose a patch to create such an API (with docs and tests), that would be
Jack Diederich jackd...@gmail.com added the comment:
how about an equivalent path instead of equal path? The result of
ntpath.join(ntpath.split(path)) should point to the same location even if it
isn't literally the same string.
--
nosy: +jackdied
Éric Araujo mer...@netwok.org added the comment:
Hello
There is a typo in the changed text that prevents correct reST interpretation.
s/```/``/ would fix it.
Regards
--
nosy: +merwok
___
Python tracker rep...@bugs.python.org
Alexander Belopolsky alexander.belopol...@gmail.com added the comment:
I think I finally understand what is going on and ready to make what I believe
is a reasonable proposal.
Setting _DARWIN_C_SOURCE (or _DARWIN_UNLIMITED_GETGROUPS) replaces getgroups
with a Darwin extension, but does not
Brian Curtin cur...@acm.org added the comment:
Whoops, sorry. Here is the change as requested. It reverts to the old way when
needed, which hardcodes the version to VER_NT_WORKSTATION as it effectively
used to.
Outside of a hackish and incredibly slow method, I have no idea how to figure
Ronald Oussoren ronaldousso...@mac.com added the comment:
I'm -1 w.r.t changing the definition of _DARWIN_C_SOURCE, at least not without
a thorough investigation on what else this changes. Removing _DARWIN_C_SOURCE
without setting some other defines may well cause in removal of functions from
Jack Diederich jackd...@gmail.com added the comment:
+1, the C patch looks good to me. The test file needs a new test that checks
the 'with' behavior. Also, what changed so that the test now needs to ignore
AttributeErrors in play_sound_file()?
--
nosy: +jackdied
Changes by Michael Foord mich...@voidspace.org.uk:
--
nosy: -michael.foord
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7900
___
___
Alexander Belopolsky alexander.belopol...@gmail.com added the comment:
The patch (I reviewed timemodule-gmtime-trunk.diff) looks mostly correct. One
problem that I see is that it will likely produce compiler warnings on the
systems without timegm and mktime. The warnings will be due to
Ronald Oussoren ronaldousso...@mac.com added the comment:
I'm probably being extremely dense at the moment, but the program below doesn't
behave as I'd expect on OSX:
#include unistd.h
int main(void)
{
#if 1
gid_t gids[] = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16,
17,
Senthil Kumaran orsent...@gmail.com added the comment:
There is a OLD Known issue that base64.encodestring will insert in a line-break
in the authorization header. See Issue1574068
I fear, you might have done something like this.
base64string = base64.encodestring('%s:%s'
Alexander Belopolsky alexander.belopol...@gmail.com added the comment:
On Tue, Feb 23, 2010 at 12:16 PM, Ronald Oussoren rep...@bugs.python.org
wrote:
..
How did you detect that the modern getgroups implementation doesn't
query the kernel? That would be very odd.
I wish I could say that I
New submission from Mads Kiilerich m...@kiilerich.com:
I had an issue which seems to be very similar to issue4690 - but different.
I created a subclass of asyncore.dispatcher, .create_socket added it to the
channel map, but then .bind failed (because the port was in use - my bad),
.listen
Eric Smith e...@trueblade.com added the comment:
This version of the patch adds support for classic classes and adds tests.
Documentation still needs to be written.
Again, this diff is against trunk.
If anyone wants to review this, in particular the tests that exercise
Alexander Belopolsky alexander.belopol...@gmail.com added the comment:
David,
Can you unset normal priority from this issue. Regardless of whether it is a
crash or not a crash, this is a regression for which there is no user
work-around. It looks like normal priority makes the issue
Gael gael.peglia...@makina-corpus.com added the comment:
Amaury Forgeot d'Arc a écrit :
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
Interesting. What is the content of sys.path when you start ./python from the
build tree?
I will tell you tomorrow, I have no access to the
Thomas Heller thel...@ctypes.org added the comment:
CThunkObject is not registered correctly with the cycle GC. The attached
patch, against trunk, fixes the issue.
--
keywords: +patch
Added file: http://bugs.python.org/file16345/issue7957.patch
___
Jeremy Hylton jer...@alum.mit.edu added the comment:
The patch looks pretty good.
I'd factor out the common error-checking code (common between
LOAD_DEREF and DELETE_DEREF) into a helper function.
It would also be good to add some test cases.
Jeremy
On Tue, Feb 23, 2010 at 9:38 AM, Guido van
Wim w...@.org added the comment:
I just ran across yet another implementation of sendmsg support for python
sockets, whose feature set seems to complement Kalman Gergely's implementation:
http://www.pps.jussieu.fr/~ylg/PyXAPI by Yves Legrandgerard
(Out of curiosity, people have been
Dirkjan Ochtman dirk...@ochtman.nl added the comment:
Fixed in SVN, r78384.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1537721
___
___
Changes by Dirkjan Ochtman dirk...@ochtman.nl:
--
assignee: - djc
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1537721
___
Georg Brandl ge...@python.org added the comment:
Fixed in r78385, thanks!
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8000
___
Jack Diederich jackd...@gmail.com added the comment:
We talked about it at sprints and the semantics are ambiguous and there are
alternatives.
Ambiguous:
def show_funcs(*args): print(args)
class A():
run = partial(1)
ob = A()
ob.run(2,3)
Should this print (self, 1, 2, 3) or (1,
Jack Diederich jackd...@gmail.com added the comment:
correction:
run = partial(1)
should have been
run = partial(show_funcs, 1)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4331
___
R. David Murray rdmur...@bitdance.com added the comment:
How does this make it easier to review the built documentation? Most web
browsers are happy to read off disk, and that seems easier than firing up a
server.
(I would actually have a use case for this, since the machine I build the docs
Dirkjan Ochtman dirk...@ochtman.nl added the comment:
I do almost all of my development on servers, so while I guess I could mount
the remote file system and do that, that's not very easy to do (since I access
my servers from multiple boxes, some of which run Windows, etc). I guess if not
Ronald Oussoren ronaldousso...@mac.com added the comment:
I've filed issue #7681618 in Apple's bugtracker with the attached file
getsetgroups-bug.tar as a way to reproduce the issue.
I want to wait a little to see how Apple responds (although I don't expect a
quick response).
--
Brett Cannon br...@python.org added the comment:
That's a package decision by Ubuntu, not Python, and thus not a bug on our end.
Thanks for the report anyway.
--
nosy: +brett.cannon
resolution: - invalid
status: open - closed
___
Python tracker
Ronald Oussoren ronaldousso...@mac.com added the comment:
quoting compat(5) on OSX 10.6.2:
quote
Defining _POSIX_C_SOURCE or _DARWIN_C_SOURCE causes library and kernel
calls to conform to the SUSv3
standards even if doing so would alter the behavior of functions used in
10.3.
Jeremy Hylton jer...@alum.mit.edu added the comment:
Is deferred blocker a higher priority?
Jeremy
On Tue, Feb 23, 2010 at 4:37 PM, Georg Brandl rep...@bugs.python.org wrote:
Georg Brandl ge...@python.org added the comment:
High is not high enough :)
--
priority: high - deferred
R. David Murray rdmur...@bitdance.com added the comment:
Ah, windows. And if it was there, I'd probably use it. Let's see what Georg
things.
--
nosy: +georg.brandl
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8004
Jack Diederich jackd...@gmail.com added the comment:
I've been digging into the patch. Is there a reason sendmsg() wants an
iterable of buffers instead of just accepting a str? The list-of-buffers more
closely matches the underlying syscall but I'm not sure what the python benefit
is,
STINNER Victor victor.stin...@haypocalc.com added the comment:
Commited: r78392 (r78394).
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7649
___
Dirkjan Ochtman dirk...@ochtman.nl added the comment:
Sure, I'm happy to fix up the little things. The point is that this works and
solves (IMO) a real problem, so I wonder if we can get this in.
--
___
Python tracker rep...@bugs.python.org
Jack Diederich jackd...@gmail.com added the comment:
one of the other sprinters just pointed out that Modules/_multiprocessing.c
(py3k branch) uses sendmsg/recvmsg internally to pass file descriptors back and
forth. The code is very short and readable.
--
Jeremy Hylton jer...@alum.mit.edu added the comment:
On Sat, Dec 6, 2008 at 1:28 PM, Benjamin Peterson
rep...@bugs.python.org wrote:
Benjamin Peterson musiccomposit...@gmail.com added the comment:
I think I may have been merging add_ast_fields when I wrote the patch.
Here's a new patch
I also notice that the Grammar in the PEP is more complicated:
nonlocal_stmt ::=
nonlocal identifier (, identifier)*
[= (target_list =)+ expression_list]
| nonlocal identifier augop expression_list
The Grammar in the patch is:
+global_stmt: 'global' NAME (',' NAME)* [',']
R. David Murray rdmur...@bitdance.com added the comment:
Georg: could we also add a targets in the Docs Makefile and make.bat to fire up
the script?
--
resolution: - accepted
versions: +Python 3.2
___
Python tracker rep...@bugs.python.org
New submission from Leo Shklovskii l...@thermopylae.net:
I just upgraded to Windows 7, reinstalled all of my environment and am running
into a completely crazy issue.
The repro is: cmd.exe - python.exe - Paste in a string more than 230 chars.
conhost.exe crashes and I get an error that ends
Changes by R. David Murray rdmur...@bitdance.com:
--
priority: - high
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8007
___
___
Python-bugs-list
Brett Cannon br...@python.org added the comment:
Now that Dino has commit privileges and I just gave him the coordinator role,
he can do the commit himself. =) Went ahead and assigned this issue to him.
And Dino, it would be helpful if you changed your username on the tracker to
match your
R. David Murray rdmur...@bitdance.com added the comment:
Committed to py3k in r78400. Had to add the make_request_and_skipIf decorator
to test_xmplrc as well.
--
priority: - normal
status: open - closed
versions: +Python 3.2
___
Python tracker
R. David Murray rdmur...@bitdance.com added the comment:
Well, 'we' and 'they' are pretty much the same team. If you want to enhance
the openid support that Martin added to the tracker, you can submit a patch
against the code in the repository.
--
priority: - normal
R. David Murray rdmur...@bitdance.com added the comment:
Committed to trunk in r78412 and 2.6 in r78413.
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
Brian Curtin cur...@acm.org added the comment:
I'm not able to reproduce this any of the python.org installers, or the current
trunk. I am running 64-bit Windows 7, though, whereas you are running 32-bit. I
don't imagine there should be a difference, but I'll try to obtain a 32-bit box
and
Leo Shklovskii l...@thermopylae.net added the comment:
The only serious tweaking I've done has been to disable cleartype and
completely remove Segoe UI. Adobe Photoshop didn't react nicely to that; it is
possible that there's something in conhost that's hardcoded to Segoe as well.
I'm not
Brian Curtin cur...@acm.org added the comment:
I did both of those things and I'm still not able to reproduce it. Do you have
any specific language settings enabled?
One thing I meant to ask in my last message was if you had a copy of what you
tried to paste into the window. I've pasted a
Leo Shklovskii l...@thermopylae.net added the comment:
Ok, I've traced this down to the pyreadline package. Specifically 1.5 -
http://pypi.python.org/pypi/pyreadline/1.5 - I really appreciate your help
Brian and I'm sorry to have wasted your time. I'm filing a bug with the
pyreadline
Ronald Oussoren ronaldousso...@mac.com added the comment:
I'd expect more failures, the failure is caused by the way the sysconfig module
calculates the root of the installation, and whether or not it is running in
the source tree.
Tarek: could you please review sysconfig.patch?
--
68 matches
Mail list logo