-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove selinux once and for all #4
base: halium-7.1
Are you sure you want to change the base?
Conversation
We dont use selinux, it only causes problems, so let's disable it. Change-Id: I31bc39e1960978e92c4a856aacb026ed95b0fb0a
…elinux Change-Id: I522dc6c27fdbc52ceb8e2b1dc85edfd854ebf837
Looks good to me, if you have build and run tested it, feel free to merge. |
And oh well.. there are conflicts.. pls2fix. |
Bump @mariogrip |
I just tested it, and still works. there is no conflicts now |
Also removal of /system/sepolicy is needed in order for this to work |
Should we merge? |
When apply patch got this error This error is generated in function property_init in system_properties.cpp (system/core/init) |
I'm totally for merging this, this is currently required for suzu (Sony Xperia X) to succeed in building. |
Taking another look at the build result, applying this patch makes things worse. |
We dont use selinux, it only causes problems, so let's disable it.
Change-Id: I31bc39e1960978e92c4a856aacb026ed95b0fb0a