Re: [PATCH 1/2] More parallel make -j4 fixes for txt

Hi Reini,

>     README, README.de, README.es, clisp.c, ... all clash
>     with parallel builds using the same intermediate
>     txt binaries and txt.c sources.
>     Rename them.
> 
>     Use a $(( $i + 1 )) shell function, which should be portable across
>     all shells.
>     See https://github.com/rurban/clisp/issues/1

Thanks, I applied this with modifications:
  1. No, $(( $i + 1 )) is not portable shell syntax. The portable shell
     syntax is `expr $i + 1`.
     For portable shell syntax, always reference the chapter
     "Portable Shell Programming" [1].
  2. Here too, I like a more descriptive name: 'gen-foo.c', not 'txt4.c'.
     (The original code did not do this, because it was bound to Windows 8+3
     file name limitations, and 'gen-README.de.c' was an invalid file name
     in that environment.)

Bruno

[1] https://www.gnu.org/software/autoconf/manual/autoconf.html#Portable-Shell


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
clisp-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/clisp-devel