This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author girtsf
Recipients girtsf
Date 2018-08-28.19:00:56
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1535482856.6.0.56676864532.issue34534@psf.upfronthosting.co.za>
In-reply-to
Content
importlib.resources does not seem to work with packages that don't have __init__.py present. Since 3.3+ generally there is no need to create empty __init__.py, as directories are automatically treated as packages even without the file present. So my expectation would be that importlib.resources would follow the same suit. 

Repro, where I expect that both "mod1" and "mod2" would return the files when contents() is called:

$ find .
.
./mod2
./mod2/data.txt
./test.py
./mod1
./mod1/__init__.py
./mod1/data.txt

$ cat test.py
import importlib.resources

print('mod1:', list(importlib.resources.contents('mod1')))
print('mod2:', list(importlib.resources.contents('mod2')))

$ python3.7 test.py
mod1: ['__init__.py', '__pycache__', 'data.txt']
mod2: []    # <---- here I would expect to see files too


Looking at the source of Lib/importlib/resources.py, there is the following code in contents():

    [..]
    # Is the package a namespace package?  By definition, namespace packages
    # cannot have resources.  We could use _check_location() and catch the
    # exception, but that's extra work, so just inline the check.
    elif package.__spec__.origin is None or not package.__spec__.has_location:
        return ()
    [..]

This is the branch that is taken - but it's not necessarily a namespace package here, at least to my understanding. Is there a way to make the code here distinguish between namespace packages and implicit non-namespace packages here, and allow contents() (and other functions) to work?
History
Date User Action Args
2018-08-28 19:00:56girtsfsetrecipients: + girtsf
2018-08-28 19:00:56girtsfsetmessageid: <1535482856.6.0.56676864532.issue34534@psf.upfronthosting.co.za>
2018-08-28 19:00:56girtsflinkissue34534 messages
2018-08-28 19:00:56girtsfcreate