Re: [PATCH] init-install-efi.sh: Avoid /mnt/mtab creation if already present

[Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>]

Tests were done in the same manner as described in the bugzilla 
(core-image-sato-sdk image and genericx86-64 machine). In my case, the 
only HW I tested was the minnowboard. The testing consists of booting a 
live image and installing the image. After this patch, installation went 
fine.


On 08/21/2015 03:02 PM, Benjamin Esquivel wrote:
> Hi Leo, this fix looks good to me, can you mention how did you test
> this?
>
> On Mon, 2015-08-03 at 15:01 +0000,
> leonardo.sandoval.gonzalez@linux.intel.com wrote:
>> From: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
>>
>> The base-files recipe installs /mnt/mtab (it is a softlink of
>> /proc/mounts),
>> so if an image includes the latter, there is no new to created it
>> again inside
>> the install-efi.sh script, otherwise an error may occur as indicated
>> on the
>> bug's site.
>>
>> [YOCTO #7971]
>>
>> Signed-off-by: Leonardo Sandoval <
>> leonardo.sandoval.gonzalez@linux.intel.com>
>> ---
>>   meta/recipes-core/initrdscripts/files/init-install-efi.sh | 6 +++++-
>>   1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-core/initrdscripts/files/init-install
>> -efi.sh b/meta/recipes-core/initrdscripts/files/init-install-efi.sh
>> index f339b30..665d04a 100644
>> --- a/meta/recipes-core/initrdscripts/files/init-install-efi.sh
>> +++ b/meta/recipes-core/initrdscripts/files/init-install-efi.sh
>> @@ -109,7 +109,11 @@ rm -f /etc/udev/scripts/mount*
>>   umount ${device}* 2> /dev/null || /bin/true
>>
>>   mkdir -p /tmp
>> -cat /proc/mounts > /etc/mtab
>> +
>> +# Create /etc/mtab if not present
>> +if [ ! -e /etc/mtab ]; then
>> +    cat /proc/mounts > /etc/mtab
>> +fi
>>
>>   disk_size=$(parted ${device} unit mb print | grep Disk | cut -d" "
>> -f 3 | sed -e "s/MB//")
>>
>> --
>> 1.8.4.5
>>