[issue35967] Better platform.processor support

[Jason R. Coombs]

Jason R. Coombs  added the comment:

Reading further, the 'sysctl' call seems to only be for BSD 
(https://www.freebsd.org/cgi/man.cgi?sysctl(3)). I could find the man page for 
sysctl for BSD but not Linux. There is a _sysctl in Linux 
(http://man7.org/linux/man-pages/man2/sysctl.2.html), but it's use is 
discouraged and it doesn't provide the necessary information.

Now I suspect that the aforementioned GNU coreutils 'uname' implementation is 
only for non-Linux systems, as none of the underlying system calls are relevant 
on Linux. I expect if one compiled that uname on Linux, 'uname -p' would emit 
'unknown'.

Meaning I still don't know how to get a 'uname -p' result on Linux (without 
invoking uname -p).

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36239] gettext: GNUTranslations doesn't parse properly comments in description

[Julien Palard]

Julien Palard  added the comment:

After some research I found a few comments around comments being marked as 
starting by #-#-#-#-# and ending with #-#-#-#-#, not just starting with #.

In gettext-0.19.8.1 sources for example:

$ grep -r '#-#-#-#-' | head
gettext-tools/misc/po-mode.el:#-#-#-#-#  file name reference  #-#-#-#-#
gettext-tools/misc/po-mode.el:  (let* ((marker-regex "^#-#-#-#-#  \\(.*\\)  
#-#-#-#-#\n")
gettext-tools/src/msgl-cat.c:  char *id = xasprintf ("#-#-#-#-# 
 %s  #-#-#-#-#",

Or more precisly in `gettext-tools/tests/msgcat-10`:

# Verify msgcat of two files, when the header entries have different comments
# but the same contents. The resulting header entry is not marked fuzzy,
# because the #-#-#-#-# are only in comments and do not necessarily require
# translator attention; in other words, an msgstr which is valid in both input
# files is also valid in the result.

I'm however surprised not to find much of "#-#-#-#-#" in the source code, like 
if they are just looking a single # like you do here.

Not sure which one is the better, eliminating lines with a pair of #-#-#-#-# or 
lines starting with a #, both looks OK to me (we're only speaking about the 
header here, not the msgstr, so it won't have much impact).

Personally I'd go for eliminating #-#-#-#-# as this is the only case we've 
seen, and is the "documented" one in the GNU gettext test cases.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue35967] Better platform.processor support

[Jason R. Coombs]

Jason R. Coombs  added the comment:

Best I can tell, neither sysinfo nor sysctl are exposed in any way to Python, 
so it may not be possible to accurately load the processor information from 
those system calls without writing a wrapper in C. What I might try is to 
experiment with ctypes to see if I can prove the concept.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue35435] Discourage external links to old docs.

[Julien Palard]

Julien Palard  added the comment:

Let's not have duplicate issues, so I'm closing this in favor of 
https://github.com/python/python-docs-theme/issues/24.

Thanks Terry for noticing.

--
resolution:  -> duplicate
stage: needs patch -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue35967] Better platform.processor support

[Jason R. Coombs]

Jason R. Coombs  added the comment:

Aha! It seems the 'sysinfo' call is for Solaris: 
https://docs.oracle.com/cd/E23823_01/html/816-5167/sysinfo-2.html

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36240] Definitions of time

[Robert Billing]

New submission from Robert Billing :

https://docs.python.org/3.7/library/time.html contains the text "UTC is 
Coordinated Universal Time (formerly known as Greenwich Mean Time, or GMT)". 
This is not strictly true. Referring to 
https://en.wikipedia.org/wiki/Coordinated_Universal_Time the definition of UTC 
is in terms of frequency standards, GMT in terms of astronomy. Hence with GMT 
each minute has exactly 60 seconds, but the length of the second may vary 
slightly to account for changes in the Earth's rotation. With UTC each second 
is the same length, but "leap seconds" can be inserted or removed giving 59 and 
61 second minutes. The leap seconds keep the two systems in sync to less than 
one second.

This of course only matters for the most critical applications, but it would be 
worth documenting correctly.

--
assignee: docs@python
components: Documentation
messages: 337482
nosy: Robert Billing, docs@python
priority: normal
severity: normal
status: open
title: Definitions of time
type: enhancement
versions: Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue35967] Better platform.processor support

