Re: mkdir -p -- minor cleanup

Hi Paul, * Paul Eggert wrote on Mon, Apr 24, 2006 at 10:36:43PM CEST: > > If I understand you and Ralf correctly about race conditions, here is > a proposed patch to should fix the condition on POSIX conforming > hosts. On older hosts there's no hope that I can see in general, but > it's not wor

Re: mkdir -p -- minor cleanup

Stepan Kasal <[EMAIL PROTECTED]> writes: > All my complaints were considered, and we have Alexandre's approval, > so I guess you could check it in. OK, thanks, I did that. If I understand you and Ralf correctly about race conditions, here is a proposed patch to should fix the condition on POSIX

Re: mkdir -p -- minor cleanup

Hello Ralf, On Sat, Apr 22, 2006 at 08:44:06PM +0200, Ralf Wildenhues wrote: > It was already noted that `-m MODE' is understood by mkdir, by > install_sh, and by mkinstalldirs (the latter would not be used > any more, I'm just noting it here). > > However, we can't be sure that there is no race

Re: mkdir -p -- minor cleanup

* Stepan Kasal wrote on Wed, Apr 19, 2006 at 01:08:57PM CEST: > > And $(mkdir_p) cannot be used with an option anyway. It was already noted that `-m MODE' is understood by mkdir, by install_sh, and by mkinstalldirs (the latter would not be used any more, I'm just noting it here). However, we can

Re: mkdir -p -- minor cleanup

Hello Paul. On Wed, Apr 19, 2006 at 04:18:04PM -0700, Paul Eggert wrote: > Stepan Kasal <[EMAIL PROTECTED]> writes: > > And $(mkdir_p) cannot be used with an option anyway. > > But under the patch I proposed, "$(mkdir_p) -m 444" would be allowed, > for example. (Well, not ``for example'', -m is

Re: mkdir -p -- minor cleanup

Stepan Kasal <[EMAIL PROTECTED]> writes: > I think it is more practical to have $(mkdir_p) set to 'mkdir -p --'. Yes, that is a possibility. > And you can often write > $(mkdir_p) $(directory_list) > without knowing exactly the names of the directories. It's natural to > forget that the d

Re: mkdir -p -- minor cleanup

Hello, On Wed, Apr 19, 2006 at 01:02:30AM -0700, Paul Eggert wrote: > [...], but now that > $(mkdir_p) is intended to mean 'mkdir -p' I think it's more consistent > for the expansion to not include the '--'. I think it is more practical to have $(mkdir_p) set to 'mkdir -p --'. And you can often

mkdir -p -- minor cleanup

In reviewing Bruno Haible's proposed patch to move $(mkdir_p) from Automake to Autoconf I noticed an inconsistency in how $(mkdir_p) is set. Normally it is set to 'mkdir -p --', but it might be set to '$(install_sh) -d' or to '