Re: [Puppet Users] 2.7.2rc changed the way variable names are parsed

2011-07-26 Thread Nigel Kersten
On Tue, Jul 26, 2011 at 8:21 AM, vagn scott wrote: > On 07/26/2011 10:54 AM, Nigel Kersten wrote: > >> >> I guess it's just a case of being a defensive shell programmer, but I've >> always considered curlies-with-double-quotes to be essential with all >> interpolated strings. >> > > A better way

Re: [Puppet Users] 2.7.2rc changed the way variable names are parsed

2011-07-26 Thread vagn scott
On 07/26/2011 10:54 AM, Nigel Kersten wrote: I guess it's just a case of being a defensive shell programmer, but I've always considered curlies-with-double-quotes to be essential with all interpolated strings. A better way to handle it would be to make curlies mandatory, and then change the

Re: [Puppet Users] 2.7.2rc changed the way variable names are parsed

2011-07-26 Thread Nigel Kersten
On Tue, Jul 26, 2011 at 7:52 AM, vagn scott wrote: > On 07/26/2011 10:32 AM, Nigel Kersten wrote: > >> >> Allowing a trailing hyphen doesn't seem like the most optimal solution >> though >> > That doesn't matter so much. But, expanding the character set for > variables without even mentioni

Re: [Puppet Users] 2.7.2rc changed the way variable names are parsed

2011-07-26 Thread vagn scott
On 07/26/2011 10:32 AM, Nigel Kersten wrote: Allowing a trailing hyphen doesn't seem like the most optimal solution though That doesn't matter so much. But, expanding the character set for variables without even mentioning it in the release notes has caused me some pain. You really sh

Re: [Puppet Users] 2.7.2rc changed the way variable names are parsed

2011-07-26 Thread Nigel Kersten
On Tue, Jul 26, 2011 at 5:29 AM, vagn scott wrote: > > In 2.7.2rc2 the '-' is now > parsed as part of the variable name. > Is this a deliberate change? > I'm trying to track down the actual bug where this got resolved, but yes, I believe this was deliberately done, and primarily because we allow