{"action":"created","issue":{"url":"https://api.github.com/repos/pytest-dev/pytest/issues/1597","repository_url":"https://api.github.com/repos/pytest-dev/pytest","labels_url":"https://api.github.com/repos/pytest-dev/pytest/issues/1597/labels{/name}","comments_url":"https://api.github.com/repos/pytest-dev/pytest/issues/1597/comments","events_url":"https://api.github.com/repos/pytest-dev/pytest/issues/1597/events","html_url":"https://github.com/pytest-dev/pytest/pull/1597","id":159178219,"number":1597,"title":"Ensure that a module within a namespace package can be found by --pyargs","user":{"login":"taschini","id":7879276,"avatar_url":"https://avatars.githubusercontent.com/u/7879276?v=3","gravatar_id":"","url":"https://api.github.com/users/taschini","html_url":"https://github.com/taschini","followers_url":"https://api.github.com/users/taschini/followers","following_url":"https://api.github.com/users/taschini/following{/other_user}","gists_url":"https://api.github.com/users/taschini/gists{/gist_id}","starred_url":"https://api.github.com/users/taschini/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/taschini/subscriptions","organizations_url":"https://api.github.com/users/taschini/orgs","repos_url":"https://api.github.com/users/taschini/repos","events_url":"https://api.github.com/users/taschini/events{/privacy}","received_events_url":"https://api.github.com/users/taschini/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-06-08T14:29:48Z","updated_at":"2016-06-09T08:05:35Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/pytest-dev/pytest/pulls/1597","html_url":"https://github.com/pytest-dev/pytest/pull/1597","diff_url":"https://github.com/pytest-dev/pytest/pull/1597.diff","patch_url":"https://github.com/pytest-dev/pytest/pull/1597.patch"},"body":"This PR supersedes #1568 and addresses the problem highlighted in #1567 and in part #478.\r\n\r\nSee #1568 for the description of the test setup.\r\n\r\nUnlike #1568, this PR does not attempt to emulate the import machinery to determine the filename of a  package or directory, delegating instead most of the work to the ``pkgutil`` module of the standard library.\r\n\r\n**NB** In case of a non-top-level module or package, ``pkgutil.find_loader`` *will* import the parent package or packages."},"comment":{"url":"https://api.github.com/repos/pytest-dev/pytest/issues/comments/224826875","html_url":"https://github.com/pytest-dev/pytest/pull/1597#issuecomment-224826875","issue_url":"https://api.github.com/repos/pytest-dev/pytest/issues/1597","id":224826875,"user":{"login":"taschini","id":7879276,"avatar_url":"https://avatars.githubusercontent.com/u/7879276?v=3","gravatar_id":"","url":"https://api.github.com/users/taschini","html_url":"https://github.com/taschini","followers_url":"https://api.github.com/users/taschini/followers","following_url":"https://api.github.com/users/taschini/following{/other_user}","gists_url":"https://api.github.com/users/taschini/gists{/gist_id}","starred_url":"https://api.github.com/users/taschini/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/taschini/subscriptions","organizations_url":"https://api.github.com/users/taschini/orgs","repos_url":"https://api.github.com/users/taschini/repos","events_url":"https://api.github.com/users/taschini/events{/privacy}","received_events_url":"https://api.github.com/users/taschini/received_events","type":"User","site_admin":false},"created_at":"2016-06-09T08:05:35Z","updated_at":"2016-06-09T08:05:35Z","body":"Appveyor caught a little problem with a test setup using PyPy on Windows: I'll fix that.\r\n\r\n@nicoddemus I'll incorporate your feedback.  The reason why that try-except clause is needed is that sometimes ``_parsearg()`` gets invoked with ``AssertionRewritingHook``, which does not define a get_filename method, already in place.  Hence, the exception you'll get is an ``AttributeError``.\r\n\r\nI'll catch explicitly that Exception and add a comment to explain why that is necessary."}}
{"actor":{"display_login":"taschini"}}