There has already been a patch merged with the master branch (external/yaffs2 bd009e4d493deaf90c38da1cdbaa1db69db073ba). For now I would cherry-pick. I'll see what we can do about getting that patch applied to the other branches as well.
On Wed, Aug 1, 2012 at 10:39 PM, Joshua Brindle <method@xxxxxxxxxxxxxxx> wrote:
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@xxxxxxxxxx>
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@xxxxxxxxxx>
---
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@xxxxxxxxxxxxx with
the words "unsubscribe selinux" without quotes as the message.
