* [PATCH] libselinux: Change the location of _selinux.so
@ 2016-06-27 8:46 Petr Lautrbach
2016-07-01 16:59 ` Stephen Smalley
0 siblings, 1 reply; 2+ messages in thread
From: Petr Lautrbach @ 2016-06-27 8:46 UTC (permalink / raw)
To: selinux
There was a change in swig-3.10 to use importlib instead of imp. While
the implementation with imp looked for _selinux.so also into the same directory
as __init__.py is, a new module with importlib searchs only standard paths.
It means that we need to move _selinux.so from $(PYLIBDIR)/site-packages/selinux/
to $(PYLIBDIR)/site-packages/.
Fixes:
>>> import selinux
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/usr/lib64/python2.7/site-packages/selinux/__init__.py", line 21, in <module>
_selinux = swig_import_helper()
File "/usr/lib64/python2.7/site-packages/selinux/__init__.py", line 20, in swig_import_helper
return importlib.import_module('_selinux')
File "/usr/lib64/python2.7/importlib/__init__.py", line 37, in import_module
__import__(name)
ImportError: No module named _selinux
Signed-off-by: Petr Lautrbach <plautrba@redhat.com>
---
libselinux/src/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libselinux/src/Makefile b/libselinux/src/Makefile
index d94163e..37d01af 100644
--- a/libselinux/src/Makefile
+++ b/libselinux/src/Makefile
@@ -156,7 +156,7 @@ install: all
install-pywrap: pywrap
test -d $(PYLIBDIR)/site-packages/selinux || install -m 755 -d $(PYLIBDIR)/site-packages/selinux
- install -m 755 $(SWIGSO) $(PYLIBDIR)/site-packages/selinux/_selinux.so
+ install -m 755 $(SWIGSO) $(PYLIBDIR)/site-packages/_selinux.so
install -m 755 $(AUDIT2WHYSO) $(PYLIBDIR)/site-packages/selinux/audit2why.so
install -m 644 $(SWIGPYOUT) $(PYLIBDIR)/site-packages/selinux/__init__.py
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] libselinux: Change the location of _selinux.so
2016-06-27 8:46 [PATCH] libselinux: Change the location of _selinux.so Petr Lautrbach
@ 2016-07-01 16:59 ` Stephen Smalley
0 siblings, 0 replies; 2+ messages in thread
From: Stephen Smalley @ 2016-07-01 16:59 UTC (permalink / raw)
To: Petr Lautrbach, selinux
On 06/27/2016 04:46 AM, Petr Lautrbach wrote:
> There was a change in swig-3.10 to use importlib instead of imp. While
> the implementation with imp looked for _selinux.so also into the same directory
> as __init__.py is, a new module with importlib searchs only standard paths.
> It means that we need to move _selinux.so from $(PYLIBDIR)/site-packages/selinux/
> to $(PYLIBDIR)/site-packages/.
>
> Fixes:
>>>> import selinux
> Traceback (most recent call last):
> File "<stdin>", line 1, in <module>
> File "/usr/lib64/python2.7/site-packages/selinux/__init__.py", line 21, in <module>
> _selinux = swig_import_helper()
> File "/usr/lib64/python2.7/site-packages/selinux/__init__.py", line 20, in swig_import_helper
> return importlib.import_module('_selinux')
> File "/usr/lib64/python2.7/importlib/__init__.py", line 37, in import_module
> __import__(name)
> ImportError: No module named _selinux
>
> Signed-off-by: Petr Lautrbach <plautrba@redhat.com>
Thanks, applied.
> ---
> libselinux/src/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libselinux/src/Makefile b/libselinux/src/Makefile
> index d94163e..37d01af 100644
> --- a/libselinux/src/Makefile
> +++ b/libselinux/src/Makefile
> @@ -156,7 +156,7 @@ install: all
>
> install-pywrap: pywrap
> test -d $(PYLIBDIR)/site-packages/selinux || install -m 755 -d $(PYLIBDIR)/site-packages/selinux
> - install -m 755 $(SWIGSO) $(PYLIBDIR)/site-packages/selinux/_selinux.so
> + install -m 755 $(SWIGSO) $(PYLIBDIR)/site-packages/_selinux.so
> install -m 755 $(AUDIT2WHYSO) $(PYLIBDIR)/site-packages/selinux/audit2why.so
> install -m 644 $(SWIGPYOUT) $(PYLIBDIR)/site-packages/selinux/__init__.py
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-07-01 16:59 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-27 8:46 [PATCH] libselinux: Change the location of _selinux.so Petr Lautrbach
2016-07-01 16:59 ` Stephen Smalley
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.