zipimport loader.get_data() requires absolute zip file path

[warsaw]

BPO	32866
Nosy	@warsaw, @serhiy-storchaka, @nnja

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2019-01-17.18:39:23.541>
created_at = <Date 2018-02-18.02:49:21.472>
labels = ['3.8']
title = 'zipimport loader.get_data() requires absolute zip file path'
updated_at = <Date 2019-01-17.18:40:01.610>
user = 'https://github.com/warsaw'

bugs.python.org fields:

activity = <Date 2019-01-17.18:40:01.610>
actor = 'nnja'
assignee = 'none'
closed = True
closed_date = <Date 2019-01-17.18:39:23.541>
closer = 'barry'
components = []
creation = <Date 2018-02-18.02:49:21.472>
creator = 'barry'
dependencies = []
files = []
hgrepos = []
issue_num = 32866
keywords = []
message_count = 3.0
messages = ['312296', '325727', '333893']
nosy_count = 3.0
nosy_names = ['barry', 'serhiy.storchaka', 'nnja']
pr_nums = []
priority = 'normal'
resolution = 'works for me'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue32866'
versions = ['Python 3.8']

[warsaw]

Over in https://gitlab.com/python-devs/importlib_resources/issues/48 we have a report of a FileNotFoundError when trying to read a resource from a zip file. Upon further debugging, I found that zipimport's loader.get_data() raises an unexpected OSError. Interestingly, if the path to the zip file is absolute, everything works as expected, but if the path is relative, then it fails.

There's probably a missing abspath() in there somewhere, but as zipimport is written in C, I really didn't spend much time digging around in gdb.

[serhiy-storchaka]

zipimport has been rewritten in pure Python (bpo-25711).

[warsaw]

I believe this bug does not affect Python 3.8:

(Using a Python 3.8 virtualenv):

% python demo.pyz
Reading: resource.txt
Length: 19
% python pwd/demo.pyz
Reading: resource.txt
Length: 19

I think it's too risky (and too much work, given it would have to be ported to the C implementation of zipimport) to change this in earlier Pythons. So, closing.