Ronald Oussoren added the comment:
Eddie,
I have read the thread and still don't see why breaking existing code is a good
thing to do.
As I wrote earlier there are two issues you're trying to address with one patch:
1) Not all types in the CPython tree are initialised using Py
Ronald Oussoren added the comment:
win32com is part of pywin32, which is a 3th-party extension. Please check with
that project first (if there is bug it is probably on that side).
--
components: +Windows
nosy: +paul.moore, ronaldoussoren, steve.dower, tim.golden, zach.ware
resolution
Change by Ronald Oussoren :
--
nosy: +ronaldoussoren
___
Python tracker
<https://bugs.python.org/issue34543>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ronald Oussoren added the comment:
IMHO SystemError is the wrong exception, that exception is primarily used to
signal implementation errors.
BTW. I can reproduce crashes in a couple of runs of your scriptlet:
Python 3.7.0 (v3.7.0:1bf9cc5093, Jun 26 2018, 23:26:24)
[Clang 6.0 (clang
Ronald Oussoren added the comment:
It's not as easy as that, the SystemError in the original report is caused by
invalid use of a C-API due to partial initialisation of an _struct.Struct
instance.
The solution is likely two-fold:
1) Ensure that __new__ fully initialises the fields in
Ronald Oussoren added the comment:
@DeKrain: I agree
--
components: +Extension Modules -Library (Lib)
___
Python tracker
<https://bugs.python.org/issue34
Ronald Oussoren added the comment:
I'm not to happy about the crash, the interpreter shouldn't crash like this.
This may well be Tk related though.
--
resolution: -> not a bug
stage: -> resolved
status: open -> pending
type: -> crash
Ronald Oussoren added the comment:
The primary Python.org installer supports 10.9 and later by way of being
compiled on 10.9, and there is also an installer that supports 10.6.
There currently is no promise that building on 10.13 results in code that works
on earlier versions of macOS
Ronald Oussoren added the comment:
I get the same error, also with python3.7. Both for homebrew and a python.org
installer.
--
components: +macOS
nosy: +ned.deily, ronaldoussoren
versions: +Python 3.7
___
Python tracker
<https://bugs.python.
Ronald Oussoren added the comment:
Binaries currently work on the system you compiled on and any later release of
macOS. That's the only "promise" there currently is.
Anything better requires work, and hence someone willing to do that work.
I'd love to be able to build
Ronald Oussoren added the comment:
There's not enough information in the issue to be sure, but I'm inclined to say
this is a 3th-party problem. To be sure someone will have to reproduce the
problem in a debugger, or provide a stack trace of the crash.
The initial message says
Ronald Oussoren added the comment:
Yes it should, the same problem has been present from the start.
--
___
Python tracker
<https://bugs.python.org/issue31
Ronald Oussoren added the comment:
Linking with newer SDKs is needed to get access to some newer features and
APIs. That's less relevant for Python itself, but can be relevant for the
embedding application. You can still compile Python itself on an older
machine, then link it in
Ronald Oussoren added the comment:
IMHO there doesn't need to be a new API unless the semantics change w.r.t
Py_TYPE (for example by adding an API that returns an owned reference instead
of a borrowed reference). Py_TYPE can work with an opaque version of PyObject
by turning it i
Ronald Oussoren added the comment:
@vstinner: another option is to ignore "use_vfork" on platforms that don't have
POSIX_SPAWN_USEVFORK. Using vfork or not is primarily a optimisation, ignoring
the flag should not result in different behaviour (other than speed).
Ronald Oussoren added the comment:
Good question. A comment on
<https://stackoverflow.com/questions/2731531/faster-forking-of-large-processes-on-linux>
says that glibc already uses vfork where appropriate, explicitly using the
flag may not be necessary.
Note that I haven't veri
Ronald Oussoren added the comment:
vfork() is more dangerous than fork() because the parent and child processes
share memory (not copy-on-write, but really the same memory). Whether or not
this affects posix_spawn depends on its implementation (to give a very vague
statement).
Glibc
Ronald Oussoren added the comment:
@Zorg: you don't need to sell the usefulness of this functionality. Its just
that the python developers are volunteers and hence have limited time to work
on python.
--
___
Python tracker
&
Ronald Oussoren added the comment:
I get the impression that go uses vfork on Linux, not posix_spawn, because go
doesn't use libc (based on reading the referenced issue, not on deep knowledge
of go and its implementation).
I do wonder why glibc's implementation of posix_spawn d
Ronald Oussoren added the comment:
I did some more research:
* <https://sourceware.org/bugzilla/show_bug.cgi?id=10354> seems to indicate
that glibc switched to a different implementation of posix_spawn that uses
clone(2) instead of fork()/vfork() to avoid some problems with vfork().
Ronald Oussoren added the comment:
Note that the POSIX_SPAWN_USEVFORK may not do anything at this point (see the
link in my previous message, if I read the code correctly POSIX_SPAWN_USEVFORK
is no longer used in the implementation of posix_spawn(3)).
Even if it did do something the user
Ronald Oussoren added the comment:
Same here, even if I learned more about the implementation of posix_spawn than
I should have to know ;-)
--
___
Python tracker
<https://bugs.python.org/issue34
Ronald Oussoren added the comment:
The pull request is wrong, the use of __PYVENV_LAUNCHER__ in pythonw.c is
correct and should not be removed. Where the current code goes wrong is that it
doesn't clear the environment as soon as possible.
This patch should basically do the trick
Change by Ronald Oussoren :
--
pull_requests: +8922
___
Python tracker
<https://bugs.python.org/issue22490>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ronald Oussoren added the comment:
I've added PR 9516 that just clears __PYVENV_LAUNCHER__ from the environment as
soon as it is no longer needed.
A more involved change would be to change both the interpreter and the stub
executable to avoid the need to use an environment variable i
Ronald Oussoren added the comment:
To use the framework build you either have to install, use or set
"DYLD_FRAMEWORK_PATH" in the environment (See the definition of RUNFORSHARED in
Makefile for the value to use).
To properly test the this issue you have to install (or trick the s
Ronald Oussoren added the comment:
I've added back port labels for 3.6 and 3.7.
AFAIK 3.5 is closed for bugfixes at this point (except for security fixes).
--
___
Python tracker
<https://bugs.python.org/is
Ronald Oussoren added the comment:
I had the some problem when using mypy. What does reproduce the issue is the
following:
* Create *two* venv environments
* Install mypy in one of them
* Activate the other
* Run the subprocess call in the activated venv
I then get a message about not being
Ronald Oussoren added the comment:
Do you have a $PYTHONPATH in your environment?
For some reason the 3.7 install is looking in the site-packages directory for
the 2.7 install. This is probably configuration on your system.
--
___
Python tracker
Ronald Oussoren added the comment:
This is not a bug. You are searching for the character range from "+" to "*",
which is an invalid range because "+" > "*".
--
nosy: +ronaldoussoren
resolution: -> not a bug
stage: -> resolv
Ronald Oussoren added the comment:
What do you think the problem is? The output of the script is what I'd expect
it to be.
Note that str.find() returns -1 when the needle is not present (and the first
offset where the needle is found when it is present).
--
nosy: +ronaldous
Ronald Oussoren added the comment:
I suspect that there may also be confusion about the line ending in multiline
strings. Is there any documentation on the fact that this is always "\n" and
not "\r\n" (even on Windows)?
The closest I've found is the Lexical Anal
Ronald Oussoren added the comment:
@pahskazan: It is correct that '\r\n' is not found in those strings, Python
source code is always parsed after conversion to "Unix" line endings with "\n"
between lines. This is basicly the same as the "universal newl
Ronald Oussoren added the comment:
To completely do this requires two things:
1) Upgrade SQLite included in a number of binary distributions
2) Update the sqlite extension to allow creating custom window functions in
Python
The latter is definitely something that cannot be done in a
Ronald Oussoren added the comment:
My latter points makes it possible to define custom windowing functions in
Python, similar to how it is already possible to define other SQLite functions
in Python.
--
___
Python tracker
<ht
Ronald Oussoren added the comment:
The error is printed by libc, there's nothing we can do about that.
--
___
Python tracker
<https://bugs.python.org/i
Ronald Oussoren added the comment:
FWIW I don't see the problem either, VM running 10.14.1 (beta) with Python
3.7.0.
--
___
Python tracker
<https://bugs.python.org/is
Ronald Oussoren added the comment:
IMHO implementing "__index__" for UUID would not the correct solution. That
method is meant be used by integer-like objects. One of the side effects of
implementing "__index__" is that indexing lists with UUID instances would start
to
Ronald Oussoren added the comment:
I'm not convinced that any change is needed, this is completely expected
behaviour (and not special to modules).
The following code also raises RecursionError:
class VerboseObject:
def __setattr__(self, nm, value):
print(f"Setti
Ronald Oussoren added the comment:
I agree with Ned. The code fragment works for me as well on 10.14.1 and I've
used simular code in the past on different versions of macOS as well.
@ssbarnea:
- What is the hostname as shown in the "Sharing" preference pane? Does the name
Ronald Oussoren added the comment:
AFAIK dis.dis only decompiles bytecode in python 2, as is mentioned in the
documentation at
<http://docs.python.org/2/library/dis.html?highlight=dis#dis.dis>. In Python 3
dis.dis can compile a string with Python source to bytecode and decompile that
Ronald Oussoren added the comment:
Mark: if you know Python you can answer that question yourself by reading the
code of the subprocess and os modules.
>From a fairly short glance at the code I'd say that _wrap_close is not
>obsolete. It is a wrapper about a file object for th
Ronald Oussoren added the comment:
On 28 May, 2013, at 10:28, Elazar Gershuni wrote:
>
> Elazar Gershuni added the comment:
>
> Is it standard procedure to raise an unrelated exception in when an invalid
> parameter is passed? I did not encounter any other library function
Ronald Oussoren added the comment:
The automatic compilation of source code cannot be added to the 2.7 version of
the dis module because that would be a new feature and we don't add new
features in bugfix releases. The feature was added to Python 3.x in issue
#6507, and was first includ
Ronald Oussoren added the comment:
I can confirm the problem. It appears to be a stack overflow, when I increase
the stack size of the main thead (by adding "-Wl,-stack_size,2faf000" to the
link command for BUILDPYTHON) the crash in test_json goes away.
Appearently the default max
Ronald Oussoren added the comment:
LGTM
--
___
Python tracker
<http://bugs.python.org/issue18080>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ronald Oussoren added the comment:
The attached patch fixes the problem on OSX by increasing the maximum stack
size of the main thread from 8M (the default) to 16M.
NOTE: The -Wl,-stack_size,... option cannot be added to LDFLAGS, ld errors out
when that option is used when linking a shared
Ronald Oussoren added the comment:
The update fixes the name error mention in rietveld.
--
Added file:
http://bugs.python.org/file30399/issue-18075-osx-stacksize-update.txt
___
Python tracker
<http://bugs.python.org/issue18
Ronald Oussoren added the comment:
I'm +1 on the general idea, but have some remarks anyway:
1) "for all users" vs. "current user"
This likely requires a post-install script to fix up the load command's in
binaries: on OSX binaries contain absolute paths to the l
Ronald Oussoren added the comment:
BTW. There is a completely different way for distributing Python: drop the
entire installer and provide a zipfile containing a single application bundle.
The most likely target for the application is IDLE, with a new menu item for
making the command-line
Ronald Oussoren added the comment:
There appears to be an unrelated edit to configure/configure.ac in the patch
(moving ARCH_RUN_32BIT to another location), otherwise the patch looks fine.
I agree that it isn't worthwhile to try to port EasyDialogs to an API that is
available, that&#x
Ronald Oussoren added the comment:
That sucks. A patch should be easy, but I probably won't get around to that
until sunday.
--
___
Python tracker
<http://bugs.python.org/is
Ronald Oussoren added the comment:
I'd prefer to keep the default recursion limit the same as on Linux, changing
the recursion limit on OSX would just introduce an unnecessary difference
between the two platforms.
The patch changes the maximum stack size from 8 to 16 MByte, neither of
Ronald Oussoren added the comment:
On 30 May, 2013, at 14:50, STINNER Victor wrote:
>
> STINNER Victor added the comment:
>
>> The patch changes the maximum stack size from 8 to 16 MByte, neither of
>> which is huge and shouldn't cause other problems.
>
Ronald Oussoren added the comment:
Linking with "-framework Python" is always a bad idea because you have no
control over which version of Python you link with other than by changing
global system state (the Current link). Also: include files aren't included
using the framew
Ronald Oussoren added the comment:
There is a python3.3 in .../Python.framework/Versions/3.3/lib because
.../Python.framework/Versions/3.3 is basicly a regular unix install with some
trivial changes (in particular, there is a Python shared library in the root of
the tree, there is a
Ronald Oussoren added the comment:
I've tested a patch for the 2.7 branch on a 10.5 machine (which also failed to
build without the patch), and will commit once I've finished running the
testsuite on the 3.3 branch as well.
--
___
Pyth
Ronald Oussoren added the comment:
Does the compiler use the correct header files? The error messages seem to
indicate that the 2.7 headers are used instead of the 3.4 ones (all missing
identifiers were introduced in py3k and are not in 2.x)
FWIW, the distutils tests work fine on my machine
Ronald Oussoren added the comment:
The problem appears to be gone in current releases (at least on OSX 10.8 with
Tk 8.5) and I therefore propose to close the issue.
--
nosy: +ned.deily
___
Python tracker
<http://bugs.python.org/issue1512
Ronald Oussoren added the comment:
On 7 Jun, 2013, at 16:54, Brett Cannon wrote:
>
> If that's true, does this simply mean I need to build Python without any
> installed libraries to avoid this conflict?
That's correct. There are other bugreports about other p
Ronald Oussoren added the comment:
I'm reworking the patch in #14455 and one of the things I intend to do is to
add a keyword argument that controls whether or not the keys in dictionaries
will be sorted by plistlib.
--
___
Python tracker
Ronald Oussoren added the comment:
I've attached issue14455-v2.txt with an updated patch. The patch is still very
much a work in progress, I haven't had as much time to work on this as I'd like.
This version:
* Should apply cleanly to the tip of the default branch
* Move a
Changes by Ronald Oussoren :
Added file: http://bugs.python.org/file30525/issue14455-v3.txt
___
Python tracker
<http://bugs.python.org/issue14455>
___
___
Python-bug
Ronald Oussoren added the comment:
v3 is still a work in progress, and still fails some tests
* Replaced test data by data generated by a helper script (to make it
easier to update)
* Use 'subtest' feature of unittest library in 3.4
* Small tweaks to plist library (the dump/load/d
Ronald Oussoren added the comment:
See also:
#18168: request for the sort_keys option
#11101: request for an option to ignore 'None' values when writing
#9256: datetime.datetime objects created by plistlib don't include timezone
information
(and looking at the code I'd sa
New submission from Ronald Oussoren:
Super() walks the MRO and looks for the looked-up attribute in the __dict__ of
types on the MRO. This can cause problems when classes implement
__getattribute__: a class on the MRO can implement a method that super() won't
find because it isn
Ronald Oussoren added the comment:
I didn't, but the attached script should do the trick. The code suffers badly
from copy&paste editing, but more or less has the same behavior as PyObjC.
It defines a subclass of list (MyList) with an append method, and more
importantly also defines
Ronald Oussoren added the comment:
The test failure I'm getting is caused by a difference in the order in which
items are written to the archive. I'm working on a fix.
--
___
Python tracker
<http://bugs.python.o
Ronald Oussoren added the comment:
Looks like it. That issue was closed without a good reason though.
--
___
Python tracker
<http://bugs.python.org/issue18
Ronald Oussoren added the comment:
v4 passes the included tests.
The testsuite isn't finished yet.
--
Added file: http://bugs.python.org/file30530/issue14455-v4.txt
___
Python tracker
<http://bugs.python.org/is
Ronald Oussoren added the comment:
The fifth version of the patch should be much cleaner.
Changes:
* Coding style cleanup, the new code uses PEP8 conformant names for
methods and variables.
* Explicitly make private classes private by prefixing their name with
an underscore (including the
Ronald Oussoren added the comment:
I've attached a first very rough patch to demonstrate what I was rambling about.
With this patch and a patched version of PyObjC I can use the builtin super
class with the proxy classes for Objective-C classes.
Open issues:
* Does this need a pep?
Ronald Oussoren added the comment:
Second version of the poc adds a way to implement the hook in Python code, and
supers_updated.py uses that hook.
Still no tests, no documentation and with awfull naming :-)
--
Added file: http://bugs.python.org/file30547/supers_updated.py
Ronald Oussoren added the comment:
.
--
___
Python tracker
<http://bugs.python.org/issue18181>
___
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.pyth
Ronald Oussoren added the comment:
The buildbot seems to be happy right now (at least as far as getaddrinfo is
concerned).
--
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
Ronald Oussoren added the comment:
I don't think it is possible to avoid adding a new slot. The default
implementation of tp_getattro (PyObject_GenericGetAttr) looks in the instance
dict, while super does not (because it wants a less specific implementation).
PyObject_GenericGetAttr
Ronald Oussoren added the comment:
added draft of a pep-style proposal
--
Added file: http://bugs.python.org/file30556/super-hook-proposal.rst
___
Python tracker
<http://bugs.python.org/issue18
Ronald Oussoren added the comment:
Slightly updated patch (different names, added some documentation)
--
Added file: http://bugs.python.org/file30555/issue-18181-poc-v3.txt
___
Python tracker
<http://bugs.python.org/issue18
Ronald Oussoren added the comment:
The attached patch explicitly mentions that stdin/stdout/stderr are opened as
binary streams when universal_newlines is False.
I'm not convinced that adding this text will solve the confusion, subprocess
has fairly complex documentation due to the ric
Ronald Oussoren added the comment:
I don't agree that adding a table/picture is the only thing that can be done,
or even that it is a good idea.
IMHO the subprocess documentation is clear enough about the distinction between
bytes and string, especially in the section about conven
Changes by Ronald Oussoren :
--
assignee: docs@python -> ronaldoussoren
stage: needs patch -> commit review
___
Python tracker
<http://bugs.python.org/i
Ronald Oussoren added the comment:
>From reading the source of Python/marshal.c it seems that the read function's
>raise an exception on I/O errors, but don't return a specific value (that is,
>sentence starting with "It appears that" is wrong).
PyMarshal_ReadLo
Ronald Oussoren added the comment:
I'd prefer to have new variants of the PyErr_SetFromErrno* functions where the
errno value is explicitly passed in instead of adding a side-channel for
passing in the value.
--
nosy: +ronaldoussoren
___
P
New submission from Ronald Oussoren:
Changeset a3559c8c614b added -Werror=statement-after-declaration to the CFLAGS
for compiler that support it.
This new flags is fine for CPython itself (which is explicitly writting in C89
style to support older compilers and Microsoft Visual Studio), but
Ronald Oussoren added the comment:
Is "-Werror=statement-after-declaration" the right solution anyway? This errors
out on just one of the incompatibilities between C89 and C99, although it is
one that is easy to get wrong accidentally.
Gcc (and clang) also support '-std=c89&#
Ronald Oussoren added the comment:
I had a patch that only changed distutils.sysconfig.get_config_var('CFLAGS')
when not building python, but then noticed that this doesn't work: the setup.py
file for stdlib extensions fetches information from the toplevel sysco
New submission from Ronald Oussoren:
While working on a fix for #18211 I noticed two problems with the setup.py
that's used to build the stdlib extensions.
1) setup.py uses sysconfig instead of distutils.sysconfig. The sematics of the
two modules a slighty different.
2) The block of
Ronald Oussoren added the comment:
The setup.py issues are in #18255.
--
dependencies: +CPython setup.py problems
___
Python tracker
<http://bugs.python.org/issue18
Ronald Oussoren added the comment:
So much for obviously correct changes...
1) fix typo in import statement (from sysconfig import import ...)
2) setup.py uses sysconfig.get_path and that's not present in
distutils.sysconfig.
(Sigh..., why are there two modules with almost but not ent
Ronald Oussoren added the comment:
I just noticed the patch is not good enough: "python-config --cflags" still
prints the -Werror flag because it is now a shell script and doesn't use
sysconfig data at all.
--
___
Python
New submission from Ronald Oussoren:
Changeset c0370730b364 introduced a shell-script implementation of
python-config (see issue #16235). The older python implementation is still
present in $(srcdir)/Misc and generated by Makefile.pre.in.
--
messages: 191416
nosy: doko, ronaldoussoren
Ronald Oussoren added the comment:
Also, the shell-script version of python-config uses the -f option of readlink
and that option is not supported on OSX 10.8 (or older versions, I haven't
looked at the 10.9 beta yet)
Furthermore the entire readlink command is not present in HP-UX
Ronald Oussoren added the comment:
I don't agree. Distutils still uses distutils.sysconfig for configuration
information and setup.py wants to adjust that configuration, it should
therefore use distutils.sysconfig instead of the global one.
That said, it would be better to change dist
Ronald Oussoren added the comment:
I'd be +1 on extending the zipfile API in 3.4 (with a documentation update in
older releases as suggested by anatoly):
* Add a method or property to ZipInfo for (un)packing
the external_attr field
* Add an keyword argument to Zipfile.extrac
Ronald Oussoren added the comment:
See also #15795
--
___
Python tracker
<http://bugs.python.org/issue18262>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ronald Oussoren added the comment:
Closing this as a duplicate of #15795 (which already has a patch)
--
resolution: -> duplicate
superseder: -> Zipfile.extractall does not preserve file permissions
___
Python tracker
<http://bugs.p
Ronald Oussoren added the comment:
On first glance the patch looks good. I haven't tested it with the current
trunk though.
--
nosy: +ronaldoussoren
___
Python tracker
<http://bugs.python.org/is
Ronald Oussoren added the comment:
The correct format string for Py_ssize_t with sprintf is
"%"PY_FORMAT_SIZE_T"d" (defined and explained in pyport.h).
--
nosy: +ronaldoussoren
___
Python tracker
<http://bug
Ronald Oussoren added the comment:
See also #18269.
--
nosy: +ronaldoussoren
___
Python tracker
<http://bugs.python.org/issue3354>
___
___
Python-bugs-list mailin
Ronald Oussoren added the comment:
I don't think that a new parameter format is sufficient. For the message in
this call the new parameter isn't even needed, you'd "just" have to ensure that
enough information is available in convertsimple to create a more useful
mes
1301 - 1400 of 2445 matches
Mail list logo
