public inbox for [email protected]  
help / color / mirror / Atom feed
Fir mkdir command in "Short Version" for installing from source
5+ messages / 3 participants
[nested] [flat]

* Fir mkdir command in "Short Version" for installing from source
@ 2022-03-13 16:05 Daniel Westermann (DWE) <[email protected]>
  2022-03-14 09:08 ` Re: Fir mkdir command in "Short Version" for installing from source Daniel Gustafsson <[email protected]>
  0 siblings, 1 reply; 5+ messages in thread

From: Daniel Westermann (DWE) @ 2022-03-13 16:05 UTC (permalink / raw)
  To: [email protected] <[email protected]>

Hi,

if you follow that example step by step, the mkdir command will fail, obviously:
https://www.postgresql.org/docs/current/install-short.html

Is it worth fixing that with "mkdir -p ..."? 

Regards
Daniel




^ permalink  raw  reply  [nested|flat] 5+ messages in thread

* Re: Fir mkdir command in "Short Version" for installing from source
  2022-03-13 16:05 Fir mkdir command in "Short Version" for installing from source Daniel Westermann (DWE) <[email protected]>
@ 2022-03-14 09:08 ` Daniel Gustafsson <[email protected]>
  2022-03-14 14:01   ` Re: Fir mkdir command in "Short Version" for installing from source Tom Lane <[email protected]>
  0 siblings, 1 reply; 5+ messages in thread

From: Daniel Gustafsson @ 2022-03-14 09:08 UTC (permalink / raw)
  To: Daniel Westermann (DWE) <[email protected]>; +Cc: [email protected] <[email protected]>

> On 13 Mar 2022, at 17:05, Daniel Westermann (DWE) <[email protected]> wrote:

> if you follow that example step by step, the mkdir command will fail, obviously:
> https://www.postgresql.org/docs/current/install-short.html
> 
> Is it worth fixing that with "mkdir -p ..."? 

Well, it will fail unless the user already had /usr/local/pgsql/, but I hear
what you're saying.  Though I bet almost anyone following this will be able to
figure it out but there is little to no value in not being correct.

--
Daniel Gustafsson		https://vmware.com/






^ permalink  raw  reply  [nested|flat] 5+ messages in thread

* Re: Fir mkdir command in "Short Version" for installing from source
  2022-03-13 16:05 Fir mkdir command in "Short Version" for installing from source Daniel Westermann (DWE) <[email protected]>
  2022-03-14 09:08 ` Re: Fir mkdir command in "Short Version" for installing from source Daniel Gustafsson <[email protected]>
@ 2022-03-14 14:01   ` Tom Lane <[email protected]>
  2022-03-14 14:28     ` Re: Fir mkdir command in "Short Version" for installing from source Daniel Gustafsson <[email protected]>
  0 siblings, 1 reply; 5+ messages in thread

From: Tom Lane @ 2022-03-14 14:01 UTC (permalink / raw)
  To: Daniel Gustafsson <[email protected]>; +Cc: Daniel Westermann (DWE) <[email protected]>; [email protected] <[email protected]>

Daniel Gustafsson <[email protected]> writes:
> On 13 Mar 2022, at 17:05, Daniel Westermann (DWE) <[email protected]> wrote:
>> Is it worth fixing that with "mkdir -p ..."? 

> Well, it will fail unless the user already had /usr/local/pgsql/, but I hear
> what you're saying.  Though I bet almost anyone following this will be able to
> figure it out but there is little to no value in not being correct.

I think that at the time that was written, mkdir didn't universally
have a "-p" option.  But POSIX has required that for a very long
time, so it's probably safe to assume it now.

A larger issue is that blind "mkdir -p" may not produce the ownerships
and permissions you want for the parent directories.  That's not a
topic to get into in a one-line summary, though.

			regards, tom lane





^ permalink  raw  reply  [nested|flat] 5+ messages in thread

* Re: Fir mkdir command in "Short Version" for installing from source
  2022-03-13 16:05 Fir mkdir command in "Short Version" for installing from source Daniel Westermann (DWE) <[email protected]>
  2022-03-14 09:08 ` Re: Fir mkdir command in "Short Version" for installing from source Daniel Gustafsson <[email protected]>
  2022-03-14 14:01   ` Re: Fir mkdir command in "Short Version" for installing from source Tom Lane <[email protected]>
@ 2022-03-14 14:28     ` Daniel Gustafsson <[email protected]>
  2022-03-14 14:54       ` Re: Fir mkdir command in "Short Version" for installing from source Daniel Westermann (DWE) <[email protected]>
  0 siblings, 1 reply; 5+ messages in thread

