* [PATCH] ntp: fix perl path in scripts
@ 2014-01-22 7:47 Kai Kang
2014-01-22 11:31 ` Martin Jansa
0 siblings, 1 reply; 3+ messages in thread
From: Kai Kang @ 2014-01-22 7:47 UTC (permalink / raw)
To: openembedded-devel
Scripts ntptrace and ntp-wait use /bin/perl directly that cause
ntp-utils depends on /bin/perl but no package provides it. It should use
'${USRBINPATH}/env perl'.
Signed-off-by: Kai Kang <kai.kang@windriver.com>
---
meta-networking/recipes-support/ntp/ntp.inc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/meta-networking/recipes-support/ntp/ntp.inc b/meta-networking/recipes-support/ntp/ntp.inc
index 9ea24e9..8ada604 100644
--- a/meta-networking/recipes-support/ntp/ntp.inc
+++ b/meta-networking/recipes-support/ntp/ntp.inc
@@ -54,6 +54,8 @@ do_install_append() {
sed -i 's!/etc/!${sysconfdir}/!g' ${D}${sysconfdir}/init.d/ntpd ${D}${bindir}/ntpdate-sync
sed -i 's!/var/!${localstatedir}/!g' ${D}${sysconfdir}/init.d/ntpd ${D}${bindir}/ntpdate-sync
sed -i 's!^PATH=.*!PATH=${base_sbindir}:${base_bindir}:${sbindir}:${bindir}!' ${D}${bindir}/ntpdate-sync
+ sed -i '1s!/bin/perl!${USRBINPATH}/env perl!' ${D}${sbindir}/ntptrace
+ sed -i '1s!/bin/perl!${USRBINPATH}/env perl!' ${D}${sbindir}/ntp-wait
install -d ${D}/${sysconfdir}/default
install -m 644 ${WORKDIR}/ntpdate.default ${D}${sysconfdir}/default/ntpdate
--
1.8.1.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ntp: fix perl path in scripts
2014-01-22 7:47 [PATCH] ntp: fix perl path in scripts Kai Kang
@ 2014-01-22 11:31 ` Martin Jansa
2014-02-19 2:19 ` Kang Kai
0 siblings, 1 reply; 3+ messages in thread
From: Martin Jansa @ 2014-01-22 11:31 UTC (permalink / raw)
To: openembedded-devel
[-- Attachment #1: Type: text/plain, Size: 1697 bytes --]
On Wed, Jan 22, 2014 at 03:47:38PM +0800, Kai Kang wrote:
> Scripts ntptrace and ntp-wait use /bin/perl directly that cause
> ntp-utils depends on /bin/perl but no package provides it. It should use
> '${USRBINPATH}/env perl'.
Next time please add [meta-networking] tag, but it's already in right
patchwork bundle so that you don't need to resend it.
>
> Signed-off-by: Kai Kang <kai.kang@windriver.com>
> ---
> meta-networking/recipes-support/ntp/ntp.inc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/meta-networking/recipes-support/ntp/ntp.inc b/meta-networking/recipes-support/ntp/ntp.inc
> index 9ea24e9..8ada604 100644
> --- a/meta-networking/recipes-support/ntp/ntp.inc
> +++ b/meta-networking/recipes-support/ntp/ntp.inc
> @@ -54,6 +54,8 @@ do_install_append() {
> sed -i 's!/etc/!${sysconfdir}/!g' ${D}${sysconfdir}/init.d/ntpd ${D}${bindir}/ntpdate-sync
> sed -i 's!/var/!${localstatedir}/!g' ${D}${sysconfdir}/init.d/ntpd ${D}${bindir}/ntpdate-sync
> sed -i 's!^PATH=.*!PATH=${base_sbindir}:${base_bindir}:${sbindir}:${bindir}!' ${D}${bindir}/ntpdate-sync
> + sed -i '1s!/bin/perl!${USRBINPATH}/env perl!' ${D}${sbindir}/ntptrace
> + sed -i '1s!/bin/perl!${USRBINPATH}/env perl!' ${D}${sbindir}/ntp-wait
>
> install -d ${D}/${sysconfdir}/default
> install -m 644 ${WORKDIR}/ntpdate.default ${D}${sysconfdir}/default/ntpdate
> --
> 1.8.1.2
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ntp: fix perl path in scripts
2014-01-22 11:31 ` Martin Jansa
@ 2014-02-19 2:19 ` Kang Kai
0 siblings, 0 replies; 3+ messages in thread
From: Kang Kai @ 2014-02-19 2:19 UTC (permalink / raw)
To: openembedded-devel
On 2014?01?22? 19:31, Martin Jansa wrote:
> On Wed, Jan 22, 2014 at 03:47:38PM +0800, Kai Kang wrote:
>> Scripts ntptrace and ntp-wait use /bin/perl directly that cause
>> ntp-utils depends on /bin/perl but no package provides it. It should use
>> '${USRBINPATH}/env perl'.
> Next time please add [meta-networking] tag, but it's already in right
> patchwork bundle so that you don't need to resend it.
OK, thanks.
--Kai
>
>> Signed-off-by: Kai Kang <kai.kang@windriver.com>
>> ---
>> meta-networking/recipes-support/ntp/ntp.inc | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/meta-networking/recipes-support/ntp/ntp.inc b/meta-networking/recipes-support/ntp/ntp.inc
>> index 9ea24e9..8ada604 100644
>> --- a/meta-networking/recipes-support/ntp/ntp.inc
>> +++ b/meta-networking/recipes-support/ntp/ntp.inc
>> @@ -54,6 +54,8 @@ do_install_append() {
>> sed -i 's!/etc/!${sysconfdir}/!g' ${D}${sysconfdir}/init.d/ntpd ${D}${bindir}/ntpdate-sync
>> sed -i 's!/var/!${localstatedir}/!g' ${D}${sysconfdir}/init.d/ntpd ${D}${bindir}/ntpdate-sync
>> sed -i 's!^PATH=.*!PATH=${base_sbindir}:${base_bindir}:${sbindir}:${bindir}!' ${D}${bindir}/ntpdate-sync
>> + sed -i '1s!/bin/perl!${USRBINPATH}/env perl!' ${D}${sbindir}/ntptrace
>> + sed -i '1s!/bin/perl!${USRBINPATH}/env perl!' ${D}${sbindir}/ntp-wait
>>
>> install -d ${D}/${sysconfdir}/default
>> install -m 644 ${WORKDIR}/ntpdate.default ${D}${sysconfdir}/default/ntpdate
>> --
>> 1.8.1.2
>>
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
--
Regards,
Neil | Kai Kang
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-02-19 2:19 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-22 7:47 [PATCH] ntp: fix perl path in scripts Kai Kang
2014-01-22 11:31 ` Martin Jansa
2014-02-19 2:19 ` Kang Kai
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.