[Jason R. Coombs]

Jason R. Coombs  added the comment:

The first call I see in that routine is to "sysinfo", but the signature of that 
function doesn't match what I find in the [man pages for that 
function](http://man7.org/linux/man-pages/man2/sysinfo.2.html). So that 
function must be coming from elsewhere.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue35967] Better platform.processor support

[Marc-Andre Lemburg]

Marc-Andre Lemburg  added the comment:

Thanks. It would be good to do some before/after tests on popular
platforms, e.g. a few Linuxes, MacOS, Windows.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue31256] xml.etree.ElementTree: add support for doctype in tostring method

[Bernt Røskar Brenna]

Bernt Røskar Brenna  added the comment:

See also: https://bugs.python.org/issue36227

PR: https://github.com/python/cpython/pull/12225

--
nosy: +Bernt.Røskar.Brenna

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue31256] xml.etree.ElementTree: add support for doctype in tostring method

[Bernt Røskar Brenna]

Change by Bernt Røskar Brenna :


--
keywords: +patch
pull_requests: +12225
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36227] Add default_namespace argument to xml.etree.ElementTree.tostring()

[Bernt Røskar Brenna]

Bernt Røskar Brenna  added the comment:

I pushed changes to the PR. I also added xml_declaration and default_namespace 
to the tostringlist() method.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36239] gettext: GNUTranslations doesn't parse properly comments in description

[STINNER Victor]

STINNER Victor  added the comment:

Attached files:

* comments.po: PO file with a comment in headers
* messages.mo: comments.po compiled with msgfmt
* parse.py: Python script to parse messages.mo

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36239] gettext: GNUTranslations doesn't parse properly comments in description

[STINNER Victor]

Change by STINNER Victor :


Added file: https://bugs.python.org/file48197/messages.mo

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36239] gettext: GNUTranslations doesn't parse properly comments in description

[STINNER Victor]

Change by STINNER Victor :


Added file: https://bugs.python.org/file48196/comments.po

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36239] gettext: GNUTranslations doesn't parse properly comments in description

[STINNER Victor]

Change by STINNER Victor :


Added file: https://bugs.python.org/file48195/parse.py

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36239] gettext: GNUTranslations doesn't parse properly comments in description

[STINNER Victor]

New submission from STINNER Victor :

When a translation .po file contains a comment in headers, it's kept when 
compiled as .mo by msgfmt.

Example with test.po:
---
msgid ""
msgstr ""
"Content-Type: text/plain; charset=UTF-8\n"
"Plural-Forms:  nplurals=2; plural=(n != 1);\n"
"#-#-#-#-#  plo.po (PACKAGE VERSION)  #-#-#-#-#\n"
---

Compile it with "msgfmt". Parse the output file messages.mo using test.py 
script:
---
import gettext, pprint
with open("messages.mo", "rb") as fp:
t = gettext.GNUTranslations()
t._parse(fp)
pprint.pprint(t._info)
---

Output on Python 3.7.2:
---
{'content-type': 'text/plain; charset=UTF-8',
 'plural-forms': 'nplurals=2; plural=(n != 1);\n'
 '#-#-#-#-#  plo.po (PACKAGE VERSION)  #-#-#-#-#'}
---

Output of Fedora Python 2.7.15 which contains a fix:
---
{'content-type': 'text/plain; charset=UTF-8',
 'plural-forms': 'nplurals=2; plural=(n != 1);'}
---

I'm not sure that keeping the comment as part of plural forms is correct. 
Comments should not be ignored?

I made my test on Fedora 29: msgfmt 0.19.8.1, Python 3.7.2.

Links:

* https://bugs.python.org/issue1448060#msg27754
* https://bugs.python.org/issue1475523
* https://bugzilla.redhat.com/show_bug.cgi?id=252136

Fedora has a patch since 2007 to ignore comments:
https://src.fedoraproject.org/rpms/python2/blob/master/f/python-2.5.1-plural-fix.patch

I can easily convert the patch to a PR, maybe with a test. The question is more 
if the fix is correct or not.

--
components: Library (Lib)
messages: 337476
nosy: mdk, vstinner
priority: normal
severity: normal
status: open
title: gettext: GNUTranslations doesn't parse properly comments in description
versions: Python 2.7, Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue35967] Better platform.processor support

