[issue32731] getpass.getuser() raises an unspecified exceptions (ImportError, OSError, etc)

2018-01-31 Thread Yaron de Leeuw

Yaron de Leeuw <m...@jarondl.net> added the comment:

What kind of exception did you have in mind? As the 'someone' you mentioned, I 
can contribute this.

btw, os.getuid() on windows raises an attribute error, so we need to check that 
too.

--

___
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue32731>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue32731] getpass.getuser() raises an unspecified exceptions (ImportError, OSError, etc)

2018-01-31 Thread Yaron de Leeuw

Change by Yaron de Leeuw <m...@jarondl.net>:


--
nosy: +jarondl

___
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue32731>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue26901] Argument Clinic test is broken

2017-08-03 Thread Yaron de Leeuw

Yaron de Leeuw added the comment:

I have done some diving and git bisecting into this. There are at least two 
issues:

1. `Clinic.__init__` no longer works without passing in a filename. Because 
soon after `if filename` in line 1726 [1] it calls `d('file')` which only works 
if there is a `file` buffer.
This makes the odd line 38 [2] in clinic_test.py to fail miserably with 
sys.exit. Deleting this line results in a proper test failure (22 test fail out 
of 54), which is already much better than the sys exit message.

2. `Clinic` has shifted from using `field_destinations` to 
`destination_buffers` in issue 23500. The test code did not change accordingly.


This means that we need to adapt the test code to the use 
`destination_buffers`, and add it to the regular test suite. The whole 
situation could only have happened because the test were not run regularly.

I would like to tackle this, but I am not entirely sure yet what should the 
differences between FakeClinic and Clinic be. So I am studying it. Hints are 
welcome.




[1] 
https://github.com/python/cpython/blob/6969eaf4682beb01bc95eeb14f5ce6c01312e297/Tools/clinic/clinic.py#L1726

[2] 
https://github.com/python/cpython/blob/6969eaf4682beb01bc95eeb14f5ce6c01312e297/Tools/clinic/clinic_test.py#L38

--
nosy: +jarondl

___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue26901>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue25711] Rewrite zipimport from scratch

2017-08-03 Thread Yaron de Leeuw

Yaron de Leeuw added the comment:

What is the status of this work? Is there anything I can do to help make this 
happen?

--
nosy: +jarondl

___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue25711>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue31109] zipimport argument clinic conversion

2017-08-03 Thread Yaron de Leeuw

Yaron de Leeuw added the comment:

Oh that's a cool idea which I was not aware of.

--

___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue31109>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue31109] zipimport argument clinic conversion

2017-08-02 Thread Yaron de Leeuw

Changes by Yaron de Leeuw <m...@jarondl.net>:


--
pull_requests: +3028

___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue31109>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue31109] zipimport argument clinic conversion

2017-08-02 Thread Yaron de Leeuw

New submission from Yaron de Leeuw:

Convert zipimport to use the argument clinic.
I will submit the PR shortly.

--
components: Argument Clinic, Library (Lib)
messages: 299667
nosy: jarondl, larry, twouters
priority: normal
severity: normal
status: open
title: zipimport argument clinic conversion
type: enhancement
versions: Python 3.7

___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue31109>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue26253] tarfile in stream mode always set zlib compression level to 9

2017-08-01 Thread Yaron de Leeuw

Yaron de Leeuw added the comment:

I have submitted a PR on GitHub https://github.com/python/cpython/pull/2962

--
nosy: +jarondl
versions: +Python 3.7 -Python 3.6

___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue26253>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue26440] tarfile._FileInFile.seekable is broken in stream mode

2017-07-31 Thread Yaron de Leeuw

Yaron de Leeuw added the comment:

_Stream provides seek, but only positive seeking is allowed. Is that considered 
seekable? Also, maybe _Stream should inherit from io.BaseIO. WDYT?

--
nosy: +jarondl

___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue26440>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue26253] tarfile in stream mode always set zlib compression level to 9

2017-07-31 Thread Yaron de Leeuw

Changes by Yaron de Leeuw <m...@jarondl.net>:


--
pull_requests: +3009

___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue26253>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com