Re: [Automake-NG] [PATCH 26/26] [ng] texi: remove some crufty code to support obsolete environment

[Akim Demaille]

Le 16 juin 2012 à 23:30, Stefano Lattarini a écrit :

> * lib/am/texi-vers.am (%STAMPVTI%): We don't really care anymore about
> older systems whose 'mv' program can't move across file systems, nor do
> we care about past problems of GNU mv in the AmigaDOS environment.  So
> simplify the code accordingly.
> 
> Signed-off-by: Stefano Lattarini <address@hidden>
> ---
> lib/am/texi-vers.am |   12 +++++-------
> 1 file changed, 5 insertions(+), 7 deletions(-)
> 
> diff --git a/lib/am/texi-vers.am b/lib/am/texi-vers.am
> index ec2f366..1148c50 100644
> --- a/lib/am/texi-vers.am
> +++ b/lib/am/texi-vers.am
> @@ -33,13 +33,11 @@ am__dist_common += %VTEXI% %STAMPVTI%
>       echo "@set UPDATED-MONTH $$2 $$3"; \
>       echo "@set EDITION $(VERSION)"; \
>       echo "@set VERSION $(VERSION)") > %VTI%.tmp
> -## Use cp and rm here because some older "mv"s can't move across
> -## filesystems.  Furthermore, GNU "mv" in the AmigaDOS environment
> -## can't handle this.
> -     @cmp -s %VTI%.tmp %VTEXI% \
> -       || (echo "Updating %VTEXI%"; \
> -           cp %VTI%.tmp %VTEXI%)
> -     address@hidden -f %VTI%.tmp
> +     @if cmp -s %VTI%.tmp %VTEXI%; then \
> +       echo "Updating %VTEXI%" && cp %VTI%.tmp %VTEXI%; \

You mean mv -f I guess.

I don't see what %VTEXI% is, but I would bet that address@hidden, or something
like that, would suffice.

> +     else \
> +       rm -f %VTI%.tmp;
> +     fi;
>       @cp %VTEXI% $@
> 
> mostlyclean-am: mostlyclean-%VTI%
> -- 
> 1.7.9.5
> 
>