[Buildroot] [PATCH v2] Enable vlc to run as root
Joao Pinto
Joao.Pinto at synopsys.com
Wed Sep 16 10:38:52 UTC 2015
Hello Thomas,
On 9/16/2015 11:36 AM, Thomas Petazzoni wrote:
> Hello Joao,
>
> Thanks for this new iteration. It looks good, except for one minor nit
> that isn't very important, but worth mentioning.
>
> On Wed, 16 Sep 2015 09:26:49 +0100, jpinto wrote:
>> From: Joao Pinto <jpinto at synopsys.com>
>>
>> A great number of embedded systems only include the root user.
>> This patch has the goal of enabling vlc to be ran as root.
>>
>> Signed-off-by: Joao Pinto <jpinto at synopsys.com>
>> Tested-by: Joao Pinto <jpinto at synopsys.com>
>
> It doesn't make sense to include a Tested-by from you on your patch. We
> obviously expect that you have tested your own patch. Tested-by tags
> (as well as Acked-by or Reviewed-by tags) are meant to be given by
> *other* developers, after testing and/or reviewing your patch.
>
> The other minor nit is that the title of the commit still doesn't match
> the Buildroot best practice, which consists in using titles having the
> following format:
>
> <package>: <description>
>
> So in your case, it should be:
>
> vlc: enable to run as root
>
Understood, I will have it in mind in future contributions.
> Both issues can be fixed by the Buildroot maintainer when he applies
> the patch, but I wanted to let you know so you can make things better
> for your next contribution!
>
> Thanks!
>
> Thomas
>
Thanks,
Joao
More information about the buildroot
mailing list