Re: [PATCH 3/4] _AS_LN_S_PREPARE: default to 'cp -pR' on OS/2

Eric Blake <[email protected]>
Newsgroups gmane.comp.sysutils.autoconf.patches
Organization Red Hat, Inc.
Message-ID <[email protected]>
On 09/22/2014 12:59 AM, KO Myung-Hun wrote:
> On OS/2, any others than kLIBC have no symlinks.

Any other what?  Do you mean:

On OS/2, only kLIBC provides symlinks, and not all executables link with
kLIBC

?  Or something else?

> 
> * lib/m4sugar/m4sh.m4 (_AS_LN_S_PREPARE): Subject.
> ---
>  lib/m4sugar/m4sh.m4 |    9 +++++----
>  1 files changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/lib/m4sugar/m4sh.m4 b/lib/m4sugar/m4sh.m4
> index a94999e..8f6d184 100644
> --- a/lib/m4sugar/m4sh.m4
> +++ b/lib/m4sugar/m4sh.m4
> @@ -1189,12 +1189,13 @@ fi
>  if (echo >conf$$.file) 2>/dev/null; then
>    if ln -s conf$$.file conf$$ 2>/dev/null; then
>      as_ln_s='ln -s'
> -    # ... but there are two gotchas:
> +    # ... but there are three gotchas:
>      # 1) On MSYS, both `ln -s file dir' and `ln file dir' fail.
>      # 2) DJGPP < 2.04 has no symlinks; `ln -s' creates a wrapper executable.
> -    # In both cases, we have to default to `cp -pR'.
> -    ln -s conf$$.file conf$$.dir 2>/dev/null && test ! -f conf$$.exe ||
> -      as_ln_s='cp -pR'
> +    # 3) On OS/2, any others than kLIBC has no symlinks.
> +    # In three cases, we have to default to `cp -pR'.
> +    ln -s conf$$.file conf$$.dir 2>/dev/null && test ! -f conf$$.exe &&
> +      test -z "$OS2_SHELL" || as_ln_s='cp -pR'

Is the goal here to unconditionally use 'cp -pR' if OS2_SHELL is set?

>    elif ln conf$$.file conf$$ 2>/dev/null; then
>      as_ln_s=ln
>    else
> 

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc (application/pgp-signature, 539 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
Comment: Public key at http://people.redhat.com/eblake/eblake.gpg

iQEcBAEBCAAGBQJUIENCAAoJEKeha0olJ0NqCXAH/3km8AtHaqC73ip/4H71qW+s
paMhGLvrrR6M3ANVC7/l2AjMKB+3TSO6CXd9aGX3nhkwu8aeWrcpZsyls6YekVB+
DFOFkdBHQRbsoYWmtPUufO2uhGNTYGRLTQ+ad6hiLLN8COhCCCA7e8oiTAcFpMml
vdwEeHoekla4z2MDugBYLkFcEWLrFYjU6fDPm8G0YBeL35yC8H7Ctis+Q5AJE5zf
GHIdlWgwK+LWYvF2bLd5stjmZrQ1I7pB2a7/7A2pMHlXnfVX9BeL23MrNUVJ11ih
hM/zs0waqAar3RloR6kuXQ7F6s7bqi5lmCpAIi2NqI1LwsrIv7Iga4lzh+m0pmg=
=qqBG
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.