Jeremy Kloth added the comment:
Not to sound needy, but could the patch be looked into being integrated soon?
This problem had only occurred once or twice a month however it has caused
failures three times just in the last week.
--
___
Python
Jeremy Kloth added the comment:
Thank you! There are no failures due to the patch and now its just a wait and
see if test_winreg will misbehave again.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17065
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17480
___
___
Python
Jeremy Kloth added the comment:
It hangs on both my Windows buildbot and development machine.
The patch is tested and works correctly on Windows.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17077
Jeremy Kloth added the comment:
I would suggest that this be bumped to release blocker as currently passing
bytes to communicate with universal_newlines=True on Windows causes the process
to hang (see issue17077).
The backported test case passes on Windows without any changes to subprocess.py
New submission from Jeremy Kloth:
Attached is a patch to test_tools that gets it back to a run-able state.
--
components: Tests
files: test_tools.diff
keywords: patch
messages: 180925
nosy: jkloth, serhiy.storchaka
priority: normal
severity: normal
status: open
title: Fix test_tools
New submission from Jeremy Kloth:
Attached is an attempt at fixing the sporadic failures of test_mailbox on the
AMD64 Windows buildbot.
It fails due to access errors on some directories which leads me to believe the
helper functions in test.support should fix the problem
New submission from Jeremy Kloth:
test_winreg fails sporadically on the AMD64 Windows buildbot. Looking at the
test, it appears that concurrent runs of the test would fail if different
processes attempted to modify the test key at the same time.
The attached patch resolves this by using
Jeremy Kloth added the comment:
Actually, the support functions (as of 3.3) attempt to work around the access
errors. They attempt to wait (to a point) for a successful operation before
returning to the caller. See issue15496 for details.
It is usually the case that the previous operation
New submission from Jeremy Kloth:
The 64-bit linker doesn't mangle the dllexport'ed module init function (from
PyMODINIT_FUNC) so it causes an exported name conflict. We cannot just remove
that name from export_symbols as the module may not have used PyMODINIT_FUNC on
its init function
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16769
___
___
Python
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16668
___
___
Python
Jeremy Kloth added the comment:
The problem exhibited is not coming from the os.walk() implementation, but from
the use of a byte-string as the argument to it.
The directories are created with unicode literals and therefore the argument
must also be a unicode literal (u'.') for them
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16612
___
___
Python
Jeremy Kloth added the comment:
On Thu, Nov 22, 2012 at 3:11 PM, Antoine Pitrou rep...@bugs.python.org wrote:
The AMD64 Windows 7 buildbot shows weird build failures in ctypes:
http://buildbot.python.org/all/buildslaves/kloth-win64
The _ctypes_d.pyd was considered to be in use by the system
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
keywords: +patch
Added file: http://bugs.python.org/file27660/029d1cdf6422.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15968
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +brian.curtin, loewis, tim.golden
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15968
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16218
___
___
Python
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16188
___
___
Python
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16175
___
___
Python
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16131
___
___
Python
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16047
___
___
Python
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16049
___
___
Python
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15963
___
___
Python
New submission from Jeremy Kloth:
This patch incorporates Tcl/Tk/Tix into the MSVC build in the same fashion as
OpenSSL has been done.
Highlights:
- A new project, tcltk, is added that simply calls the Python script
build_tkinter.py to build the externals.
- New helper module PCbuild
Jeremy Kloth added the comment:
This test is still intermittently failing on the AMD64 Windows7 SP1 buildbot:
http://buildbot.python.org/all/builders/AMD64%20Windows7%20SP1%203.x/builds/630
Any chance the patch could be committed?
--
___
Python
New submission from Jeremy Kloth:
The attached patch fixes the compiler options used for building the x64
platform.
- In VC10 the passing /GS- as an additional option has been replaced by the
MSBuild BufferSecurityCheck command
- The /USECL:xxx option is only for the VSExtComp plugin
Jeremy Kloth added the comment:
I did forgot t mention that this change also silences a compiler warning:
cl : Command line warning D9025: overriding '/GS' with '/GS-'
The /USECL:xxx option does not provoke a warning is it is being
evaluated as '/U SECL:xxx' (undefine macro)
--
nosy
Jeremy Kloth added the comment:
Here is the patch implementing option #2
--
keywords: +patch
nosy: +jkloth
Added file: http://bugs.python.org/file26947/test_startfile.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15526
Jeremy Kloth added the comment:
I think the reason that it is only this buildbot which fails is that the
other Windows buildbots don't use multiple processes. Therefore they don't
use a different dir for each test.
That might be it. Also the failure possibly only happens when
multiple
Jeremy Kloth added the comment:
However #1 is the reason that is bug exists in the first place. The
designer of the test guessed wrong on the magic value for the
timeout. There will never be a correct timeout value as it varies
from machine to machine and from workload to workload on a given
Jeremy Kloth added the comment:
Unfortunately, this is a legitimate failure of the test. The test
(actually the support code) is attempting to remove a directory that
is the current directory of an active process. The test has
documented this issue and attempted to work around it by adding
New submission from Jeremy Kloth:
os.stat fails when called on a file that is pending delete but is still in the
directory listing.
This in turn causes os.path.exists to return the wrong result.
Attached is a test case demonstrating this broken behavior.
--
components: Library (Lib
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +brian.curtin, loewis, pitrou, tim.golden -jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15556
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15556
___
___
Python
Jeremy Kloth added the comment:
Traceback (most recent call last):
File stat-bug.py, line 12, in module
print('stat', os.stat(pathname))
PermissionError: [Error 5] Access is denied: '\\Users\\Jeremy\\test.tmp'
--
___
Python tracker rep
Jeremy Kloth added the comment:
Why do you think the behavior is broken? It looks right to me - it's not
possible to get file information for a file that is scheduled for deletion.
However you can when using MSVCRT's stat() function or even
FindFirstFile directly.
--
nosy
Jeremy Kloth added the comment:
I've updated the comment in the patch to reflect Martin's concern.
Martin is partially correct in that the handle opened in the stat() call will
not prolong the pending status. It is due to the fact that it does not open
the handle with any sharing mode set
Jeremy Kloth added the comment:
With the latest changes, is there anything left preventing the
inclusion of this patch?
Without some change, the Win64 buildbot is relatively irrelevant as it
is nearly always in a state of failure due to these errors
Jeremy Kloth added the comment:
Not that some change isn't necessary, but what else are you running on your
build slave? I ran a Windows 2008 R2 x64 slave for some time and it never had
issues around file/directory removal. I only had to decommission it because
the physical machine became
Jeremy Kloth added the comment:
What are those programs exactly? A buildbot should ideally have a lean system
install that does not interfere with the tests running.
My development machine has add'l programs, not the buildbot machine.
Sorry is there was any confusion. I get the same
Jeremy Kloth added the comment:
Tim Golden added the comment:
I'm tentative about using ctypes this just because I don't believe we use it
anywhere else.
ctypes is already used later in test_support so I figured it was fine
to use for this as well
Jeremy Kloth added the comment:
I wonder why it couldn't use os.listdir to find out whether the directory is
empty, and os.stat to find out whether a specific file or directory still
exists.
It is possible to do the same thing with just os.listdir. The use of
the Find*File functions
Jeremy Kloth added the comment:
OK, here is another patch that uses just os.listdir
--
Added file: http://bugs.python.org/file26663/support.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15496
Jeremy Kloth added the comment:
I must also add that the proposed solution works well within the test suite as
the access denied error can also occur when creating subsequent files, not just
removing them.
This solution eliminates the need to wrap all creation calls with access denied
Jeremy Kloth added the comment:
Well, test_startfile seems to be a common issue on that buildbot.
I wouldn't really call it common (twice in the last 30 runs).
However I would wager that the PermissionError is being caused by an
unknown (without having Process Monitor running when the error
Jeremy Kloth added the comment:
I have just completed an upgrade of the Win64 buildbot slave with a
faster hard drive and the issue is now much more pronounced.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15526
Jeremy Kloth added the comment:
This is a (near) duplicate of issue7443, I think.
Partially so it seems. However, my testing with Process Monitor (from
sysinterals) shows that most of the access denied errors occur when
removing directories.
The blind rename-remove dance doesn't work when
Jeremy Kloth added the comment:
I'm getting more than thousand warnings:
$ CFLAGS=-Wconversion -Wno-sign-conversion ./configure
$ LC_ALL=C make -s -j8 21 | tee log
$ grep Wconversion log | wc -l
1163
My Win64 buildbot currentlyhas 544 conversion warnings, but it seems
something has
New submission from Jeremy Kloth:
It would seem that enabling GCC's type conversion/truncation warnings would be
a good thing for Python builds.
This would bring GCC builds in line with MSVC builds and reduce the burden on
the Windows developers in fixing them. Also, it would bring attention
New submission from Jeremy Kloth:
Currently, removing directories during testing on Windows w/NTFS can causing
sporadic failures due to access denied errors. This is caused by other
processes getting a handle to the directory itself (change notifications) or a
handle to a file within
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15431
___
___
Python
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
The buildbot service account is a standard user (per the buildbot
servce installation directions). When logged on, the user can access
the share. Just when logged on as a service is when it cannot.
After much searching, I still cannot
New submission from Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
The registry key HKEY_PERFORMANCE_DATA is not accessible for non-interactive
users (e.g., buildbot as a service).
The following patches skip the offending test when the tests unless the they
are run from an interactive
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
Added file: http://bugs.python.org/file26369/winreg-3.x.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15334
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
I've changed the malware scanner to disable real-time protection and
have disabled scanning of the buildbot directories.
Note that I could reproduce intermittent failures when using '-j' of
regrtest, but with these changes they no longer
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
I've now also turned off indexing for the Buildbot account (and
folders). Hopefully this is the last directory walking process
interfering with the tests.
--
___
Python tracker rep
Jeremy Kloth jeremy.kloth+python-trac...@gmail.com added the comment:
The OpenSSL (64-bit build) does not pass its own tests, while the 32-bit build
from the same source works just fine.
Stefan, how did you get your 64-bit build to work
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
I've narrowed this down to a CPU capabilities issue with OpenSSL.
What CPU are you using? I have a Q9450 (Core2 Quad)
If I set OPENSSL_ia32cap to 0 the self tests work. I'm now off to
debug OpenSSL CPUID support
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
Ok, the final verdict on this issue is that nasm is to blame. I had
nasm 2.09.02 installed. I upgraded it to version 2.10.01 and
magically OpenSSL started working.
From the nasm changelog is seems that the added instruction sets in
2.10
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
Hmm, obviously Perl *is* found. That leaves nasm.
nasm.exe was installed for myself (hence testing worked). I've now
made it available to the buildbot account as well.
Note that I have Perl installed but *not* on my PATH (on purpose
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
The patch, as it stands, is clearly incorrect (as has been reviewed and
acknowledged).
The most recent patch is correct.
As for the changes to project files and the solution file: none of these
changes should be necessary at all
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
As for the changes to project files and the solution file: none of these
changes should be necessary at all, unless it's actually the case that the
AMD64-Debug build doesn't work (i.e. neither in the buildbot, nor from the
solution
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
I can believe that there are issues with the batch files. That's why I propose
to look at one issue at a time (but it's not my priority at the moment).
That would mean that you do not care that the buildbots are failing.
That is exactly
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
Added file: http://bugs.python.org/file26095/5924b376d15f.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15102
Jeremy Kloth jeremy.kloth+python-trac...@gmail.com added the comment:
Now that my buildbot is up and building (and failing at that) could these
changes be committed? It would get the buildbot to at least start testing
changes to Python proper
New submission from Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
The buildbot scripts do not work for the 64-bit targets.
Firstly, the /p:UseEnv=True parameter to msbuild causes the 32-bit only
projects (make_buildinfo and make_versioninfo) to fail due to architecture
mismatch
Jeremy Kloth jeremy.kloth+python-trac...@gmail.com added the comment:
In addition to the fixes from issue15102, the only way I could get the ssl
project to build successfully was to add the Perl installation (C:\Perl\bin) to
my PATH.
--
___
Python
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
Should I then open another issue just to track that bug? Have you
even tried using build_ssl.py *without* Perl? The changes required to
get that to work seem fairly extensive.
--
___
Python
Jeremy Kloth jeremy.kloth+python-trac...@gmail.com added the comment:
OK, I have discovered my issue(s) building OpenSSL. When I first downloaded
the openssl-1.0.1c external, the timestamps for the .asm files ended up being
older than their corresponding .pl sources therefore triggering
Jeremy Kloth jeremy.kloth+python-trac...@gmail.com added the comment:
I forgot to add that with the patch the comment wrt Perl is truly correct.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15052
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
The unrelated changes do seem to be an issue with the diff generated by Rietveld
--
nosy: +jeremy.kloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15102
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
On Mon, Jun 18, 2012 at 3:27 PM, Antoine Pitrou rep...@bugs.python.org wrote:
If I understand, the proposed changes are the following:
https://bitbucket.org/jkloth/cpython-buildbot-amd64/changeset/2a20cee18add
That seems to be correct
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
I would suggest trying to build OpenSSL (build_ssl.py) *without* Perl
installed. On Windows (7,64-bit at least) a dialog box pops up many
times asking which program to use to run XXX.pl.
This is using the externals from svn.python.org
New submission from Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
The comment regarding a Perl installation not being required is no longer true
with regards to OpenSSL 1.0+ (at least 1.0.0j and 1.0.1c). A Perl script(s) is
used to generate source files within the generated Makefiles
Jeremy Kloth jeremy.kloth+python-trac...@gmail.com added the comment:
Please read the link which you posted. Quoting the second paragraph, second
sentence:
Default namespace declarations do not apply directly to attribute names;
and from the third paragraph, third sentence:
The namespace
Jeremy Kloth jeremy.kloth+python-trac...@gmail.com added the comment:
The attached patch seems to work as-is. That is, just testing for `self.path`
as the prefix. On Windows, at least, the paths in RECORD are always absolute.
Further changes will be necessary, of course, once changes
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12895
___
___
Python
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11805
___
___
Python
Jeremy Kloth jeremy.kloth+python-trac...@gmail.com added the comment:
The failure was due to the sdist command having different default formats for
each platform (i.e., POSIX: gztar, Win32: zip). Patch attached.
--
keywords: +patch
nosy: +jkloth
Added file: http://bugs.python.org
Changes by Jeremy Kloth jeremy.kl...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12729
___
___
Python-bugs-list
Changes by Jeremy Kloth jeremy.kl...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11921
___
___
Python-bugs-list
Changes by Jeremy Kloth jeremy.kl...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10217
___
___
Python-bugs-list
Changes by Jeremy Kloth jeremy.kl...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10175
___
___
Python-bugs-list
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
A quick look with Dependency Walker gives me the following:
Python 2.4,2.5 -- MSVC .NET 2003 (7.1)
Python 2.6,2.7,3.0,3.1 -- MSVC 2008 (9.0)
Note these are only for the official python.org builds. Each version can also
be built using
Changes by Jeremy Kloth jeremy.kl...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6792
___
___
Python-bugs-list
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
Note that this behavior is only present when running IDLE. Python command-line
does not show this oddity.
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
for #1 instead of importing `build`/`install` commands for their options, why
not define the options only on the `configure` command? The B/I commands could
just lookup their needed options from the configure command.
On that note, how
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
How would the build or install command get the options in this case:
python setup.py configure --prefix=$HOME build install
Remember that commands can be chained (and that commands can run other
commands). I would like to be able to do
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
WRT #5, it would be possible to use section headers within the configure.cache
file to indicate a particular platform build. Very much like how 4Suite's
`config` command stored the cache file
Changes by Jeremy Kloth jeremy.kl...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8254
___
___
Python-bugs-list
Jeremy Kloth jeremy.kl...@gmail.com added the comment:
I have developed a patch that adds the ability to disable SSLv2, SSlv3 and
TLSv1 when using the SSLv23 method. It changes Modules/_ssl.c, Lib/ssl.py and
Doc/library/ssl.rst.
--
keywords: +patch
nosy: +jeremy.kloth
Added file: http
301 - 392 of 392 matches
Mail list logo