On Fri, Jan 29, 2021 at 8:37 AM Jouke Witteveen <j.wittev...@gmail.com> wrote: > 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?
Another month, another reminder. Cheers, - Jouke