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

Reply via email to