[Buildroot] [PATCH] Config.in: add BR2_DL_DIR_OPTS

Yann E. MORIN yann.morin.1998 at free.fr
Thu Nov 1 11:18:55 UTC 2018


Thomas, All,

On 2018-11-01 12:01 +0100, Thomas Petazzoni spake thusly:
> On Thu, 1 Nov 2018 10:06:16 +0100, Yann E. MORIN wrote:
> > On 2018-10-31 14:39 -0700, Markus Mayer spake thusly:
> > > Provide a simple mechanism to pass extra arguments to "mkdir" when
> > > creating download directories. This can be helpful if one needs the
> > > download directories to be writable multiple users ("shared download
> > > cache").  
> > 
> > In retrospect, I think our enforcing of umask globally is wrong.
> > 
> > It was added in commit bee5745ccc2 (Makefile: don't depend on the umask)
> > under the requirements that modes in target/ be predicatble.
> > 
> > Although that is indeed a good reason, I think we should have instead
> > ensured those modes with a plain chmod on target/ during the
> > target-finalize step.
> 
> But is it possible to have a plain "chmod" on target/ that will
> properly preserve whatever packages have decided to use as permissions
> when installing files ?

Right, but specifying modes at install time is not reliable anyway, as
it is not running root. The reason for explicitly setting permissions at
install time is mostly due to packages wanting to set suid bits, which
can't work. There can be outliers too, but they mostly are corner cases
which should be fixed not to set permissions.

So packages should instead provide FOO_PERMISSIONS, or users should use
permissions tables.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'



More information about the buildroot mailing list