[pacman-dev] [PATCH 05/11] makepkg: undeclared local variables
aepd87 at gmail.com
Thu Jun 17 09:33:13 EDT 2010
On Thu, Jun 17, 2010 at 8:55 AM, Allan McRae <allan at archlinux.org> wrote:
> On 17/06/10 22:44, Andres P wrote:
>> Tight variable scoping should avoid further regressions with new patches
>> variable overriding (see what ac5c2fd09 fixed).
> Variable scoping had nothing to do with what ac5c2fd09 fixed. Maybe you
> mean another commit?
It had everything to do with it. Functions are big as it is, meaning
the namespace is pretty cramped.
By chance there was a variable named changelog and install in scope.
Do remind that the original suggestion was to make that a seperate
function, giving it its own set of local variables.
More information about the pacman-dev