[Jason R. Coombs]

Jason R. Coombs  added the comment:

It won't be possible in general to emit what the function returned before, as 
`uname` is a symbolic reference to an arbitrary executable, which can vary by 
platform and release and local environment.

What I might be able to do is find the implementation of "uname" and see if 
there's a way to get the value from the same source. I did find what I believe 
is the [canonical 
source](https://github.com/coreutils/coreutils/blob/66e2daa689fefec9ed201a04696b9f52d049d89a/src/uname.c#L301-L343).

I'll explore if those calls can be translated to Python.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36237] data_files / Install Additional Files written unclearly such that it's not obvious what parameter is what

[jt]

jt  added the comment:

Ok I am now realizing after more tests I actually read the docs correctly, and 
in that sense they're not ambiguous.

It's just that I tried to use data_files for something that it's not for: I did 
look at package_data first and I get that's what I SHOULD be using, however I 
want to add in a file that isn't in the package root in the original source 
folder. (package is in "./src", file I want to add in is in "./") package_data 
seems unable to do this, so I thought data_files would get me there, but 
apparently not in any reasonable way.

Closing this since I was apparently just surprised it didn't do what I want, 
and didn't actually misread anything as I thought I had

--
stage:  -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36227] Add default_namespace argument to xml.etree.ElementTree.tostring()

[Bernt Røskar Brenna]

Bernt Røskar Brenna  added the comment:

I will add xml_declaration and push to the existing PR.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36233] xml ElementTree quotation marks of xml version string

[Karthikeyan Singaravelan]

Change by Karthikeyan Singaravelan :


--
nosy: +eli.bendersky, scoder, serhiy.storchaka

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36238] distutils complains "package init file 'xxx/__init__.py' not found (or not a regular file)" when using Cythonized __init__.pyx

[jt]

New submission from jt :

distutils spits out a warning:

package init file 'xxx/__init__.py' not found (or not a regular file)

... when using Cythonized __init__.pyx instead. However, the installed package 
works absolutely fine, it can be imported & used perfectly, so this warning 
seems bogus.

I checked, and this warning is generated inside distutils/command/build_py.py 
in the build_py class in method check_package(). I suggest that this warning 
isn't generated in case a C extension is found for the __init__ module, checked 
in whatever way is appropriate at this stage (e.g. by seeing if there's an 
__init__.pyx)

--
components: Distutils
messages: 337472
nosy: dstufft, eric.araujo, jt
priority: normal
severity: normal
status: open
title: distutils complains "package init file 'xxx/__init__.py' not found (or 
not a regular file)" when using Cythonized __init__.pyx
versions: Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36237] data_files / Install Additional Files written unclearly such that it's not obvious what parameter is what

[jt]

New submission from jt :

I find the following doc section found at 
https://docs.python.org/3.7/distutils/setupscript.html#installing-additional-files
 about data_files somewhat unclear:

```
setup(...,
  data_files=[('bitmaps', ['bm/b1.gif', 'bm/b2.gif']),
  ('config', ['cfg/data.cfg']),
  ('/etc/init.d', ['init-script'])]
 )

Each (directory, files) pair in the sequence specifies the installation 
directory and the files to install there.

...

The directory should be a relative path. It is interpreted relative to the 
installation prefix (Python’s sys.prefix for system installations; 
site.USER_BASE for user installations).
```

This gives me no clue what the installation actually is relative to, since e.g. 
sys.prefix is just `/usr` - surely I'm not supposed to specify 
"lib64/python3.7/site-packages//file" as a target?

That is probably not how that text is supposed to be read and I'm sure 
packaging expert understand what sort of prefix is actually meant, but could 
this be more elaborated on, maybe an actual example for a fake package of 
whether this is e.g. the package folder root after the install inside the site 
packages, or the site packages folder itself, or ...?

--
assignee: docs@python
components: Documentation
messages: 337471
nosy: docs@python, jt
priority: normal
severity: normal
status: open
title: data_files / Install Additional Files written unclearly such that it's 
not obvious what parameter is what
versions: Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36236] Python crash on macOS when CWD is invalid

[Pablo Galindo Salgado]

Change by Pablo Galindo Salgado :


--
keywords: +patch
pull_requests: +12224
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36236] Python crash on macOS when CWD is invalid

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

This is happening because _Py_wgetcwd returns NULL (although the underliying 
getcwd system call populates the `fullpath` variable correctly) but its caller, 
_PyPathConfig_ComputeArgv0, does not check the return value:

_Py_wgetcwd(fullpath, Py_ARRAY_LENGTH(fullpath));
 argv0 = fullpath;

 and its caller, pymain_run_python, interprets a failure in 
_PyPathConfig_ComputeArgv0 as a memory problem:

 PyObject *path0 = _PyPathConfig_ComputeArgv0(config->argc,
  config->argv);
 if (path0 == NULL) {
 err = _Py_INIT_NO_MEMORY();
 goto done;
 }

--
nosy: +pablogsal

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36235] distutils.sysconfig.customize_compiler() overrides CFLAGS var with OPT var if CFLAGS env var is set

[STINNER Victor]

Change by STINNER Victor :


--
keywords: +patch
pull_requests: +12223
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36236] Python crash on macOS when CWD is invalid

[László Kiss Kollár]

New submission from László Kiss Kollár :

CPython crashes with "pymain_compute_path0: memory allocation failed" when 
attempting to execute it with a library module from a previously deleted 
directory.

To reproduce:

cd ~/tmp/python_crash

rm -rf ~/tmp/python_crash

python3.7 -m pdb
Fatal Python error: pymain_compute_path0: memory allocation failed
ValueError: character U+ef3a8fe0 is not in range [U+; U+10]

Current thread 0x00011060e5c0 (most recent call first):

--
components: macOS
messages: 337469
nosy: lkollar, ned.deily, ronaldoussoren
priority: normal
severity: normal
status: open
title: Python crash on macOS when CWD is invalid
versions: Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36235] distutils.sysconfig.customize_compiler() overrides CFLAGS var with OPT var if CFLAGS env var is set

[STINNER Victor]

New submission from STINNER Victor :

When a C extension is built by distutils, 
distutils.sysconfig.customize_compiler() is used to configure compiler flags. 
Extract of the code:

(cc, cxx, opt, cflags, ccshared, ldshared, shlib_suffix, ar, ar_flags) = \
get_config_vars('CC', 'CXX', 'OPT', 'CFLAGS',
'CCSHARED', 'LDSHARED', 'SHLIB_SUFFIX', 'AR', 'ARFLAGS')

...
if 'CFLAGS' in os.environ:
cflags = opt + ' ' + os.environ['CFLAGS']
ldshared = ldshared + ' ' + os.environ['CFLAGS']
if 'CPPFLAGS' in os.environ:
cpp = cpp + ' ' + os.environ['CPPFLAGS']
cflags = cflags + ' ' + os.environ['CPPFLAGS']
ldshared = ldshared + ' ' + os.environ['CPPFLAGS']
...

If the CFLAGS environment variable is set, the 'CFLAGS' configuration variable 
is overriden with the 'OPT' configuration variable: cflags = opt + ...

This bug has been fixed since 2013 in Fedora and RHEL by this patch:
https://src.fedoraproject.org/rpms/python2/blob/master/f/00168-distutils-cflags.patch

RHEL bug report:
https://bugzilla.redhat.com/show_bug.cgi?id=849994

I converted that patch to a pull request.

--
components: Build
messages: 337468
nosy: vstinner
priority: normal
severity: normal
status: open
title: distutils.sysconfig.customize_compiler() overrides CFLAGS var with OPT 
var if CFLAGS env var is set
versions: Python 2.7, Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36234] test_os: add tests for invalid uid type

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +1

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36230] Please sort assertSetEqual's output

[Windson Yang]

Windson Yang  added the comment:

My point is careful about the non-sortable object. My mistake, this should be 
an enhancement, not a bug.

--
versions:  -Python 2.7, Python 3.4, Python 3.5, Python 3.6, Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36228] Support coercion of complex to float/int

[Mark Dickinson]

Mark Dickinson  added the comment:

> So I suggest:
> 1. Methods __floor__ and __ceil__ for complex object.

What's the use-case for these? It's not a particularly natural operation, and 
I've never had a need for a complex "floor" operation, either as a 
mathematician or as a developer. Do you have an example of real-world code that 
would benefit?

For those rare cases where this is needed, it isn't that hard to spell out 
`complex(floor(z.real), floor(z.imag))`, or to write a custom `complex_floor` 
function.

