On 17 June 2014 17:23, Mark Brown <broo...@linaro.org> wrote: > On 17 June 2014 12:18, Vishal Bhoj <vishal.b...@linaro.org> wrote: > >> On 17 June 2014 16:41, Fathi Boudra <fathi.bou...@linaro.org> wrote: >> >>> Vishal, >>> >>> 1. against which tree should it be applied? >>> >> I have tested it against TC2 with the LSK tree. I was not sure if the >> patches directly go to LSK. I thought it should first go into config >> fragment tree. >> > > I don't know what the "config fragment tree" is but the LSK doesn't use it > and you've not CCed - I guess you should send this to Andrey too for > inclusion in linux-linaro? I've added him. > I intended to add Andrey as well but somehow missed him.
> > >> 2. do you want it enabled by default for all android builds? >>> >> > >> Moving ahead SELinux is mandatory for Android so it needs to be enabled >> across all the platforms we care for. Linux-linaro and LSK are validated on >> VExpress platform so I am enabling the userspace bits for VExpress to start >> with. >> > > OK, I'll a change to the LSK which enables these options - I can't see any > reason to restrict them to Android so I won't just apply this as-is. Please > include information like the above in the changelogs for patches, this is > just generally good practice and will help people figure out why the change > is being made and if it is sensible to apply it. > Will keep this in mind.
_______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev