On Fri, Dec 25, 2020 at 7:00 PM Jouke Witteveen <j.wittev...@gmail.com> wrote: > > * NEWS: Use "local" instead of the incorrect "lexically-scoped". > * doc/make.texi: Refer to let/foreach variables as local variables. > --- > This is an erratum on the addition of $(let ...). During an early review > of $(let ...), thutt cautioned that it did not implement "full semantic > scoping" [sic]. While I did not understand fully what they meant by > that, I countered that it was not intended to determine a scope based on > for example which file a definition of a rule occurred in, but simply by > the parentheses delimiting the let expression. Only in that sense was it > lexical scoping. Technically, make variables are dynamically scoped. > > This patch replaces "lexically scoped" not by "dynamically scoped", but > by "local", since that is the whole point after all. It also includes > variables with local scope (from let and foreach) in several other > places where variables are discussed, and makes explicit that variables > in make are dynamically scoped.
I think this is ready to go in and that it would be wise to not release with the unchanged NEWS. Was this patch simply missed due to end-of-year activities? Regards, - Jouke
