Re: [PATCH build/] Fix mkyaffs2image call to add a '/' to the mount point

[Joshua Brindle <method@manicmethod.com>]

Note, I have no idea why this just started happening but it is 
consistent on my 4.1.1_r3 checkout. I don't know what the ramifications 
of changing the mountpoint are (e.g., is that used by something other 
than selabel_lookup?) so I haven't submitted it to AOSP.

Joshua Brindle wrote:
> From: Joshua Brindle<jbrindle@tresys.com>
>
> selabel_lookup requires the mountpoint start with a '/' to match specifications in the file_context file. Change mkyaffs2image call to add a '/' to the mount point argument
>
> Change-Id: I255b661822401f9253ef58a2a4bdada85fc96b23
>
> Signed-off-by: Joshua Brindle<jbrindle@tresys.com>
> ---
>   tools/releasetools/build_image.py |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/releasetools/build_image.py b/tools/releasetools/build_image.py
> index a615d1a..610e634 100755
> --- a/tools/releasetools/build_image.py
> +++ b/tools/releasetools/build_image.py
> @@ -56,7 +56,7 @@ def BuildImage(in_dir, prop_dict, out_file):
>       build_command.append(out_file)
>       if "selinux_fc" in prop_dict:
>         build_command.append(prop_dict["selinux_fc"])
> -      build_command.append(prop_dict["mount_point"])
> +      build_command.append("/%s" % prop_dict["mount_point"])
>
>     print "Running: ", " ".join(build_command)
>     p = subprocess.Popen(build_command);

--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.