[OE-core] [PATCH] runqemu: Ensure ROOTFS path is absolute

[Alexandru Palalau]

On 2/22/2014 6:34 PM, Khem Raj wrote:
>
> On Feb 22, 2014, at 4:55 AM, Paul Eggleton <paul.eggleton at linux.intel.com> wrote:
>
>> On Friday 21 February 2014 21:57:37 Khem Raj wrote:
>>> On Feb 20, 2014, at 12:57 PM, Saul Wold <sgw at linux.intel.com> wrote:
>>>> There is a problem if a relative path is passed to the kernel for NFS
>>>> usage
>>>> that it will not correctly find it, so ensure that the ROOTFS path is
>>>> absolute.
>>>>
>>>> [YOCTO #2807]
>>>>
>>>> Signed-off-by: Saul Wold <sgw at linux.intel.com>
>>>> ---
>>>> scripts/runqemu | 4 +++-
>>>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/scripts/runqemu b/scripts/runqemu
>>>> index 0db7ad6..57c5de4 100755
>>>> --- a/scripts/runqemu
>>>> +++ b/scripts/runqemu
>>>> @@ -462,7 +462,9 @@ if [ -z "$ROOTFS" -a "x$FSTYPE" != "xvmdk" ]; then
>>>>
>>>>         error "Unable to determine default rootfs for MACHINE [$MACHINE]"
>>>>
>>>>     fi
>>>>
>>>> fi
>>>> -# ROOTFS is now set for all cases
>>>> +# ROOTFS is now set for all cases, now expand it to be an absolute path,
>>>> it should exist at this point +
>>>> +ROOTFS=`realpath $ROOTFS`
>>>
>>> I think this change also means we need to add dependency on realpath package
>>> as prerequisite for build host.
>>
>> realpath is part of coreutils.
>
> realpath is package of its own not coreutils, at least on ubuntu

On CentOS 6.5 realpath is not part of coreutils and is not part of a 
default repo. I could find it only on RPMforge.

>
>> Another alternative would be readlink -f (but
>> that also appears to be part of coreutils.)
>
>
> this might be better options since coreutils is more or less
> given on any distro
>
>>
>> Cheers,
>> Paul
>>
>> --
>>
>> Paul Eggleton
>> Intel Open Source Technology Centre
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>


-- 
Alexandru Palalau