All XULRunner files have the executable bit set

Question

All XULRunner files have the executable bit set

Closed this issue 10 years ago · 7 comments

Apparently all the files in the XULRunner dep have the executable bit set. I don't know if it's required, however it causes warnings in the debian package. If this is wanted we can change the executable bit when building the debian package.

Answer 1 · 2014-02-25T09:13:43.000Z

I think it's required for libraries and binaries, better look at documentation to be sure.

Answer 2 · 2014-02-25T09:49:10.000Z

Yes, but not for CSS files and similar.

Answer 3 · 2014-02-25T18:44:23.000Z

FWIW the permissions on the xulrunner files are changed here: make-xulrunner-tarball.sh#L42

Answer 4 · 2014-02-25T21:21:05.000Z

I don't know what it exactlty does, but there's also a statement which sets some files to 644, which is the desired setting for non-executables.

Answer 5 · 2014-03-06T09:20:24.000Z

@AntoineTurmel On GNU/Linux shared libraries should be set to non-executable/0644. http://lintian.debian.org/tags/shlib-with-executable-bit.html

ld.so seems to be the only exception, as described in the lintian tag overrides file:

ld.so must be executable, otherwise the system will not work

libc6: shlib-with-executable-bit */ld-2.17.so 0755

Answer 6 · 2014-06-03T19:36:59.000Z

I think this issue is because it could be tedious to go through and set the permissions individually.
make-xulrunner-tarball.sh gets called from xr_pack_deps.sh#L33.
There must be a reason the permissions are being "fixed" in the first place, but the git history doesn't help.

Answer 7 · 2014-07-17T19:18:43.000Z

I think this should be fixed by d752333