[haiku-commits] Re: haiku: hrev51182 - in src: system/kernel/fs bin/multiuser system/runtime_loader

  • From: waddlesplash <waddlesplash@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sat, 27 May 2017 23:58:05 -0400

On Sat, May 27, 2017 at 3:05 PM, Ingo Weinhold <ingo_weinhold@xxxxxx> wrote:

Firstly, I don't see why the runtime loader would need execute permission.
It is not meant to be executed and AFAICT the permission isn't
checked/required anywhere.

The ticket discussion and ttcoder's testing seemed to indicate that
this solved the problem -- probably not because of the exemode
specifically, but because it sets the bits for all users, indeed.

Secondly, it isn't exactly a huge amount of work to have the package tool
adjust the permissions of added files and directories. It means adding a
switch to the tool ("create" and "add" commands), propagating the value to
the code responsible (i.e. add a flag [1] and use it), and modifying the
code that currently just copies the permission values to adjust them [2].

If no one cares to implement a proper solution, the issue can't be that
pressing. Given that a fairly simple work-around is to build with a less
restrictive umask, I don't see a good reason to add a hack-around. Those
temporary band-aid solutions tend to stay forever, so I'm generally wary of
introducing them.

I think the buildbots have a more restrictive umask, hence this
problem. I'll talk to ttcoder and see what's up on his end, and make a
note to take a closer look at this asap. Thanks for the info!

-waddlesplash

Other related posts: