Change by Segev Finer :
--
nosy: +Segev Finer
___
Python tracker
<https://bugs.python.org/issue28686>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Segev Finer :
--
nosy: +Segev Finer
___
Python tracker
<https://bugs.python.org/issue39010>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2368
___
Python tracker
<http://bugs.python.org/issue23451>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Segev Finer:
We are getting:
Warning C4005 'HAVE_MEMMOVE': macro redefinition _elementtree
c:\users\segev\prj\python\cpython\modules\expat\winconfig.h 34
Warning C4267 '=': conversion from 'size_t' to 'uns
Changes by Segev Finer :
--
pull_requests: +2369
___
Python tracker
<http://bugs.python.org/issue30726>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
type: -> compile error
___
Python tracker
<http://bugs.python.org/issue30726>
___
___
Python-bugs-list mailing list
Unsubscrib
Changes by Segev Finer :
--
pull_requests: +2377
___
Python tracker
<http://bugs.python.org/issue30731>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Segev Finer:
..\Modules\_winapi.c(886): warning C4090: 'function': different 'const'
qualifiers
..\Modules\_winapi.c(848): warning C4101: 'wenvironment_size': unreferenced
local variable
Added in d174d24a5d37d1516b885dc7c82f71ecd5930700
Changes by Segev Finer :
--
pull_requests: +2422
___
Python tracker
<http://bugs.python.org/issue30726>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
It seems that the issue repeats in pyexpat.vcxproj. I submitted a PR for that
one as well.
--
___
Python tracker
<http://bugs.python.org/issue30
Changes by Segev Finer :
--
versions: +Python 3.5, Python 3.6
___
Python tracker
<http://bugs.python.org/issue30726>
___
___
Python-bugs-list mailing list
Unsub
Segev Finer added the comment:
I tried the PR locally and I didn't get the warnings. :)
--
___
Python tracker
<http://bugs.python.org/issue30745>
___
___
New submission from Segev Finer:
This is caused by _GNU_SOURCE being defined here:
https://github.com/python/cpython/blob/c08177a1ccad2ed0d50898c2731b518c631aed14/configure.ac#L124,
But also getting defined by AC_USE_SYSTEM_EXTENSIONS
(https://github.com/python/cpython/blob
Segev Finer added the comment:
All that remains is back porting https://github.com/python/cpython/pull/2375
and than this issue is fixed.
--
___
Python tracker
<http://bugs.python.org/issue30
Changes by Segev Finer :
--
pull_requests: +2551
___
Python tracker
<http://bugs.python.org/issue9566>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
I submitted a PR which fixes some more warnings. The ones I haven't fixed are
detailed in the PR description.
--
nosy: +Segev Finer
versions: +Python 3.7
___
Python tracker
<http://bugs.python.org/i
Changes by Segev Finer :
--
pull_requests: +2556
___
Python tracker
<http://bugs.python.org/issue9566>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2641
___
Python tracker
<http://bugs.python.org/issue30726>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2642
___
Python tracker
<http://bugs.python.org/issue30726>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
Python 2.7 diverged
https://github.com/python/cpython/commit/ab3b0ade505ce07a3d5ec4fbc991a154242732e6.
It's only missing _CRT_SECURE_NO_WARNINGS. :P
--
___
Python tracker
<http://bugs.python.org/is
Changes by Segev Finer :
--
pull_requests: +2643
___
Python tracker
<http://bugs.python.org/issue30726>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
> The approach used to suppress this warning will also suppress other warnings
> that might be fixable. Hard to say it's definitely a good thing in this case,
> though in general I agree that selectively suppressing warnings that cannot
> b
Changes by Segev Finer :
--
pull_requests: +2736
___
Python tracker
<http://bugs.python.org/issue30797>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2739
___
Python tracker
<http://bugs.python.org/issue30797>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2740
___
Python tracker
<http://bugs.python.org/issue30797>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
The fix is now in 3.5-3.7 and 2.7, unless you intend to also back port to 3.4
and 3.3, this can be closed.
--
___
Python tracker
<http://bugs.python.org/issue30
New submission from Segev Finer:
Shows in buildbots:
/home/dje/cpython-buildarea/3.x.edelsohn-sles-z/build/Modules/_json.c: In
function ‘scanner_new’:
/home/dje/cpython-buildarea/3.x.edelsohn-sles-z/build/Modules/_json.c:1212:5:
warning: comparison is always false due to limited range of data
Changes by Segev Finer :
--
pull_requests: +2749
___
Python tracker
<http://bugs.python.org/issue30911>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
Introduced in
https://github.com/python/cpython/commit/ac5bbd43bc7b769c13ae0412cb28a3521f4d4ff1
which is only in 3.7 that means this wasn't ever released. I can send a PR
with a NEWS entry if it still need
Segev Finer added the comment:
Serhiy's change added tests to test_json, so you can say this unearthed ref
leaks rather than introduced them. Probably needs a separate bpo issue and fix.
--
___
Python tracker
<http://bugs.python.org/is
New submission from Segev Finer:
>From builtbots:
test_json leaked [4, 4, 4] references, sum=12
test_json leaked [3, 3, 3] memory blocks, sum=9
1 test failed again:
test_json
Probably unearthed by:
https://github.com/python/cpython/pull/2692
https://github.com/python/cpython/pull/2
Segev Finer added the comment:
Created http://bugs.python.org/issue30936
--
___
Python tracker
<http://bugs.python.org/issue30911>
___
___
Python-bugs-list mailin
Segev Finer added the comment:
I think you will need to save and restore _filters_version and _onceregistry.
Since _filters_version is a static variable in _warnings.c you will probably
also have to modify it somehow to make it accessible to warnings.py.
--
components: +Library (Lib
Segev Finer added the comment:
Try with:
.\python.bat Tools\msi\make_zip.py -a amd64 -o py37-amd64.zip
It requires amd64 instead of x64. Probably can add a choices
(https://docs.python.org/3/library/argparse.html#choices) argument there to
make it more robust.
--
nosy: +Segev
New submission from Segev Finer:
Add an argument --include-py to Tools/msi/make_zip.py that allows including the
Python modules sources and not only bytecode.
I found it useful while running Twisted's test suite in PR 2318, since I got an
error otherwise. I don't remember what that
Changes by Segev Finer :
--
pull_requests: +2857
___
Python tracker
<http://bugs.python.org/issue30986>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2859
___
Python tracker
<http://bugs.python.org/issue30222>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
We are getting those warnings because distutils builds extension modules
without /GL (https://msdn.microsoft.com/en-us/library/0zza0de8.aspx) but with
/LTCG (https://msdn.microsoft.com/en-us/library/xbf3tbeh.aspx). We should
either add /GL to truly enable whole
Segev Finer added the comment:
Investigated some more. We are adding /GL for release but not in debug. But
adding /LTCG for both debug and release.
--
___
Python tracker
<http://bugs.python.org/issue27
Changes by Segev Finer :
--
pull_requests: +2866
___
Python tracker
<http://bugs.python.org/issue27790>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
I used the full one. I doubt virtualenv (used by tox) will work with the
embedding layout :P.
I remember getting an exception from something because it was missing the .py
source files but I don't remember what it was and wasn't able to reprodu
Segev Finer added the comment:
I did make a PR which only changes -a/--arch to a choices argument.
https://github.com/python/cpython/pull/2808
I guess the PR you are talking about is
https://github.com/python/cpython/pull/2750/files which changes this entirely
Changes by Segev Finer :
--
pull_requests: +2894
___
Python tracker
<http://bugs.python.org/issue30916>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Segev Finer:
..\Modules\_tracemalloc.c(88): warning C4359: '': Alignment
specifier is less than actual alignment (8), and will be ignored.
For this to actually work we simply need to use #pragma pack instead.
--
components: Extension Modules, Window
Changes by Segev Finer :
--
pull_requests: +2895
___
Python tracker
<http://bugs.python.org/issue31018>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2896
___
Python tracker
<http://bugs.python.org/issue31018>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: -2896
___
Python tracker
<http://bugs.python.org/issue31018>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2897
___
Python tracker
<http://bugs.python.org/issue31018>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2902
___
Python tracker
<http://bugs.python.org/issue9566>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2920
___
Python tracker
<http://bugs.python.org/issue31018>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2919
___
Python tracker
<http://bugs.python.org/issue31018>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2942
___
Python tracker
<http://bugs.python.org/issue9566>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Segev Finer:
>python -X tracemalloc=30 -m unittest
>test.test_asyncio.test_events.ProactorEventLoopTests.test_create_server_ssl_verify_failed
Running Debug|x64 interpreter...
lib\asyncio\sslproto.py:330: ResourceWarning: unclosed transport
source=self)
Object all
Changes by Segev Finer :
--
pull_requests: +2953
___
Python tracker
<http://bugs.python.org/issue30916>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2955
___
Python tracker
<http://bugs.python.org/issue9566>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
@haypo https://github.com/python/cpython/pull/2904
--
___
Python tracker
<http://bugs.python.org/issue9566>
___
___
Python-bug
Changes by Segev Finer :
--
components: +asyncio
nosy: +yselivanov
___
Python tracker
<http://bugs.python.org/issue31048>
___
___
Python-bugs-list mailin
Changes by Segev Finer :
--
title: ResourceWarning in
test_asyncio.test_events..ProactorEventLoopTests.test_create_server_ssl_verify_failed
-> ResourceWarning in
test_asyncio.test_events.ProactorEventLoopTests.test_create_server_ssl_verify_fai
Changes by Segev Finer :
--
pull_requests: +2973
___
Python tracker
<http://bugs.python.org/issue31048>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
Since PR 2750 is merged, I think this can be closed.
--
___
Python tracker
<http://bugs.python.org/issue30222>
___
___
Python-bug
Segev Finer added the comment:
The only warnings that remain are:
1. The one from gcmodule.c due to dtrace, which is handled in PR 2852.
2. The peephole warnings which I'm not sure about. Are overflows possible there
or not? That is, if there are already internal caps that prevent the
New submission from Segev Finer:
This is caused by
https://github.com/python/cpython/blob/caa1280d1ee5f828f346b585169a7592371d3faa/Lib/pdb.py#L1096
which always passes our own current stdin and stdout, and this triggers the
conditional in
https://github.com/python/cpython/blob
Changes by Segev Finer :
--
nosy: +Segev Finer
___
Python tracker
<http://bugs.python.org/issue9325>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2996
___
Python tracker
<http://bugs.python.org/issue31078>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
It was fixed here:
https://github.com/python/cpython/commit/cc16be85c0b7119854c00fb5c666825deef641cf#diff-1308fc2ab05154373172f10e931c78f0L986.
So it's fixed for 3.6 and 3.7, but not in 3.5.
--
nosy: +Segev
Changes by Segev Finer :
--
pull_requests: +3040
___
Python tracker
<http://bugs.python.org/issue16865>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +3175
___
Python tracker
<http://bugs.python.org/issue30747>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +3174
___
Python tracker
<http://bugs.python.org/issue9566>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Segev Finer:
_sha3 is missing a commented out line from Setup.dist like the rest of the
modules. It's really only used if someone wants to edit Setup.dist to build
with the _sha3 module statically. Mostly a trivial change to add it in.
--
components:
Changes by Segev Finer :
--
pull_requests: +3311
___
Python tracker
<http://bugs.python.org/issue31328>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +3324
___
Python tracker
<http://bugs.python.org/issue31328>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
It's been a while since this got any attention...
--
___
Python tracker
<http://bugs.python.org/issue19764>
___
___
Pytho
Segev Finer added the comment:
Added the 4th version after review by eryksun (In rietveld).
--
Added file:
http://bugs.python.org/file46814/windows-subprocess-close-fds-v4.patch
___
Python tracker
<http://bugs.python.org/issue19
Segev Finer added the comment:
Added the 5th version after another review by eryksun (In rietveld).
--
Added file:
http://bugs.python.org/file46819/windows-subprocess-close-fds-v5.patch
___
Python tracker
<http://bugs.python.org/issue19
Segev Finer added the comment:
Oh LOL!!! I missed the fact that Python finally moved to GitHub!
Rebased the patch on top of the Git master XD (And removed accidentally
committed code... sorry...)
I still submitted as a patch since I don't know if the infrastructure handles
moving a patch
Segev Finer added the comment:
OK Rietveld definitely punted on the git patch (I guess it's only for the old
Mercurial repo, I don't think it actually even support Git...)
I will try re-submitting the patch as a PR so that it can be revie
New submission from Segev Finer:
subprocess triggers a debug assertion in the CRT on failure to execute the
process due to closing the pipe *handles* in the except clause using os.close
rather than .Close() (os.close closes CRT file descriptors and not handles).
In addition to that once this
Changes by Segev Finer :
--
pull_requests: +1345
___
Python tracker
<http://bugs.python.org/issue29191>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +1346
___
Python tracker
<http://bugs.python.org/issue30121>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
The PR was merged for Python 3.7. But this also affects older Python 3 versions
that include liblzma/_lzma as far as I can tell. So I'm not closing this issue
just yet, so that it will serve as a remainder, in case you do want to back
port this.
Thank
Segev Finer added the comment:
GitHub PR bit rotting away... :P
Just a friendly reminder :)
--
___
Python tracker
<http://bugs.python.org/issue19764>
___
___
New submission from Segev Finer:
Found due to this: https://github.com/pytest-dev/py/issues/103
I'm going to submit a PR.
--
components: IO, Windows
messages: 294975
nosy: Segev Finer, paul.moore, steve.dower, tim.golden, zach.ware
priority: normal
severity: normal
status: open
Changes by Segev Finer :
--
pull_requests: +1992
___
Python tracker
<http://bugs.python.org/issue30544>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
We might want to open a separate issue for the FD redirection issue since this
one was really about the _io._WindowsConsoleIO.write GetLastError() issue.
Also see https://github.com/pytest-dev/pytest/pull/2462 where I discuss this
more.
Still not sure why it
New submission from Segev Finer:
_WindowsConsoleIO works by taking the handle out of the file descriptors of the
stdio handles (get_osfhandle) and reading/writing to it directly using
ReadConsoleW/WriteConsoleW.
The problem is that some Python code wants to do file descriptor level
Changes by Segev Finer :
--
nosy: +eryksun
___
Python tracker
<http://bugs.python.org/issue30555>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
type: -> behavior
___
Python tracker
<http://bugs.python.org/issue30544>
___
___
Python-bugs-list mailing list
Unsubscrib
Changes by Segev Finer :
--
pull_requests: +2007
___
Python tracker
<http://bugs.python.org/issue30555>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Segev Finer:
There is a ref leak in sys.getwindowsversion due to using PyTuple_Pack and
forgetting that it increments the ref count of passed in objects. Found here:
https://github.com/python/cpython/pull/1927#issuecomment-306019512.
Clearly shows up in python -m test -R 3
Changes by Segev Finer :
--
nosy: +eryksun
___
Python tracker
<http://bugs.python.org/issue30567>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2019
___
Python tracker
<http://bugs.python.org/issue30567>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Segev Finer:
I'm hitting the following warnings when building in VS2015 Update 3 in 32-bit:
liblzma:
warning C4028: formal parameter {} different from declaration
warning C4113: 'FARPROC' differs in parameter lists from '{}'
war
Changes by Segev Finer :
--
pull_requests: +2170
___
Python tracker
<http://bugs.python.org/issue30631>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Segev Finer :
--
pull_requests: +2171
___
Python tracker
<http://bugs.python.org/issue9566>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Segev Finer :
According to https://tools.ietf.org/html/rfc6152 you should only send an 8bit
email body when the server advertises 8BITMIME in the EHLO, and you should
declare it with BODY=8BITMIME in the MAIL command.
The default cte_type is 8bit for the new default email
New submission from Segev Finer :
The new text parameter in subprocess was documented for subprocess.run but not
for subprocess.Popen in the Sphinx documentation. It is documented in the
docstring for both though.
--
assignee: docs@python
components: Documentation
messages: 311949
Change by Segev Finer :
--
keywords: +patch
pull_requests: +5418
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue32370>
___
___
Python-
Change by Segev Finer :
--
versions: +Python 3.8
___
Python tracker
<https://bugs.python.org/issue32370>
___
___
Python-bugs-list mailing list
Unsubscribe:
Segev Finer added the comment:
Windows seems to dereference lpStartupInfo->lpAttributeList, despite it being
NULL since we had no attributes to set. Smells like a Windows bug in the
compatibility shim, since it doesn't crash in non compatibility mode.
Not setting EXTENDED_STARTUPINFO
Change by Segev Finer :
--
keywords: +patch
pull_requests: +5716
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue32818>
___
___
Python-
1 - 100 of 132 matches
Mail list logo