Also, what type should `math.floor` return when applied to a complex number? 
`math.floor` of a `float` object returns an `int`, so the analogous operation 
on a complex number should return a Gaussian integer. But we don't have a 
Gaussian integer type in standard Python, and it wouldn't be appropriate to add 
one.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36234] test_os: add tests for invalid uid type

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +12221
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36234] test_os: add tests for invalid uid type

[STINNER Victor]

STINNER Victor  added the comment:

See _Py_Uid_Converter() and _Py_Gid_Converter() of Modules/posixmodule.c.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36234] test_os: add tests for invalid uid type

[STINNER Victor]

Change by STINNER Victor :


--
keywords: +patch
Added file: https://bugs.python.org/file48194/00157-uid-gid-overflows.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36234] test_os: add tests for invalid uid type

[STINNER Victor]

New submission from STINNER Victor :

The Fedora package of python contains a downstream patch to add more tests for 
invalid uid types:
https://src.fedoraproject.org/rpms/python2/blob/master/f/00157-uid-gid-overflows.patch

I propose to make this patch upstream. More tests never hurts :-)

uid/gid overflow has been fixed bpo-4591.

The patch comes from:
https://bugzilla.redhat.com/show_bug.cgi?id=697470

Attached PRs add more tests.

--
components: Tests
messages: 337464
nosy: vstinner
priority: normal
severity: normal
status: open
title: test_os: add tests for invalid uid type
versions: Python 2.7, Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36233] xml ElementTree quotation marks of xml version string

[Johann Krauter]

New submission from Johann Krauter :

I have the problem, that a xml file is save with the following xml declaration: 
 
instead of 


I would propose to change the line number 769 in the ElementTree.py to: 
write("\n" % (
declared_encoding,))

--
components: XML
files: ElementTree.py
messages: 337463
nosy: Photoniker
priority: normal
severity: normal
status: open
title: xml ElementTree quotation marks of xml version string
type: behavior
versions: Python 3.7
Added file: https://bugs.python.org/file48193/ElementTree.py

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36232] Improve error message on dbm.open

[Marco Rougeth]

New submission from Marco Rougeth :

If dbm.open is used with the flags 'r' or 'w' (read-only) to open a file that 
doesn't exist, it raises an exception with the message "need 'c' or 'n' flag to 
open new db". It'd be better to have a more explicit error message like "db 
file doesn't exist, use 'c' or 'n' flag to open new db".

--
messages: 337462
nosy: rougeth
priority: normal
pull_requests: 12220
severity: normal
status: open
title: Improve error message on dbm.open

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36231] no "proper" header files on macOS 10.14 Mojave

[Dmitrii Pasechnik]

New submission from Dmitrii Pasechnik :

Neither Xcode nor its command-line tools on macOS 10.14 Mojave come with header 
files installed in /usr/ and other "normal" directories.

This is not documented in https://devguide.python.org/setup/#macos-and-os-x