From: Daniel Gustafsson @ 2022-03-14 14:28 UTC (permalink / raw)
  To: Tom Lane <[email protected]>; +Cc: Daniel Westermann (DWE) <[email protected]>; [email protected] <[email protected]>

> On 14 Mar 2022, at 15:01, Tom Lane <[email protected]> wrote:
> 
> Daniel Gustafsson <[email protected]> writes:
>> On 13 Mar 2022, at 17:05, Daniel Westermann (DWE) <[email protected]> wrote:
>>> Is it worth fixing that with "mkdir -p ..."? 
> 
>> Well, it will fail unless the user already had /usr/local/pgsql/, but I hear
>> what you're saying.  Though I bet almost anyone following this will be able to
>> figure it out but there is little to no value in not being correct.
> 
> I think that at the time that was written, mkdir didn't universally
> have a "-p" option.  But POSIX has required that for a very long
> time, so it's probably safe to assume it now.

Yeah, I wasn't able to find any credibly still useable system man page which
didn't list -p.  (the fact that IRIX seems to have been one of very few to have
a long option '--parents' was an interesting archeological tidbit learned.)

> A larger issue is that blind "mkdir -p" may not produce the ownerships
> and permissions you want for the parent directories.  That's not a
> topic to get into in a one-line summary, though.

Agreed, we're skipping over lots of details in this short version.

I'll to make the mkdir -p change happen later today unless someone feels
strongly about leaving it out.

--
Daniel Gustafsson		https://vmware.com/






^ permalink  raw  reply  [nested|flat] 5+ messages in thread

* Re: Fir mkdir command in "Short Version" for installing from source
  2022-03-13 16:05 Fir mkdir command in "Short Version" for installing from source Daniel Westermann (DWE) <[email protected]>
  2022-03-14 09:08 ` Re: Fir mkdir command in "Short Version" for installing from source Daniel Gustafsson <[email protected]>
  2022-03-14 14:01   ` Re: Fir mkdir command in "Short Version" for installing from source Tom Lane <[email protected]>
  2022-03-14 14:28     ` Re: Fir mkdir command in "Short Version" for installing from source Daniel Gustafsson <[email protected]>
@ 2022-03-14 14:54       ` Daniel Westermann (DWE) <[email protected]>
  0 siblings, 0 replies; 5+ messages in thread

From: Daniel Westermann (DWE) @ 2022-03-14 14:54 UTC (permalink / raw)
  To: Daniel Gustafsson <[email protected]>; Tom Lane <[email protected]>; +Cc: [email protected] <[email protected]>

>> On 14 Mar 2022, at 15:01, Tom Lane <[email protected]> wrote:
>> 
>> Daniel Gustafsson <[email protected]> writes:
>>> On 13 Mar 2022, at 17:05, Daniel Westermann (DWE) <[email protected]> wrote:
>>>> Is it worth fixing that with "mkdir -p ..."? 
>> 
>>> Well, it will fail unless the user already had /usr/local/pgsql/, but I hear
>>> what you're saying.  Though I bet almost anyone following this will be able to
>>> figure it out but there is little to no value in not being correct.
>> 
>> I think that at the time that was written, mkdir didn't universally
>> have a "-p" option.  But POSIX has required that for a very long
>> time, so it's probably safe to assume it now.

>Yeah, I wasn't able to find any credibly still useable system man page which
>didn't list -p.  (the fact that IRIX seems to have been one of very few to have
>a long option '--parents' was an interesting archeological tidbit learned.)

>> A larger issue is that blind "mkdir -p" may not produce the ownerships
>> and permissions you want for the parent directories.  That's not a
>> topic to get into in a one-line summary, though.

>Agreed, we're skipping over lots of details in this short version.

>I'll to make the mkdir -p change happen later today unless someone feels
>strongly about leaving it out.

Thank you Daniel and Tom
Regards
Daniel





^ permalink  raw  reply  [nested|flat] 5+ messages in thread


end of thread, other threads:[~2022-03-14 14:54 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2022-03-13 16:05 Fir mkdir command in "Short Version" for installing from source Daniel Westermann (DWE) <[email protected]>
2022-03-14 09:08 ` Daniel Gustafsson <[email protected]>
2022-03-14 14:01   ` Tom Lane <[email protected]>
2022-03-14 14:28     ` Daniel Gustafsson <[email protected]>
2022-03-14 14:54       ` Daniel Westermann (DWE) <[email protected]>

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox