Scripts in uInitrd loose execute permissions

[Tixy (Jon Medhurst)]

vexpress recently added a shell scripts for inclusion into the initrd to be executed at boot (See change http://review.android.git.linaro.org/#change,3240)

This script has the execute permission bit set in git but by the time it ends up in the uInitrd the execute permissions are lost.
I see that after a build has finished, the copy generated in the 'root' directory (out/target/product/vexpress/root) has the correct execute permissions so it seems the problem is in the step where the uInitrd is created.

[Bernhard Rosenkraenzer (berolinux)]

This is probably related to the way Android handles filesystem creation (probably done that way to support building on certain retarded wannabe operating systems that don't have proper permissions in their filesystems).

What files get what permissions is defined in system/core/include/private/android_filesystem_config.h

[Tixy (Jon Medhurst) (tixy)]

On Mon, 2013-01-14 at 18:09 +0000, Bernhard Rosenkraenzer wrote:
> What files get what permissions is defined in
> system/core/include/private/android_filesystem_config.h

Thanks for that vital clue :-)

Think it would probably be best to put the script in /sbin/ then?
(Or rename it to match init*)

--
Tixy

[Bernhard Rosenkraenzer (berolinux)]

Both of those should work -- another option (along the same lines) would be to patch android_filesystem_config.h to make *.sh executable, given that's what shell scripts typically are.

All 3 approaches work for me and I don't have a particular preference.

[Tixy (Jon Medhurst) (tixy)]

On Mon, 2013-01-14 at 21:59 +0000, Bernhard Rosenkraenzer wrote:
> Both of those should work -- another option (along the same lines) would
> be to patch android_filesystem_config.h to make *.sh executable

I thought of that but the wildcard matching looks like its only
supported at the end of a filename, not the start, as required for
'*.sh'

[Tixy (Jon Medhurst) (tixy)]

Fixed by moving set_irq_affinity.sh to /sbin/ See: http://review.android.git.linaro.org/#change,3247