Re: [OE-devel] [PATCH] ntp: Resolve some abnormal behaviors

[Xufeng Zhang <xufeng.zhang@windriver.com>]

On 05/31/2013 05:08 PM, Xufeng Zhang wrote:
> On 05/31/2013 04:34 PM, Paul Eggleton wrote:
>> On Friday 31 May 2013 14:18:45 Xufeng Zhang wrote:
>>> The main changes include:
>>> 1). Add ntp:ntp(user:group) to system.
>>> 2). Running ntpd dameon as ntp:ntp.
>>> 3). Move relevant files from /usr/bin to /usr/sbin.
> Thanks for your review, Paul!
>
>> I'm not sure but I think the way this has been done 
>> (--with-binsubdir=sbin) is
>> going to bypass the ability to set sbindir in the distro config. If 
>> that's the
>> case then I don't think this is a good change, although I appreciate 
>> having
>> ntpd in /usr/sbin by default would be more appropriate.
> But currently it's the proper and easy way to do that, right?
> I can think out any better way to do the same job.
Sorry, a typo, s/can/can't/


Thanks,
Xufeng

>
>>> 4). Add crypto support.
>> The support is already there, it's just off by default and that was
>> intentional. Those that need this can easily enable it in their 
>> distro config.
> I'm not sure why we do that intentional, but shouldn't the package run
> as secure as possible by default?
>
>
> Thanks,
> Xufeng
>
>> Cheers,
>> Paul
>>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>