Re: [EXTERNAL] Re: [PATCH] Cygwin: spawn: Treat empty path as the current directory.

2022-06-30 Thread Takashi Yano
On Thu, 30 Jun 2022 18:35:04 +
"Lavrentiev, Anton \(NIH/NLM/NCBI\) \[C\] wrote:
> >However, use of this feature is deprecated, and POSIX
> >notes that a conforming application shall use an explicit
> >pathname (e.g., .)  to specify the current working
> >directory.
> 
> Since "SHALL" does not mean "MUST", I think this patch is correct.

In the POSIX standard, "SHALL" is used almost interchangeably
with "MUST" as other standard documents do.

https://pubs.opengroup.org/onlinepubs/9699919799/xrat/V4_xbd_chap01.html#tag_21_01_09

-- 
Takashi Yano 


Re: [EXTERNAL] Re: [PATCH] Cygwin: spawn: Treat empty path as the current directory.

2022-06-30 Thread Brian Inglis



On 2022-06-30 12:35, Lavrentiev, Anton (NIH/NLM/NCBI) [C] via 
Cygwin-patches wrote:

However, use of this feature is deprecated, and POSIX notes that a
conforming application shall use an explicit pathname (e.g., .)  to
specify the current working directory.



Since "SHALL" does not mean "MUST", I think this patch is correct.


It appears you may be confusing POSIX's (1.5 Terminology)
*shall* (mandatory) and *should* (recommended):

"*SHALL*

For an implementation that conforms to POSIX.1-2017, describes a feature 
or behavior that is mandatory. An application can rely on the existence 
of the feature or behavior.


For an application or user, describes a behavior that is mandatory.

*SHOULD*

For an implementation that conforms to POSIX.1-2017, describes a feature 
or behavior that is recommended but not mandatory. An application should 
not rely on the existence of the feature or behavior. An application 
that relies on such a feature or behavior cannot be assured to be 
portable across conforming implementations.


For an application, describes a feature or behavior that is recommended 
programming practice for optimum portability."


--
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada

This email may be disturbing to some readers as it contains
too much technical detail. Reader discretion is advised.
[Data in binary units and prefixes, physical quantities in SI.]


RE: [EXTERNAL] Re: [PATCH] Cygwin: spawn: Treat empty path as the current directory.

2022-06-30 Thread Lavrentiev, Anton (NIH/NLM/NCBI) [C] via Cygwin-patches
>However, use of this feature is deprecated, and POSIX
>notes that a conforming application shall use an explicit
>pathname (e.g., .)  to specify the current working
>directory.

Since "SHALL" does not mean "MUST", I think this patch is correct.

Anton Lavrentiev
Contractor NIH/NLM/NCBI