[issue11888] Add C99's log2() function to the math library

Mark Dickinson added the comment: Thanks, Victor. You caught me by surprise a bit: I had some more minor changes to that patch pending, so I've committed those separately. Any news from the buildbots? -- ___ Python tracker

[issue12035] problem with installing validator.nu on windows

New submission from sulakshana : I tried following instructions given in http://about.validator.nu/#src to install validator.nu on windows. 1. changed hg clone https://bitbucket.org/validator/build build to hg.py clone https://bitbucket.org/validator/build build as HG was not getting recognise

[issue12035] problem with installing validator.nu on windows

Amaury Forgeot d'Arc added the comment: This is not a python issue. Please report your problem to the validator project. -- nosy: +amaury.forgeotdarc resolution: -> invalid status: open -> closed ___ Python tracker

[issue11888] Add C99's log2() function to the math library

STINNER Victor added the comment: > Thanks, Victor. You caught me by surprise a bit Oh, I thought that the patch was ready to be commited. > I had some more minor changes to that patch pending, > so I've committed those separately. You should add "Issue #11888: " prefix to your commit messag

[issue11877] Change os.fsync() to support physical backing store syncs

Steffen Daode Nurpmeso added the comment: Ronald Oussoren wrote (2011-05-08 10:33+0200): > Steffen, I don't understand your comment about "auto". Declaring > variables as "auto" is not necessary in C code and not used > anywhere else in Python's source code. Well, as long as i can keep my under

[issue10811] sqlite segfault with generators

Roundup Robot added the comment: New changeset c6523d6faef4 by Gerhard Haering in branch 'default': #10811: Fix recursive usage of cursors. Instead of crashing, raise a ProgrammingError now. http://hg.python.org/cpython/rev/c6523d6faef4 -- nosy: +python-dev ___

[issue11888] Add C99's log2() function to the math library

STINNER Victor added the comment: > we have to wait 12 hours or maybe one day to wait for all buildbots Oh, it's faster than expected: test_math passed on FreeBSD 6.4 3.x buildbot. I was waiting for this one because it's an old OS and many tests fail on this buildbot (because it's old but als

[issue11888] Add C99's log2() function to the math library

Roundup Robot added the comment: New changeset 565f43f6bed4 by Victor Stinner in branch 'default': Issue #11888: Use system log2() when available http://hg.python.org/cpython/rev/565f43f6bed4 -- ___ Python tracker

[issue11888] Add C99's log2() function to the math library

STINNER Victor added the comment: Issue #11888: Use system log2() when available http://hg.python.org/cpython/rev/565f43f6bed4 "I expect the system libc to use more accurate functions than Python." You know what? Mac OS X log2 is less accurate than Python log2! A log2 test failed on "x86 Tige

[issue11888] Add C99's log2() function to the math library

Mark Dickinson added the comment: > You know what? Mac OS X log2 is less accurate than Python log2! That doesn't surprise me much. Though it's probably still true that log2 from OS X is more accurate than our log2 for some other values. It's just that getting the answer wrong for a power of

[issue11888] Add C99's log2() function to the math library

Mark Dickinson added the comment: One other thought: we should check that it's not pow that's at fault here, rather than log2. The test uses math.log2(2.0**n). It would probably be better off using math.log2(ldexp(1.0, n)), or similar: the libm pow operation is also notorious for inaccura

[issue8407] expose signalfd(2) and pthread_sigmask in the signal module

Stefan Krah added the comment: There are warnings on the FreeBSD and OSX buildbots, where pthread_t is not a long. http://www.python.org/dev/buildbot/all/builders/AMD64%20FreeBSD%208.2%203.x/builds/237/steps/compile/logs/warnings%20%283%29 http://www.python.org/dev/buildbot/all/builders/AMD64%

[issue10811] sqlite segfault with generators

Roundup Robot added the comment: New changeset 2fdabf0dc8f7 by Victor Stinner in branch 'default': Issue #10811: Use TestCase.assertRaises() in the new test http://hg.python.org/cpython/rev/2fdabf0dc8f7 -- ___ Python tracker

[issue8407] expose signalfd(2) and pthread_sigmask in the signal module

Roundup Robot added the comment: New changeset 2e0d3092249b by Victor Stinner in branch 'default': Issue #8407: Use an explicit cast for FreeBSD http://hg.python.org/cpython/rev/2e0d3092249b -- ___ Python tracker _

[issue11888] Add C99's log2() function to the math library

STINNER Victor added the comment: > we should check that it's not pow that's at fault here Some tests on Mac OS X Tiger: >>> (2.0 ** -255).hex() '0x1.0p-255' => pow is correct >>> import ctypes; import ctypes.util, math >>> libc = ctypes.cdll.LoadLibrary(ctypes.util.find_library(

[issue11188] test_time error on AIX

Roundup Robot added the comment: New changeset 1f23d63b578c by Mark Dickinson in branch 'default': Issue #11188: In log2 tests, create powers of 2 using ldexp(1, n) instead of the less reliable 2.0**n. http://hg.python.org/cpython/rev/1f23d63b578c -- nosy: +python-dev

[issue11888] Add C99's log2() function to the math library

Mark Dickinson added the comment: Okay, thanks. We should still be using ldexp rather than 2.0**... in the tests, though; I've fixed this, and also fixed the incorrect (too small) range for those tests, so that all representable powers of 2 are now covered. -- _

[issue11888] Add C99's log2() function to the math library

Mark Dickinson added the comment: Grr. Got the issue number wrong in the commit message; see msg135584. New changeset 1f23d63b578c by Mark Dickinson in branch 'default': Issue #11188: In log2 tests, create powers of 2 using ldexp(1, n) instead of the less reliable 2.0**n. http://hg.python.o

[issue11188] test_time error on AIX

Changes by Mark Dickinson : -- Removed message: http://bugs.python.org/msg135584 ___ Python tracker ___ ___ Python-bugs-list mailing l

[issue9584] Allow curly brace expansion

Tim Golden added the comment: I've just rebuilt on Windows against tip. test_glob is failing: test test_glob failed -- Traceback (most recent call last): File "c:\work-in-progress\python\cpython-9584\lib\test\test_glob.py", line 135, in test_glob_curly_braces os.path.join('a', 'bcd', 'ef

[issue9584] Allow curly brace expansion

Ezio Melotti added the comment: +if sub.find(',') != -1: Please use the 'in' operator here. -- nosy: +ezio.melotti ___ Python tracker ___

[issue10666] OS X installer variants have confusing readline differences

Éric Araujo added the comment: No. You wrote this: “Here's a snippet of what I have in my startup file: [code with if 'libedit' in readline.__doc__ complication] While obviously this can be handled, it seems like an unnecessary burden on users.” My message addresses this: using read_init_fi

[issue12013] file /usr/local/lib/python3.1/lib-dynload/_socket.so: symbol inet_aton: referenced symbol not found

Éric Araujo added the comment: It would be helpful if you could attach the test log. Alternatively, if the configure and build log for your Python package is available somewhere on the web, the link would be useful. -- ___ Python tracker

[issue11968] wsgiref's wsgi application sample code does not work

Changes by Éric Araujo : -- nosy: +skrah ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org

[issue12036] ConfigParser: items() adds the vars dictionary to the result

New submission from Isaac Jurado : >From the following python code: import os from ConfigParser import ConfigParser from pprint import pprint c = ConfigParser() c.read(['test.ini']) pprint(c.items('test', raw=False, vars=os.environ)) I see the values contained in os.environ as well as in sectio

[issue10496] "import site failed" when Python can't find home directory (sysconfig._getuserbase)

Éric Araujo added the comment: In the absence of tests or doc update, can you tell in English what the new behavior is? IIUC, when the home dir is not found, all the variables that depend on it would not exist, right? Or would they be set to None? -- ___

[issue9923] mailcap module may not work on non-POSIX platforms if MAILCAPS env variable is set

Éric Araujo added the comment: Okay. Nonetheless, splitting on os.sep doesn’t cost us anything and is IMO a good practice to follow. -- ___ Python tracker ___ _

[issue8498] Cannot use backlog = 0 for sockets

Changes by Daniel Evers : Removed file: http://bugs.python.org/file17065/backlog0.diff ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue8498] Cannot use backlog = 0 for sockets

Changes by Daniel Evers : Removed file: http://bugs.python.org/file17066/backlog0_incl_doc.diff ___ Python tracker ___ ___ Python-bugs-list mai

[issue8498] Cannot use backlog = 0 for sockets

Changes by Daniel Evers : Removed file: http://bugs.python.org/file17067/socket_listen.patch ___ Python tracker ___ ___ Python-bugs-list mailin

[issue8498] Cannot use backlog = 0 for sockets

Changes by Daniel Evers : Removed file: http://bugs.python.org/file21851/server.py ___ Python tracker ___ ___ Python-bugs-list mailing list Uns

[issue8498] Cannot use backlog = 0 for sockets

Changes by Daniel Evers : Removed file: http://bugs.python.org/file21852/client.py ___ Python tracker ___ ___ Python-bugs-list mailing list Uns

[issue8498] Cannot use backlog = 0 for sockets

Changes by Daniel Evers : Removed file: http://bugs.python.org/file21879/backlog0_complete.patch ___ Python tracker ___ ___ Python-bugs-list ma

[issue8498] Cannot use backlog = 0 for sockets

Daniel Evers added the comment: Thanks, I removed the old patches and changed the unit test according to your suggestion. I kept the "usually 5" remark, because I'm not sure how reality really looks like. But feel free to suggest a patch ;) -- Added file: http://bugs.python.org/file21

[issue11743] Rewrite PipeConnection and Connection in pure Python

Roundup Robot added the comment: New changeset 1ac03e071d65 by Antoine Pitrou in branch 'default': Issue #11743: Rewrite multiprocessing connection classes in pure Python. http://hg.python.org/cpython/rev/1ac03e071d65 -- nosy: +python-dev ___ Python

[issue10496] "import site failed" when Python can't find home directory (sysconfig._getuserbase)

STINNER Victor added the comment: Main changes of the patch, if the current user has no home directory (no entry in /etc/passwd) and there is HOME environment variable: - sysconfig.get_config_vars() doesn't have a 'userbase' variable. sysconfig.get_config_var('userbase') returns None as any

[issue11743] Rewrite PipeConnection and Connection in pure Python

Antoine Pitrou added the comment: Now checked in, hopefully there will be no buildbot failures. -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> pending ___ Python tracker ___

[issue12036] ConfigParser: items() adds the vars dictionary to the result

Changes by R. David Murray : -- nosy: +lukasz.langa ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail

[issue3709] BaseHTTPRequestHandler innefficient when sending HTTP header

Roundup Robot added the comment: New changeset 25298224cb25 by Senthil Kumaran in branch 'default': Issue #3709: a flush_headers method to BaseHTTPRequestHandler which manages the http://hg.python.org/cpython/rev/25298224cb25 -- nosy: +python-dev ___

[issue3709] BaseHTTPRequestHandler innefficient when sending HTTP header

Senthil Kumaran added the comment: Added the flush_headers method and the test function. this issue can be closed now. Thanks, Andrew Schaaf. -- status: open -> closed ___ Python tracker __

[issue8305] memoview[0] creates an invalid view if ndim != 1

Changes by Antoine Pitrou : -- nosy: +mark.dickinson versions: +Python 3.3 ___ Python tracker ___ ___ Python-bugs-list mailing list Uns

[issue5231] Change format of a memoryview

Antoine Pitrou added the comment: In the mean time I had to resort to dirty hacks in 1ac03e071d65 (such as using io.BytesIO.write(), which I know is implemented in C and doesn't care about item size). At the minimum, a memoryview.getflatview() function would be nice (and probably easier to c

[issue5231] Change format of a memoryview

Changes by Antoine Pitrou : -- stage: patch review -> needs patch ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue5231] Change format of a memoryview

STINNER Victor added the comment: Read a int32 array as a raw byte string is useful, but the opposite is also useful. -- nosy: +haypo ___ Python tracker ___

[issue3709] BaseHTTPRequestHandler innefficient when sending HTTP header

Changes by Arfrever Frehtes Taifersar Arahesis : -- versions: +Python 3.3 -Python 3.2 ___ Python tracker ___ ___ Python-bugs-list maili

[issue3566] httplib persistent connections violate MUST in RFC2616 sec 8.1.4.

Changes by Alan Kennedy : -- nosy: +amak ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/

[issue11743] Rewrite PipeConnection and Connection in pure Python

Changes by Antoine Pitrou : -- status: pending -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http:/

[issue12015] possible characters in temporary file name is too few

Nick Coghlan added the comment: Python should still work reliably even if the temp directory is on a case-insensitive filesystem. Since that isn't an easy thing to determine, -1 on restoring the uppercase characters to the filenames. As for it not being fixed previously, it's a rare bug that

[issue12015] possible characters in temporary file name is too few

STINNER Victor added the comment: If you would like more entry, use a longer filename: >>> (26+10+1) ** 6 > (26+26+10+1) ** 6 False >>> (26+10+1) ** 7 > (26+26+10+1) ** 6 True -- nosy: +haypo ___ Python tracker _

[issue12015] possible characters in temporary file name is too few

Nick Coghlan added the comment: The number of characters in the filename is hardcoded in the guts of tempfile (in _RandomNameSequence.__next__ to be exact). Increasing that to 7 (or even 8) would be a straightforward change at the library level, but it can't be done from user code. -

[issue12015] possible characters in temporary file name is too few

Charles-François Natali added the comment: Could someone explain me what's the risk on a case-insensitive filesystem? Since files are created with O_CREAT|O_EXCL, I don't see where the problem is. -- nosy: +neologix ___ Python tracker

[issue12037] test_email failures under Windows with the eol extension activated

New submission from Antoine Pitrou : The title says it all. Ideally (!), changing .hgignore should be enough to fix this issue. (NB: I have only tested the default branch) == FAIL: test_byte_message_rfc822_only (test_email.Test

[issue12015] possible characters in temporary file name is too few

Nick Coghlan added the comment: Basically, adding the uppercase letters back in would provide no gain in entropy on a case-insensitive filesystem (as the upper and lower case letters will collide). Increasing the length of the random sequence would provide a consistent gain regardless of file

[issue12038] assertEqual doesn't display newline differences quite well

New submission from Antoine Pitrou : >>> tc.assertEqual("x\n" * 40 + "\n", "x\n" * 40 + "\r\n") Traceback (most recent call last): File "", line 1, in File "/home/antoine/cpython/default/Lib/unittest/case.py", line 662, in assertEqual assertion_func(first, second, msg=msg) File "/home

[issue12015] possible characters in temporary file name is too few

Raymond Hettinger added the comment: +1 for increasing the number of random characters to 8 -1 for restoring uppercase letters -- ___ Python tracker ___

[issue12036] ConfigParser: items() adds the vars dictionary to the result

Łukasz Langa added the comment: This annoyance can't be fixed in 2.x anymore since this is a change of behaviour requiring a full release. While it is very unlikely someone depended on this "feature", changing it breaks compatibility. Reclassified as `Documentation` to explicitly describe tha

[issue12015] possible characters in temporary file name is too few

Nick Coghlan added the comment: Minor clarification: after my first comment, I realised that the original change in the character set didn't actually fix anything, it just propagated the effectively smaller sample space on case-insensitive filesystems to all filesystems. Increasing the number

[issue12036] ConfigParser: Document items() added the vars dictionary to the result

Changes by Łukasz Langa : -- title: ConfigParser: items() adds the vars dictionary to the result -> ConfigParser: Document items() added the vars dictionary to the result ___ Python tracker ___

[issue12015] possible characters in temporary file name is too few

Charles-François Natali added the comment: @Nick I fully agree with you, but my point was that it doesn't make it less safe on case-insensitive filesystems. Apart from that, it's of course better to increase the length of the random sequence. -- __

[issue12036] ConfigParser: Document items() added the vars dictionary to the result

Roundup Robot added the comment: New changeset 62e2b74c3bfc by Łukasz Langa in branch '3.2': Closes #12036: ConfigParser: Document items() added the vars dictionary to the result http://hg.python.org/cpython/rev/62e2b74c3bfc New changeset 2d984cc4afb0 by Łukasz Langa in branch 'default': Merge

[issue10666] OS X installer variants have confusing readline differences

Ned Deily added the comment: Sorry, I don't see how that could help. The point I was making is that the directives accepted by GNU readline and BSD editline are completely different and one way or another the user is forced to deal with that. If you used read_init_file, you would still need

[issue12015] possible characters in temporary file name is too few

Nick Coghlan added the comment: Oh, you mean the security risk if the temporary names can be guessed? My recollection is that it is more of a problem for temporary directories than it is for temporary files, since the module can't control how files inside a temp directory get created. It's be

[issue10666] OS X installer variants have confusing readline differences

Éric Araujo added the comment: Ah, I thought that read_init_file used a different filename with libedit. Would there be a downside in implementing your suggestion labeled trivial, i.e. always use readline? -- ___ Python tracker

[issue5114] 2.7: test_threading hangs on Solaris

Charles-François Natali added the comment: Hmm. I think this was probably fixed by Gregory in issue #6643 (it's not in Python 2.7.1). Could you try with Python 3.2, or a current snapshot? -- nosy: +neologix ___ Python tracker

[issue10666] OS X installer variants have confusing readline differences

Ned Deily added the comment: Even if used a different file name, you still have to set up two different sets of directives. The main drawback to the trivial suggestion is that it continues to pull in GNU readline, which is now GPLv3-licensed, into the python.org OS X installers. In general,

[issue10666] OS X installer variants have confusing readline differences

Éric Araujo added the comment: > Even if used a different file name, you still have to set up two > different sets of directives. I was assuming this would not be a problem: the .inputrc (or hypothetical .editrc file) is written once for all applications, it’s not a Python-specific task. Wha

[issue11948] Tutorial/Modules - small fix to better clarify the modules search path

Sandro Tosi added the comment: What about the attached (v3) patch? the "This allows...experts hands." part can be remove completely if can be mis-interpreted. -- Added file: http://bugs.python.org/file21944/issue11948-v3.patch ___ Python tracker

[issue10666] OS X installer variants have confusing readline differences

Ned Deily added the comment: ́Éric, Martin was involved in the earlier discussions when the support for editline was originally added to the readline module. We've been over this ground before. See Issue6872 and Issue6877. There are a number of options here. I plan to investigate the suppl

[issue12039] test_logging: bad file descriptor on FreeBSD bot

New submission from Stefan Krah : The FreeBSD-AMD64 bot shows an error in test_logging: http://www.python.org/dev/buildbot/all/builders/AMD64%20FreeBSD%208.2%203.x/builds/243/steps/test/logs/stdio Exception in thread Thread-281: Traceback (most recent call last): File "/usr/home/buildbot/buil

[issue12039] test_logging: bad file descriptor on FreeBSD bot

Changes by Stefan Krah : -- keywords: +buildbot ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyt

[issue12040] Expose a Process.sentinel property (and fix polling loop in Process.join())

New submission from Antoine Pitrou : This patch exposes a new read-only property of multiprocessing.Process objects, named "sentinel". As the doc indicates, this is a file descriptor undex Unix, and a handle under Windows. Both are suitable for flexible polling/waiting with the appropriate OS

[issue9205] Parent process hanging in multiprocessing if children terminate unexpectedly

Antoine Pitrou added the comment: Part of the patch submitted standalone in issue12040. -- dependencies: +Expose a Process.sentinel property (and fix polling loop in Process.join()) ___ Python tracker

[issue12041] test_os test_ctypes test_wait3 causes test_wait3 error

New submission from R. David Murray : It doesn't matter which order test_os and test_ctypes run in, but if they precede test_wait3 test_wait3 will fail. When the test is immediately re-run it passes. The reason ought to be interesting once we figure it out :) Tested on linux. Doesn't appear

[issue12041] test_os test_ctypes test_wait3 causes test_wait3 error

Changes by Ezio Melotti : -- nosy: +ezio.melotti ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue12042] What's New multiprocessing example error

New submission from Davi Post : In this section: http://docs.python.org/whatsnew/2.6.html#pep-371-the-multiprocessing-package This sentence follows the first example: "A Queue is used to communicate the input parameter N and the result." In the example, the queue is used only for the result, no

[issue11948] Tutorial/Modules - small fix to better clarify the modules search path

Raymond Hettinger added the comment: Please don't make these kind of changes. Guido has been clear that the docs need to affirmatively state what the language does. Except for certain security risks or segfault risks, the docs should avoid wording along the lines of "feature x is dangerous

[issue12043] Update shutil documentation

New submission from Sandro Tosi : This issue is to keep track of the 'shutil' documentation update, since it seems to miss several information (in particular in comparison with the docstrings). I'll post patches as soon as I get them. -- assignee: docs@python components: Documentation

[issue11948] Tutorial/Modules - small fix to better clarify the modules search path

Sandro Tosi added the comment: Sorry Raymond for the bad editing, I was confused by thinking 'tutorial' is particularly meant for new-coming people. Anyhow, sorry again: I attach a patch removing the badworded part. Please let me know if anything else is missing or needs fixing. Thanks for y

[issue11948] Tutorial/Modules - small fix to better clarify the modules search path

R. David Murray added the comment: Raymond: I haven't read the examples in detail so I don't have an opinion on the appropriateness of the text, but I'm curious: while what you say certainly applies to the Language Reference, does it also apply to the Tutorial? -- nosy: +r.david.murra

[issue12041] test_os test_ctypes test_wait3 causes test_wait3 error

Antoine Pitrou added the comment: What seems to happen is that test_ctypes runs a lot of ldconfig processes, and the 10 retries in fork_wait are not enough to consume them all. -- nosy: +pitrou ___ Python tracker

[issue11948] Tutorial/Modules - small fix to better clarify the modules search path

Raymond Hettinger added the comment: The tutorial is meant for newcomers. The experience needs to be positive and not leave the reader with worries that something bad will happen if they make a misstep. People commonly use the tutorial to evaluate the language as a whole. The tutorial need

[issue12041] test_os test_ctypes test_wait3 causes test_wait3 error

Changes by Nadeem Vawda : -- nosy: +nadeem.vawda ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue12041] test_os test_ctypes test_wait3 causes test_wait3 error

Roundup Robot added the comment: New changeset 9e473917cbfb by Antoine Pitrou in branch 'default': Issue #12041: Make test_wait3 more robust. http://hg.python.org/cpython/rev/9e473917cbfb -- nosy: +python-dev ___ Python tracker

[issue11948] Tutorial/Modules - small fix to better clarify the modules search path

Raymond Hettinger added the comment: One other stylistic note. The tone of the tutorial (and all the docs) needs to be respectful of the readers intelligence. We don't presume that the readers are stupid. We lay out the relevant information, show motivating use cases, provide glossary link

[issue12044] subprocess.Popen.__exit__ doesn't wait for process end

New submission from Antoine Pitrou : I find it a bit strange that Popen.__exit__ closes all standard file descriptors leading to the child process, but doesn't wait for process end afterwards. (context management support was added in issue10554) -- messages: 135635 nosy: brian.curtin,

[issue12041] test_os test_ctypes test_wait3 causes test_wait3 error

Antoine Pitrou added the comment: This should be fixed, can you confirm? PS: I've opened issue12044 as one of the root causes. -- resolution: -> fixed stage: needs patch -> committed/rejected status: open -> pending ___ Python tracker

[issue12013] file /usr/local/lib/python3.1/lib-dynload/_socket.so: symbol inet_aton: referenced symbol not found

Alex Lai added the comment: I've installed it successfully on another server. That is enough. I haven't got time to upgrade my workstation from s10 pre-release to latest s10 release yet. Thanks for the information, Alex -- ___ Python tracker

[issue12041] test_os test_ctypes test_wait3 causes test_wait3 error

Ezio Melotti added the comment: Works for me after the commit. -- resolution: fixed -> stage: committed/rejected -> needs patch status: pending -> open ___ Python tracker ___ _

[issue12041] test_os test_ctypes test_wait3 causes test_wait3 error

Changes by Ezio Melotti : -- resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed ___ Python tracker ___

[issue12039] test_logging: bad file descriptor on FreeBSD bot

Vinay Sajip added the comment: I've made recent changes in test_logging to improve coverage, but these errors were not being raised on my local development machine (Ubuntu Jaunty). They have recently started occurring on my machine, too, so at the moment I'm suspecting some other change which

[issue12039] test_logging: bad file descriptor on FreeBSD bot

Vinay Sajip added the comment: I've attached a smaller script which shows up the error. It's not tiny, but tests the SMTP and HTTP handler tests which show the failures mentioned in Stefan's post. -- Added file: http://bugs.python.org/file21947/server_test.py

[issue12044] subprocess.Popen.__exit__ doesn't wait for process end

Brian Curtin added the comment: Seems like it would be enough to add a wait() at the end? diff -r 9e473917cbfb Lib/subprocess.py --- a/Lib/subprocess.py Mon May 09 21:17:02 2011 +0200 +++ b/Lib/subprocess.py Mon May 09 15:30:02 2011 -0500 @@ -796,6 +796,7 @@ self.stderr.close()

[issue12044] subprocess.Popen.__exit__ doesn't wait for process end

Antoine Pitrou added the comment: > Seems like it would be enough to add a wait() at the end? Probably, perhaps with a try/finally? I'm not entirely sure this is a good idea, by the way. May there be some drawbacks? -- ___ Python tracker

[issue10666] OS X installer variants have confusing readline differences

Martin v. Löwis added the comment: It's true that we should try to avoid distributing GPL'ed code with binary installers. At a minimum, we would have to provide the source of readline for download also, and we would have to warn anybody redistributing our binaries that they must include the s

[issue12015] possible characters in temporary file name is too few

STINNER Victor added the comment: If you care about security, you should not use the Python random module because it is not cryptographic. Oh oh, ssl doesn't expose RAND_bytes(). -- ___ Python tracker ___

[issue12039] test_logging: bad file descriptor on FreeBSD bot

Vinay Sajip added the comment: The rest isn't actually failing - it's just that a spurious BadStatusLine exception appears to be being raised. -- ___ Python tracker ___ ___

[issue12015] possible characters in temporary file name is too few

Antoine Pitrou added the comment: > If you care about security, you should not use the Python random > module because it is not cryptographic. Oh oh, ssl doesn't expose > RAND_bytes(). Feel free to open a separate issue and even provide a patch! --

[issue12039] test_logging: bad file descriptor on FreeBSD bot

Vinay Sajip added the comment: The change which causes the apparently spurious messages is one made by Senthil: 25298224cb25 - with the earlier revision 1ac03e071d65 the messages don't occur, as you can see here: vinay@eta-jaunty:~/projects/python/default$ hg update -r 69980 merging Lib/test/

[issue12044] subprocess.Popen.__exit__ doesn't wait for process end

Brian Curtin added the comment: Actually, I don't think the wait() is a good idea. If you want to block and infinitely wait on the process to close, you should do so explicitly. It's probably better that we try to use terminate() or kill() and raise if that fails. -- ___

[issue12044] subprocess.Popen.__exit__ doesn't wait for process end

Antoine Pitrou added the comment: > Actually, I don't think the wait() is a good idea. If you want to > block and infinitely wait on the process to close, you should do so > explicitly. Ok. > It's probably better that we try to use terminate() or kill() and raise if > that fails. Uh, I think

  1   2   >