On Fri, Aug 1, 2014 at 12:56 AM, Gary Thomas <[email protected]> wrote:
> On 2014-07-31 16:46, Andrea Adami wrote:
>>
>> Signed-off-by: Andrea Adami <[email protected]>
>> ---
>>   meta-initramfs/recipes-bsp/kexecboot/kexecboot_0.6.bb | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta-initramfs/recipes-bsp/kexecboot/kexecboot_0.6.bb
>> b/meta-initramfs/recipes-bsp/kexecboot/kexecboot_0.6.bb
>> index d0176c5..1fb5f0d 100644
>> --- a/meta-initramfs/recipes-bsp/kexecboot/kexecboot_0.6.bb
>> +++ b/meta-initramfs/recipes-bsp/kexecboot/kexecboot_0.6.bb
>> @@ -48,7 +48,7 @@ EXTRA_OECONF = "--enable-delay=2
>> --enable-evdev-rate=1000,250"
>>
>>   do_install () {
>>       install -d ${D}${bindir}
>> -    install -m 0755 kexecboot ${D}${bindir}/
>> +    install -m 0755 kexecboot ${D}${bindir}
>>       install -d ${D}/proc
>>       install -d ${D}/mnt
>>       install -d ${D}/dev
>>
>
> What is the reason for this change.  Using XXX/ is quite common
> in OE recipes.
>

Hi Gary,

thanks for reviewing, I recently spotted this slash and imho is just
one extra unnecessary char.

About the 'best practice' here, the POSIX synopsis seems not
describing an ending slash for <target>
http://pubs.opengroup.org/onlinepubs/9699919799/utilities/cp.html

Cheers

Andrea


> --
> ------------------------------------------------------------
> Gary Thomas                 |  Consulting for the
> MLB Associates              |    Embedded world
> ------------------------------------------------------------
> --
> _______________________________________________
> Openembedded-devel mailing list
> [email protected]
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
-- 
_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Reply via email to