Re: MinGw port of gawkextlib

[Andrew J. Schorr]

On Thu, May 12, 2022 at 12:53:33PM +0200, Manuel Collado wrote:
> Ok. I was wrong. Not a configure issue.

OK. Good.

> But porting to Windows required patching some files used by
> autotools. In particular "test/test.makefile", included by
> "test/Makefile.am". And then autotools had to be rerun to regenerate
> "configure" and "Makefile.in".
> 
> ---------------------
> $ diff original/gawk-mpfr-1.1.0/test/test.makefile
> code/gawk-mpfr-1.1.0/test/test.makefile
> 1c1
> < CMP = cmp
> ---
> > CMP = diff -qa --strip-trailing-cr
> 25c25
> < AWK = LC_ALL=$${GAWKLOCALE:-C} LANG=$${GAWKLOCALE:-C}
> AWKLIBPATH=../.libs:`$(GAWKPROG) 'BEGIN {print
> ENVIRON["AWKLIBPATH"]}'` PATH=$(GEBINPFX)$$PATH
> LD_LIBRARY_PATH=$(GELIBPFX)$$LD_LIBRARY_PATH
> DYLD_LIBRARY_PATH=$(GELIBPFX)$$DYLD_LIBRARY_PATH $(VALGRIND)
> $(GAWKPROG)
> ---
> > AWK = LC_ALL=$${GAWKLOCALE:-C} LANG=$${GAWKLOCALE:-C}
> AWKLIBPATH=../.libs PATH=$(GEBINPFX)$$PATH
> LD_LIBRARY_PATH=$(GELIBPFX)$$LD_LIBRARY_PATH
> DYLD_LIBRARY_PATH=$(GELIBPFX)$$DYLD_LIBRARY_PATH $(VALGRIND)
> $(GAWKPROG)
> 37c37
> <       for i in _* ; \
> ---
> >       @for i in _* ; \
> 47c47
> <       done | more
> ---
> >       done | (more || less)
> ----------------------

Your CMP patch should work on all platforms, so we can just adopt that.

And I guess you're indicating with the diffout patch that "more" is not
available on MinGW; I'd just drop the more and let the user decide if he needs
a pager.

What's going on in the middle part of the patch: it seems to be removing
the modification to AWKLIBPATH to append

`$(GAWKPROG) 'BEGIN {print ENVIRON["AWKLIBPATH"]}'`

What's the issue there?

Thanks,
Andy