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

[Stefano Lattarini]

On 06/17/2012 09:11 AM, Akim Demaille wrote:
> 
> 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.
>
Oops, fixed.  Thanks for spotting.

> I don't see what %VTEXI% is,
>
It's (usually) the auto-generated 'version.texi' file that is included in
the Texinfo manual; for example, for my Automake-NG:

  $ cat doc/version.texi
  @set UPDATED 15 June 2012
  @set UPDATED-MONTH June 2012
  @set EDITION 1.12a
  @set VERSION 1.12a

> but I would bet that address@hidden, or something like that, would suffice.
>
I don't see how; %VTEXI% must be a permanent, distributed file.  Or did
you mean '%VTI%' instead of '%VTEXI%'?

Stefano