HomePhorge

Fix for #3556 Plussignment value melding
b0e3c61a8b3bUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Fix for #3556 Plussignment value melding

The plussignment operator was constructing the new parameter value by
modifying the param object's value in place (so as to preserve the file
and line information for debugging). However, when multiple resources
are overridden by the same plussignment this would result in all of the
resources sharing the same value (the union of all the prior values and
the new value), which is wrong.

Instead, we need to give each resource its own copy of the value (e.g.,
a copy of the param object), which this patch implements.

Signed-off-by: Markus Roberts <Markus@reality.com>

Details

Provenance
Markus Roberts <Markus@reality.com>Authored on
James Turnbull <james@lovedthanlost.net>Committed on May 2 2010, 3:39 AM
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPU8a3049591a29: Fixed #3655 - Puppet doesn't find installed packages with portage provider
Branches
Unknown
Tags
Unknown

Event Timeline

James Turnbull <james@lovedthanlost.net> committed rPUb0e3c61a8b3b: Fix for #3556 Plussignment value melding (authored by Markus Roberts <Markus@reality.com>).May 2 2010, 3:39 AM