While an extra step to handle this, i.e. to install the headers, is available 
(see a discussion on https://bugs.python.org/issue34956), Apple stated that 
this workaround will disappear.

It is thus highly desirable to provide a way to deal with headers located not 
at /usr/include, but at `xcrun --show-sdk-path`/usr/include.
A small change in setup.py along the lines of the following:

--- a/setup.py
+++ b/setup.py
@@ -134,7 +134,8 @@ def macosx_sdk_root():
 cflags = sysconfig.get_config_var('CFLAGS')
 m = re.search(r'-isysroot\s+(\S+)', cflags)
 if m is None:
-sysroot = '/'
+import subprocess
+sysroot = subprocess.check_output(["xcrun", 
"--show-sdk-path"]).decode("utf-8").strip('\n')
 else:
 sysroot = m.group(1)
 return sysroot
@@ -146,6 +147,7 @@ def is_macosx_sdk_path(path):
 """
 return ( (path.startswith('/usr/') and not path.startswith('/usr/local'))
 or path.startswith('/System/')
+or path.startswith('/Applications/')
 or path.startswith('/Library/') )

with the necessary changes to enable various modules (see attachment for a 
complete POC diff against the current master), the result builds and passes all 
the (enabled) tests on an OSX 10.13 system with empty /usr/include and 
/usr/local/include containing only LZMA headers.

Needless to say, a proper patch would not modify Modules/Setup, it'd adjust 
configure.ac etc.

--
components: macOS
files: noincludedirs_OSX.patch
keywords: patch
messages: 337461
nosy: dimpase, ned.deily, ronaldoussoren
priority: normal
severity: normal
status: open
title: no "proper" header files on macOS 10.14 Mojave
type: compile error
versions: Python 3.8
Added file: https://bugs.python.org/file48192/noincludedirs_OSX.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36227] Add default_namespace argument to xml.etree.ElementTree.tostring()

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

Okay. Bernt, do you mind to add also the xml_declaration option in PR 12225 or 
create a separate PR for issue31256?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36228] Support coercion of complex to float/int

[Марат Нагаев]

Марат Нагаев  added the comment:

Oh, __floor__ can return anything.
So I suggest:
1. Methods __floor__ and __ceil__ for complex object.
2. Don't change __float__ and __int__ methods.
3. So I think it isn't nessesary to add methods floor and ceil to complex 
module.
Example:
from math import *
z=1.1+1.1j
floor(z) #1+1j, same as z.__floor__()
ceil(z) #2+2j, samse as z.__ceil__()

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36227] Add default_namespace argument to xml.etree.ElementTree.tostring()

[Stefan Behnel]

Stefan Behnel  added the comment:

lxml does not support the "default_namespace" option specifically (because its 
tree model preserves namespace prefixes), but it generally makes all 
(justifiable) serialisation options available to both tostring() and ET.write().

I think the same should apply to ElementTree. Both the "default_namespace" and 
"doctype" options seem useful regardless of the serialisation target.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36230] Please sort assertSetEqual's output

[Rémi Lapeyre]

Rémi Lapeyre  added the comment:

@rhettinger
> It's possible to have non-sortable elements in the set, so you'll either need 
> to sort on the repr of the elements or have a fallback

Yes, it is the repr that is used in the loop and that what's the sorting needs 
to be done against.


@Windson Yang
> we should compare their repr() without sort() like now.

I'm not sure to understand, can you provide more information about what you are 
thinking of?

Is there a reason to add 2.7, 3.4, 3.5, 3.6 and 3.7 to versions affected ?

As far as I can tell, this is a new feature and should only go in 3.8.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36227] Add default_namespace argument to xml.etree.ElementTree.tostring()

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

See also issue31256.

I do not know what is better: make tostring() to accept all options of write(), 
or keep it simpler. What strategy lxml supports?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue36227] Add default_namespace argument to xml.etree.ElementTree.tostring()

[Stefan Behnel]

Stefan Behnel  added the comment:

The feature seems reasonable to me and the patch looks good.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue35967] Better platform.processor support

[Marc-Andre Lemburg]

Marc-Andre Lemburg  added the comment:

As the documentation says, the API is intended as fairly portable 
implementation of the Unix uname helper across platforms. It's fine to redirect 
this directly to e.g. /proc output instead of using the executable, but in 
whatever you do here, the output of platform.uname() needs to stay compatible 
to what the function returned prior to such a change, which usually means: to 
the output of the uname helper on a system.

Could you please check that on most systems, the output remains the same ?

Thanks.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue34162] idlelib/NEWS.txt for 3.8.0 (and backports)

[miss-islington]

miss-islington  added the comment:


New changeset 02351ed1ba601445735c6a6eae6f9b1d37fae8cd by Miss Islington (bot) 
in branch '3.7':
bpo-34162: Add entries for idlelib/NEWS.txt (GH-12232)
https://github.com/python/cpython/commit/02351ed1ba601445735c6a6eae6f9b1d37fae8cd


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue34162] idlelib/NEWS.txt for 3.8.0 (and backports)

[miss-islington]

Change by miss-islington :


--
pull_requests: +12219

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue34162] idlelib/NEWS.txt for 3.8.0 (and backports)

[Terry J. Reedy]

Terry J. Reedy  added the comment:


New changeset 8a1bab92915dd5c88832706c56af2f5611181d50 by Terry Jan Reedy in 
branch 'master':
bpo-34162: Add entries for idlelib/NEWS.txt (#12232)
https://github.com/python/cpython/commit/8a1bab92915dd5c88832706c56af2f5611181d50


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com