Remove (useless) s/*/__star__/ replacement in getconf.
authorMatthijs Kooijman <matthijs@stdin.nl>
Wed, 17 Mar 2010 21:44:09 +0000 (22:44 +0100)
committerMatthijs Kooijman <matthijs@stdin.nl>
Fri, 19 Mar 2010 20:40:15 +0000 (21:40 +0100)
commit7d31791c5b913df3d2c330b51cf829dacda111ab
tree0b062329001e9b0471c148fc0177d68904cbe182
parent2e919339435fb77e0b7cf20f2f6e0bee4005b1ee
Remove (useless) s/*/__star__/ replacement in getconf.

It seems this was meant to replace * with __star__ to prevent globbing,
but getconf already handles its values properly so globbing does not
occur. Currently, this replacement was broken due to the double
backslash, causing it to replace a backslash and all following
characters with __star__, instead of a literal *. This is also confirmed
by the fact that some handlers' documentation says to use __star__
instead of * directly in the configuration (and they have special
handling to turn __star__ back into * or something else as appropriate).

Eventually, we should just support using * directly and fixing up all
handling of the variable to not cause any globbing or other expansion
(also because fixing * isn't complete, there is also ? and sometimes ~
and other special characters might be expanded as well).
lib/tools.in