[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